Changes by Alexander Belopolsky :
--
title: Smelly exports -> Smelly exports (global symbols in python not prefixed
with Py or _Py)
___
Python tracker
<http://bugs.python.org/iss
Alexander Belopolsky added the comment:
> In other words, if iadd can deal with it, why can't add?
Should ['a', 'b', 'c'] + 'def' return ['a, 'b', 'c', 'd' , 'e', 'f'] o
Alexander Belopolsky added the comment:
> I don't understand is why a += b is different from a + b
> in this respect.
When a is mutable, a += b updates it in-place, so there is no ambiguity: the
type of a cannot change. When you do a + b, there is no reason to treat a as
more des
Changes by Alexander Belopolsky :
--
type: -> feature request
versions: +Python 3.2 -Python 2.6
___
Python tracker
<http://bugs.python.org/issue9314>
___
_
Alexander Belopolsky added the comment:
Note that immutable types are consistent:
>>> x = tuple('abc')
>>> x += 'def'
Traceback (most recent call last):
File "", line 1, in
TypeError: can only concatenate tuple (not "str") to tupl
Changes by Alexander Belopolsky :
--
assignee: -> belopolsky
resolution: -> rejected
stage: -> committed/rejected
status: open -> pending
___
Python tracker
<http://bugs.pytho
Alexander Belopolsky added the comment:
This is clearly an artifact of porting to 3.x. In 2.x the code was
calls = self.calledfuncs.keys()
calls.sort()
for filename, modulename, funcname in calls:
which was translated to
for filename
Alexander Belopolsky added the comment:
> why the pending status?
To me this is a way to say "I will close this next time I look unless someone
will change my mind." "pending" has a nice property that it will change to
"open" once someone adds a comment.
Alexander Belopolsky added the comment:
Committed with a minor change to meet 80-character line limit.
See r82997 (r82999 for 3.1).
--
resolution: accepted -> fixed
stage: commit review -> committed/rejected
status: open -> closed
_
Alexander Belopolsky added the comment:
> I thought it was common to requalify a bug report into a doc bug.
That's upto you and OP. My rule of thumb is if the title reads as either code
or doc bug, it's ok to reclassify. If not, it is better to open a se
New submission from Alexander Belopolsky :
I am running the following command
$ python -m trace -C pickle-trace.d -c -m Lib/test/test_pickle.py
and getting attached file, pickle.cover, in the pickle-trace.d directory. This
does not look right. From the very beginning, module level
Changes by Alexander Belopolsky :
--
nosy: +eli.bendersky, tjreedy
___
Python tracker
<http://bugs.python.org/issue9317>
___
___
Python-bugs-list mailin
Alexander Belopolsky added the comment:
Does the trace module even work in 3.x?
I created attached trivial traceme.py file.
With python 2.7, I get
$ python2 -m trace -c -s traceme.py
lines cov% module (path)
1 100% threading (Lib/threading.py)
6 100% traceme
Alexander Belopolsky added the comment:
On Tue, Jul 20, 2010 at 11:28 PM, Eli Bendersky wrote:
> The problem indeed seems to be deeper.
>
.. or shallower. :-)
--
___
Python tracker
<http://bugs.python.org/
Alexander Belopolsky added the comment:
Here is another problem: with no file x in cwd,
$ ./python.exe -m trace -c -f x traceme.py
Skipping counts file 'x': [Errno 2] No such file or directory: 'x'
This one seems to exist in both 2.7 and 3.x. Creating emp
Alexander Belopolsky added the comment:
Comparison of count files produced by 2.7 and 3.x suggests that the problem is
in finding the source file for the module:
>>> pickle.load(open('x', 'rb'))
({('', 2): 2, ('', 1): 4, ('', 8): 20,
Alexander Belopolsky added the comment:
On Wed, Jul 21, 2010 at 12:13 AM, Eli Bendersky wrote:
..
> I'm investigating further, but this may very well be caused by different
> behavior
> of `f_code.co_filename` between Python 2 and 3.
I am afraid I am a step ahead of you.
Alexander Belopolsky added the comment:
I am attaching a proof-of-concept patch, issue9317.diff, which fixes the file
name problem at the expense of more spurious modules:
$ ./python.exe -m trace -c -s traceme.py
lines cov% module (path)
7 100% codecs (Lib/codecs.py)
10
Alexander Belopolsky added the comment:
Eli,
Your new patch makes perfect sense, but can you check of regular
python invocation uses runpy these days. If it does, it may make sense
to include it in trace at least optionally.
Also, does this fix the original problem?
--
nosy
Alexander Belopolsky added the comment:
On Jul 21, 2010, at 9:22 AM, Eli Bendersky
wrote:
>
> Eli Bendersky added the comment:
>
> <<<<
> Your new patch makes perfect sense, but can you check of regular
> python invocation uses runpy these days. If it does,
Alexander Belopolsky added the comment:
On Wed, Jul 21, 2010 at 9:45 AM, Eli Bendersky wrote:
..
> As far as I understand, when you run:
>
> py3d -m trace -C pickle-trace.d -c -m test_pickle.py
>
> The first -m flag applies to the trace module. Python uses
> runpy.run_modu
Alexander Belopolsky added the comment:
Eli,
At this point, I think it will be best to focus on unittests. I am adding
#9315 as a dependency here and will add a comment there. I think once we have
good unittest coverage the bug will become obvious.
--
dependencies: +The trace
Alexander Belopolsky added the comment:
Also, when you get a chance, please separate the __main__ coverage bug into a
separate issue and bring it up on python-dev. We should get profile guys
involved and I don't want them to be distracted by trace specific i
Alexander Belopolsky added the comment:
I am adding 2.7. It is ok to add tests to the stable series release AFAIK.
Moreover, I believe the unittests should be written for 2.7 first. Since 3.x
port of trace was done without the benefit of a test suite, it is likely that
there are many
Alexander Belopolsky added the comment:
On Wed, Jul 21, 2010 at 2:09 AM, Ray.Allen wrote:
..
> Does this means "a += b" is not the same as "a = a + b"?
For immutable a, the two are practically the same, for mutable, they
are necessarily different. This is explained in
Alexander Belopolsky added the comment:
[Copying msg111059 from issue9317]
On Wed, Jul 21, 2010 at 9:45 AM, Eli Bendersky wrote:
..
> As far as I understand, when you run:
>
>py3d -m trace -C pickle-trace.d -c -m test_pickle.py
>
> The first -m flag applies to the trace
Alexander Belopolsky added the comment:
Committed with a minor change. (There is no need to keep the source file open
during the trace run.) See r83035 (r83037 in release31-maint.)
I will open a separate issue to discuss a possible RFE related to msg111074.
--
resolution: -> fi
Changes by Alexander Belopolsky :
--
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue9314>
___
___
Python-bugs-list mailing list
Un
Alexander Belopolsky added the comment:
Interesting. As far as I can tell, struct timeval should not be used unless
HAVE_GETTIMEOFDAY is defined:
+#ifdef HAVE_GETTIMEOFDAY
+typedef struct timeval _PyTime_timeval;
+#else
+typedef struct {
+time_t tv_sec; /* seconds since Jan. 1
Alexander Belopolsky added the comment:
Is anyone interested in giving this a final review? I would like to check this
in unless someone objects or needs time to review.
--
___
Python tracker
<http://bugs.python.org/issue7
Alexander Belopolsky added the comment:
I think Reid nailed the issue. Fix is coming.
--
___
Python tracker
<http://bugs.python.org/issue9079>
___
___
Python-bug
Alexander Belopolsky added the comment:
issue9079b.diff should fix the Windows issue.
--
Added file: http://bugs.python.org/file18108/issue9079b.diff
___
Python tracker
<http://bugs.python.org/issue9
Alexander Belopolsky added the comment:
What about PC/VS7.1/pythoncore.vcproj? Is there some automation in place to
keep the project files in sync? ISTM, all the info to generate these should be
in setup.py.
The patch looks good, but I am hesitant to commit changes that I cannot test
Alexander Belopolsky added the comment:
On Wed, Jul 21, 2010 at 3:58 PM, Tim Lesher wrote:
..
> To date, I believe that the attitude toward these older build files has been
> "unsupported; use at own risk; patches welcome".
>
OK, I guess there is little risk in committing t
New submission from Alexander Belopolsky :
The -m interpreter option allows one to run library module as a script, but if
you want to debug, profile or trace the execution of the same, you must supply
the path to the module source file on the command line. The resulting
execution may also be
Changes by Alexander Belopolsky :
--
nosy: +eli.bendersky
___
Python tracker
<http://bugs.python.org/issue9325>
___
___
Python-bugs-list mailing list
Unsubscribe:
Alexander Belopolsky added the comment:
The documentation should mention somewhere that timeout can be a float. For
example, as in time.sleep docstring:
"""
sleep(seconds)
Delay execution for a given number of seconds. The argument may be
a floating p
Alexander Belopolsky added the comment:
s/Note that a result Fraction/Note that as a result, Fraction/
--
___
Python tracker
<http://bugs.python.org/issue5
New submission from Alexander Belopolsky :
Unlike sys.settrace, sys.setprofile is not described as CPython implementation
detail in Doc/library/sys.rst.
--
assignee: d...@python
components: Documentation
messages: 99
nosy: belopolsky, d...@python
priority: normal
severity: normal
Alexander Belopolsky added the comment:
Is anyone interested in reviewing this patch before it is committed? Since
there are no user-visible changes and the only non-trivial change simply adds
new tests, I think this can go in. Any refinements can be done later.
--
components
Alexander Belopolsky added the comment:
It looks like OP has lost interest and I don't have time to move it forward.
--
assignee: belopolsky ->
priority: normal -> low
stage: needs patch -> patch review
___
Python tracker
<http:
Changes by Alexander Belopolsky :
--
assignee: belopolsky ->
___
Python tracker
<http://bugs.python.org/issue1738>
___
___
Python-bugs-list mailing list
Un
Alexander Belopolsky added the comment:
Mark,
Do you agree that conditions mentioned in msg109329 and msg109340 are never
triggered?
--
___
Python tracker
<http://bugs.python.org/issue9
Alexander Belopolsky added the comment:
This looks like something that can become part of PEP 3151.
--
assignee: belopolsky ->
nosy: -Alexander.Belopolsky
stage: -> patch review
___
Python tracker
<http://bugs.python.org/
Alexander Belopolsky added the comment:
> [datetime.c] needs to be renamed in Modules/Setup.dist, and most
> importantly in PC/config.c
Fixed in issue7989d.diff, thanks.
In order to commit this patch I need an SVN advise. I would like to copy
datetime.py from sandbox to py3k in a wa
Alexander Belopolsky added the comment:
s/strictly necessary/not strictly necessary/
--
___
Python tracker
<http://bugs.python.org/issue7989>
___
___
Python-bug
Changes by Alexander Belopolsky :
Removed file: http://bugs.python.org/file17913/issue9206a.diff
___
Python tracker
<http://bugs.python.org/issue7989>
___
___
Python-bug
Alexander Belopolsky added the comment:
I can reproduce this in Apple's idle, but not in trunk or 2.7 versions. I'll
leave it open in case Ronald is interested. Antlong also reports that this
happens on windows, but I cannot verify that.
Here is my session copied from idle
Alexander Belopolsky added the comment:
Here is a simpler test: in idle2.6,
>>> '\xff'.isalpha()
True
but in idle2.7 and plain python prompt, it is False.
--
___
Python tracker
<http://bug
Alexander Belopolsky added the comment:
Here is a way to reproduce this from command line:
$ python2.6
Python 2.6.5 (r265:79359, Mar 24 2010, 01:32:55)
[GCC 4.0.1 (Apple Inc. build 5493)] on darwin
Type "help", "copyright", "credits" or "license"
Alexander Belopolsky added the comment:
Or even simpler:
$ python2.6
Python 2.6.5 (r265:79359, Mar 24 2010, 01:32:55)
[GCC 4.0.1 (Apple Inc. build 5493)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>
Alexander Belopolsky added the comment:
This is clearly a Tkinter rather than Mac issue, so I am unassigning this from
Ronald. This appears to be the same problem as the one Mark described in
msg102301.
>>> import locale
>>> locale.nl_langinfo(locale.CODESET)
'US-ASC
Changes by Alexander Belopolsky :
--
nosy: +mark.dickinson
___
Python tracker
<http://bugs.python.org/issue9335>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Alexander Belopolsky :
--
assignee: ronaldoussoren -> belopolsky
___
Python tracker
<http://bugs.python.org/issue9335>
___
___
Python-bugs-list mai
Alexander Belopolsky added the comment:
In 3.x, it is different:
>>> locale.nl_langinfo(locale.CODESET)
'UTF-8'
Victor,
This looks like your cup of tee.
--
nosy: +haypo
___
Python tracker
<http://bug
Alexander Belopolsky added the comment:
Since I am the OP of this patch, I would like a +1 from another developer
before checking this in. (Or a couple of -1s before rejecting:-)
Antoine,
Does the latest patch address your concerns?
Virgil,
If you care enough about this feature, please
Alexander Belopolsky added the comment:
On Fri, Jul 23, 2010 at 4:03 AM, Martin v. Löwis
wrote:
..
> I fail to see the bug in this report. '\xff' is a letter because the C
> library says it is.
This does not explain the difference between 2.6 and 2.7. With
attached i
Alexander Belopolsky added the comment:
Another issue that may be worth revisiting is whether or not it is OK for
_tkinter to set the locale.
"""
21.2.2. For extension writers and programs that embed Python
Extension modules should never call setlocale(), except to find out wh
Changes by Alexander Belopolsky :
Added file: http://bugs.python.org/file18148/datetimetester.py
___
Python tracker
<http://bugs.python.org/issue7989>
___
___
Python-bug
Alexander Belopolsky added the comment:
> This might be caused by the fix for issue7072.
Ronald,
You are absolutely right. Reverting r80178 in the trunk restores the old
behavior.
> msg103494:
> Fixed in r80178 (trunk), r80180 (2.6), r80182 (3.2), r80183 (3.1)
I think this can
Changes by Alexander Belopolsky :
--
assignee: -> belopolsky
components: +Extension Modules
stage: -> needs patch
type: -> feature request
___
Python tracker
<http://bugs.python.o
Alexander Belopolsky added the comment:
Accepting binary input where only letters are expected by an application is a
very common source of security holes. An application that relies on
s.isalpha() to guarantee that s does not contain non-ASCII characters when
UTF-8 locale is in use, may
Alexander Belopolsky added the comment:
I'll take a look.
Mark,
"commit review" is a stage after a patch was reviewed by a committer and is
deemed appropriate. Usually this comes with an "accepted" resolution.
Sometimes "commit review" is skipped for si
Alexander Belopolsky added the comment:
Brian, thanks for the fix and for testing. I am attaching a commit-ready patch
issue7989e.diff, that includes Brian's fix and a few white-space changes.
I hope I've resolved the SVN issue: I was working in a read-only checkout
while sandbo
Alexander Belopolsky added the comment:
Ron,
Can you add a Misc/NEWS entry summarizing your change? Also, please check if
any changes need to be made to ReST documentation, Doc/library/pydoc.rst .
Ka-Ping,
Do you want to hold on to this, you can I take it over?
--
nosy
New submission from Alexander Belopolsky :
This bikeshed have been repainted several times already, but here is a true
story.
Cast:
Me = myself
Novice = a 14-year-old boy
Laptop = a MacBook Air
Novice: How do i exit [from python prompt]?
Me: What's your best guess?
Novice: [typing]
Alexander Belopolsky added the comment:
Committed in r83112.
--
resolution: accepted -> fixed
stage: commit review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
Alexander Belopolsky added the comment:
On Fri, Jul 23, 2010 at 4:08 PM, Ron Adam wrote:
..
> Here's the new patch with the Misc/NEWS and pydoc.rst additions added to it.
s/Romoved/Removed/
--
___
Python tracker
<http://bugs.python.or
Alexander Belopolsky added the comment:
s/navagation/navigation/
Please spell-check your changes.
--
___
Python tracker
<http://bugs.python.org/issue2
Alexander Belopolsky added the comment:
+:program:`pydoc` :option:`-g` will start the server and additionally open a web
+browser to a module index page. Each served page has a navagation bar at the
+top where you can 'get' help on a individual item, 'find' all modules with
Alexander Belopolsky added the comment:
Ron,
Your latest patch does not work for me:
$ ./python.exe -m pydoc -g
Traceback (most recent call last):
File "/Users/sasha/Work/python-svn/py3k-commit/Lib/runpy.py", line 160, in
_run_module_as_main
"__main__", fname, load
Alexander Belopolsky added the comment:
> I assume that this is on *nix boxes
It's a bit of a stretch to call OSX unix, but yes:
>> Laptop = a MacBook Air
If you have not seen one of these, the control key is marked "contro
Alexander Belopolsky added the comment:
Now, as I look at it, the windows message is probably even worse. What is
"plus"? Is "Ctrl-Z" control *minus* z?
For a novice, you really have to say: press Ctrl and Z keys together and then
press Return or Enter key. Of course t
Alexander Belopolsky added the comment:
> On another note: I've sometimes wanted quit() to exit from a running
> script to the python console so I can do a little value checking.
This is what -i command line switch or PYTHONINSPECT environment variable is
for.
See pytho
Alexander Belopolsky added the comment:
Hmm. Still no luck
$ ./python.exe -m pydoc -g
Server ready at: http://localhost:7464/
Segmentation fault
The Segmentation fault happens after I enter pydoc in the search window and
press the "Search!" button.
BTW, please remove !'s
Alexander Belopolsky added the comment:
OK, the crash is due to issue9319, but as far as I understand, the faulty code
is in the error handling branch, so there must be a bug in your code as well.
--
___
Python tracker
<http://bugs.python.
Alexander Belopolsky added the comment:
> FWIW, I agree with Antoine. You cannot know in advance whether a
> partial-subclass has semantics that need to be preserved when
> flattening.
Raymond,
I have actually conceded this point to Antoine. See msg108980 above. Not only
the lat
Alexander Belopolsky added the comment:
The patch passes tests on OSX, but the old code passes the new tests as well,
so I am not sure what this patch fixes anymore. Looking back through the
messages I see that I had trouble reproducing the problem, then I succeeded,
but I don't rem
Alexander Belopolsky added the comment:
Lib/test/test_trace.py is now moved to Lib/test/test_sys_settrace.py.
3.2: r83140 r83141
3.1: r83143
2.7: r83142
--
___
Python tracker
<http://bugs.python.org/issue9
Alexander Belopolsky added the comment:
Eli,
test_trace_module.py is a good start and I would like to commit it soon. I
have a few nitpicks and a suggestion.
1. pprint module is not used in the tests so it should not be imported.
2. It is better to do run_unittest(__name__) in test_main
New submission from Alexander Belopolsky :
Attached patch makes python -m pickle unpickle the first object
from the pickle file and display it using pprint. Future enhancements may
include printing the summary (object number, object type) of the multiobject
pickles, option to control
Changes by Alexander Belopolsky :
--
nosy: +alexandre.vassalotti, mark.dickinson, pitrou
___
Python tracker
<http://bugs.python.org/issue9378>
___
___
Python-bug
Changes by Alexander Belopolsky :
--
stage: needs patch -> patch review
___
Python tracker
<http://bugs.python.org/issue7447>
___
___
Python-bugs-list mai
Alexander Belopolsky added the comment:
A nitpick:
"lol" is a very well-known acronym and "list of lists" is not the expansion
that first comes to mind.
--
nosy: +belopolsky
___
Python tracker
<http://bu
Alexander Belopolsky added the comment:
FWIW, I like the new patch better, but still have a few nitpicks:
- Starting a sentence with an argument name is a bit awkward because that makes
a sentence that starts with a lower case letter.
- There is an extra space in :class: `list`.
- It
Alexander Belopolsky added the comment:
On Sun, Jul 25, 2010 at 7:27 PM, STINNER Victor wrote:
..
> Unicode is my cup of tee, but not programs considering that bytes are
> characters.
>
What I called "your cup of tee" was 3.x returning 'UTF-8' from
locale.nl_lan
Alexander Belopolsky added the comment:
I will defer to your judgement on the level of "whiteness" that is appropriate.
I have passed your patch through 2to3, replaced test_support with support and
it looks like we have a test case for a regress
Alexander Belopolsky added the comment:
On Mon, Jul 26, 2010 at 9:48 AM, anatoly techtonik
wrote:
..
> Type exit() or press Ctrl-D to exit.
Anatoly,
I think you missed the point of the story that I posted. The young
user, who was not completely new to computers, by the way, was not
famil
Alexander Belopolsky added the comment:
On Mon, Jul 26, 2010 at 10:50 AM, Mark Lawrence wrote:
..
> 2) Python is not responsible for dotting every i and crossing every t for a
> computer novice.
Yes, but it is not responsible for teaching a novice more than one way
to exit the inter
Alexander Belopolsky added the comment:
On Mon, Jul 26, 2010 at 11:45 AM, Mark Lawrence wrote:
>
> Mark Lawrence added the comment:
>
> Could we (easily) add a third line at startup that says how to quit?
+1
Alternatively, we can add the second line to help output:
Typ
New submission from Alexander Belopolsky :
I believe any Tkinter app will exhibit the same behavior, but I discovered this
with idle and verified with wish.py:
$ ./python.exe Demo/tkinter/guido/wish.py
% quit
Traceback (most recent call last):
File "Demo/tkinter/guido/wish.py", l
Alexander Belopolsky added the comment:
Reopening as a documentation issue:
- manual entry should have a versionchanged
- use *arg* instead of 'arg'
- add a ReST entry describing -m pickletools
--
components: +Documentation
nosy: +ezio.melotti
status: clos
Alexander Belopolsky added the comment:
On Mon, Jul 26, 2010 at 12:49 AM, Eli Bendersky wrote:
..
> This raises a question: should method names of classes come after the class
> names when appearing
> in a trace? Intuitively yes, but I want to investigate some more, probably by
&
New submission from Alexander Belopolsky :
Since tkinter is a package, it requires __main__.py in order to be runnable
using -m option. Attached tkinter-m.patch fixes this.
--
assignee: belopolsky
components: Tkinter
files: tkinter-m.py
keywords: easy
messages: 111654
nosy: belopolsky
Changes by Alexander Belopolsky :
--
keywords: +needs review
stage: -> patch review
versions: +Python 3.2
___
Python tracker
<http://bugs.python.org/iss
Alexander Belopolsky added the comment:
On Jul 26, 2010, at 4:32 PM, anatoly techtonik
wrote:
>
> Why not to ship it in Python by default?
>
Because it is under GPL?
--
nosy: +Alexander.Belopolsky
___
Python tracker
<http://bug
Alexander Belopolsky added the comment:
On Mon, Jul 26, 2010 at 4:16 PM, Éric Araujo wrote:
..
> I lack context: Why is being able to run “python -m tkinter” useful?
The missing context is in issue 9384. I discovered that python -m
tkinter did not work when I was looking for a simple way
Alexander Belopolsky added the comment:
issue9387a.diff includes a note in library/tkinter.rst. The note can probably
be improved.
--
keywords: +patch
Added file: http://bugs.python.org/file18217/issue9387a.diff
___
Python tracker
<h
Changes by Alexander Belopolsky :
--
resolution: -> accepted
stage: patch review -> commit review
___
Python tracker
<http://bugs.python.org/issue9387>
___
__
Alexander Belopolsky added the comment:
This is probably Tk issue, but there might me a way that tkinter can work
around it. On OSX 10.6, the pre-installed /usr/bin/wish shows the same
behavior, but mac ports' /opt/local/bin/wish works
2901 - 3000 of 3596 matches
Mail list logo
