Changes by Guido van Rossum:
--
assignee: -> tiran
priority: -> high
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1628>
__
___
Python-bugs
Guido van Rossum added the comment:
> Here's an update version where the asyncore test server properly handles
> the EOF race condition.
Perfect! Check it in.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.
Guido van Rossum added the comment:
Agreed.
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1629>
__
___
Python-bugs-list
Changes by Guido van Rossum:
--
resolution: -> fixed
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1469>
__
___
Python
Guido van Rossum added the comment:
Focus on how using --with-wctype-functions changes things and how this
could affect the regex implementation. (I wouldn't be surprised if the
other failing tests were to to the regex bugs.)
__
Tracker <[EMAIL PROTECTED
Guido van Rossum added the comment:
This is what I was thinking of before, although I'd use "%.16g"%f and
"%.17g"%f instead of str(f) and repr(f), and I'd use float() instead
of eval().
I suspect that it doesn't satisfy Tim Peters though, because this may
de
Guido van Rossum added the comment:
[Tim: when I said "bugs" I just meant non-correct rounding. Sorry.]
On the educational issue: it's still embarrassingly easy to run into
situations where *arithmetic* using floats produces "educational"
results. Simplest case I co
Guido van Rossum added the comment:
I'm okay with adding this, it doesn't add much code and is properly
safeguarded by #ifdefs and has a configure.in patch.
I haven't reviewed the code though.
--
nosy: +gvanrossum
__
Tracker <[EMAIL
Guido van Rossum added the comment:
I don't see a patch. And I think you cannot do this without compromising
correctness, since _PyUnicode_AsDefaultEncodedString() returns the
cached value without incrementing its refcount. (The only refcount that
keeps it alive is the cache
Changes by Guido van Rossum:
--
resolution: -> accepted
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1629>
__
___
Python-bugs-list mailing li
Changes by Guido van Rossum:
--
status: pending -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1631>
__
___
Python-bugs-list mailing li
Guido van Rossum added the comment:
Let's not waste time in the bug tracker debugging some user's broken
setup. Let him contact Panda3D's customer support.
--
nosy: +gvanrossum
resolution: -> invalid
status: pending -> closed
_
Guido van Rossum added the comment:
i suggest abandoning any attempts at implementing math ourselves. I
also suggest not combining this with #1635 but reviewing and
(eventually) applying the latter first.
--
nosy: +gvanrossum
__
Tracker <[EM
Guido van Rossum added the comment:
Mostly looks good. Here are some nits.
(1) You shouldn't have to add pystrcmp.c to the VC project files since
on Windows it isn't used, right?
(2) Will the Windows input routine still accept the *old*
representations for INF and NAN? IMO that
Guido van Rossum added the comment:
This comes from line 1845 in ast.c.
It has nothing to do with execfile(), it's just a constraint on the
generated bytecode.
Fixing this would be quite complex, and referring to "this day and age"
isn't going to raise its priority. Howe
Guido van Rossum added the comment:
Would you mind submitting a proper patch for Python 2.5 and/or 2.6
generated by "svn diff" relative to an (anonymous) checkout, and adding
a unit test? Then I'd be happy to accept this and if it makes it in
time for the 2.5.2 release we&
Guido van Rossum added the comment:
Please backport.
--
assignee: -> tiran
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1638>
__
Guido van Rossum added the comment:
If you want attention, please post to python-dev if you didn't already.
Or widen the audience to python-list if you want to.
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python
Guido van Rossum added the comment:
Committed revision 59551.
--
nosy: +gvanrossum
resolution: -> accepted
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
You can fix this yourself:
for operation in list(self._operations.keys()):
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
Why not do the same for its cousin sys.setprofile()?
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
I suggest you check this in (with the Py_HUGE_VAL fix) and then see how
much of #1640 we still need.
--
resolution: -> accepted
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
Guido van Rossum added the comment:
> Historical note: Guido is probably thinking of "the old" pickle and
> marshal here, which did have problems with inf and NaN on Windows (as in
> they didn't work at all). Michael Hudson changed them to use special
> bit patter
Changes by Guido van Rossum:
--
keywords: +patch
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1549>
__
___
Python-bugs-list mailing list
Unsubs
Changes by Guido van Rossum:
--
priority: high -> normal
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1549>
__
___
Python-bugs-list mailing li
Guido van Rossum added the comment:
Here's an updated patch that applies smoothly to the current trunk.
--
components: +Extension Modules
Added file: http://bugs.python.org/file8988/python2.6-set_wakeup_fd4.diff
__
Tracker <[EMAIL PROTECTED
Guido van Rossum added the comment:
Sorry, the PEP was approved, just not yet marked as such. :-) Will do
so now.
--
nosy: +gvanrossum
resolution: postponed ->
status: pending -> open
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
BTW abc.py and _abcoll.py have been backported to 2.6 already, I propose
to backport numbers.py and test_abstract_numbers.py as well.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
To be honest, I have no idea what pybots is, and how it's affected.
Maybe you can get someone else interested in reviewing the code? I'm
focusing mostly on Py3k.
__
Tracker <[EMAIL PROTECTED]>
<htt
Guido van Rossum added the comment:
> FWIW, should PyUnicode_Check, PyTuple_Check etc. also change to
> PyUnicode_CHECK, PyTuple_CHECK etc because they are macros?
While this would technically be the right thing to do, those macros
have had a long and productive life before 2.6 so I'
Guido van Rossum added the comment:
Committed revision 59574.
I added a simple C API as well, PySignal_SetWakeupFd(fd).
Thanks all!
--
resolution: -> accepted
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Changes by Guido van Rossum:
--
assignee: -> gvanrossum
keywords: +patch
nosy: +gvanrossum
priority: -> normal
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
Sigh. I'll try to make time to review & apply this.
--
assignee: -> gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs
Guido van Rossum added the comment:
Martin, can you have a look at this?
Cartman, can you produce a unittest for the correct behavior that only
uses ASCII input (using \u instead of just typing Turkish characters)?
--
assignee: -> loewis
nosy: +loe
Guido van Rossum added the comment:
Hm. The test2.py file, when I download it, contains the two bytes
"\xc4\xb1" in the first unicode() call, and "\xc4\xb0" in the second
one. This is *always* supposed to produce a UnicodeDecodeError, since
it would use the default encoding
Guido van Rossum added the comment:
> Replacing Turkish characters with hex versions in test2.py still results
> in UnicodeDecodeError and works with python 2.4.
I'm hoping Martin can confirm this, but I suspect that this is due to
a tightening of the rules for converting from 8-bit
Guido van Rossum added the comment:
Committed revision 59576.
I found some time.
Happy pybotting!
--
resolution: -> accepted
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
I disagree that this would be harmless; this was excluded intentionally.
A (bound) method contains an instance which definitely represents state
and calling the method can easily mutate that state.
This is different from classes (which could contain state
Guido van Rossum added the comment:
I can't reproduce this.
How did you install Python?
Python 2.5.2a0 (release25-maint, Nov 27 2007, 14:00:00)
[GCC 4.0.3 (Ubuntu 4.0.3-1ubuntu5)] on linux2
Type "help", "copyright", "credits" or "lice
Guido van Rossum added the comment:
> print u"\u0069".upper()
>
> should give \u0130 (LATIN CAPITAL LETTER I WITH DOT ABOVE)
>
> print u"\u0049".lower()
>
> should give \u0131 (LATIN SMALL LETTER DOTLESS I)
>
> These transformations work fine with p
Guido van Rossum added the comment:
Then I suggest you talk to the Ubuntu developers or whoever packaged
Python 2.5.1 for it.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
> I tried downloading the latest rev of urlparse.py (59480) and it flunked
> its own unit test, "urlparse.test()" Two test cases fail.
That's not the official test -- that code should probably be deleted.
The real test is in Lib/
Guido van Rossum added the comment:
> But it should be affected by locale, thats the point of locale.setlocale
> call. This is how libc's wc functions behave.
No, the locale should only affect 8-bit string operations, never
unicode operations.
__
Trac
Guido van Rossum added the comment:
> Ok then what is the suggested way to get back the Turkish way of doing
> upper/lower on i & I ?
That's a question for Martin von Loewis. I suppose you could use 8-bit
strings exclusively. Or you could use .translate() wit
Guido van Rossum added the comment:
Martin, I'm asking you because according to svn blame I copied this from
the int/long integration branch last January.
I looked at the code of _PyLong_FitsInLong(), and I don't understand why
it wants an exact integer. The code
Guido van Rossum added the comment:
I'll take this off line.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1515>
__
___
Python-bugs-list mailing list
U
Guido van Rossum added the comment:
Well, if they do that, they apparently break inspect.py. Should they
be doing that? Why are they doing that? Can you contact the Ubuntu
folks about this?
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
Tickle the interest of one of the many folks with commit privileges.
Since 2.6 isn't going to be released for months there's no great hurry,
but waiting until the last minute would be a mistake.
__
Tracker <[EM
Guido van Rossum added the comment:
If there aren't too many differences between the 2.6 and 3.0 version of
decimal.py and your patch, do 2.6 first, then the next time we merge
stuff into 3.0 from the trunk it'll be forward-ported automatically.
Though you'd have to start
Guido van Rossum added the comment:
> Funnily,
>
> print "".encode("iso-8859-9").decode("iso-8859-9").upper()
>
> works, but
>
> print "".encode("iso-8859-9").upper().decode("iso-8859-9")
>
> not.
Guido van Rossum added the comment:
Maybe the original author (Barry Warsaw) can review this?
--
assignee: -> barry
nosy: +barry, gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
Guido van Rossum added the comment:
Two easy ways to get the functionality using 8-bit strings, assuming
you've already set your locale properly:
(1) If your data is already an 8-bit string (i.e. isinstance(data,
str)), simply use data.upper() or data.lower()
(2) If your data is Unicode
Guido van Rossum added the comment:
This can't be fixed directly -- the pthreads mutex is not an
interruptable system call.
However there's a simple (though expensive) work-around: use a very long
timeout. The timeout version of waiting for a lock is a busy-wait with a
short sleep
Guido van Rossum added the comment:
I think you misunderstand raw_input(). It just returns
sys.stdin.readline().rstrip("\n"). I don't think it's worth fixing.
The 'q' processing works fine as long as you didn't type \r first. :-)
--
nosy: +gvanros
Guido van Rossum added the comment:
I believe its original intent was to only ignore variables starting with
PYTHON, but I'm not 100% sure. Ignoring PATH seems counterintuitive.
I'm not sure about THREADDEBUG -- I'm thinking that variable should be
renamed PYTHONTHREADDEBUG?
BT
Guido van Rossum added the comment:
I agree with Tesiph, more useful behavior would be to raise an error
immediately because the argument is not a directory. If you wanted to
remove the think linked to, you could use rmtree("foo/.",
ignore_errors=True).
--
nosy: +
Guido van Rossum added the comment:
Fixed in 2.6.
Committed revision 59582.
3.0 will follow at the next merge.
--
keywords: +patch
nosy: +gvanrossum
resolution: -> accepted
status: open -> closed
versions: +Python 2.6 -Python 3.0
__
Tracker &
Guido van Rossum added the comment:
Very odd. Can you experiment with input() and sys.stdin.readline() to
see what exactly gets returned? It seems that somehow the CRLF -> LF
translation isn't working, and that should be fixed somewhere else,
not in license().
I imagine you could just
Guido van Rossum added the comment:
Thanks for the patch. I think it should raise IOError, not ValueError,
and it should use the onerror() handling used for all other errors.
Also, can you include an update for the docs in the Doc tree?
__
Tracker <[EM
Guido van Rossum added the comment:
> >>> len(x)
Please print repr(x). That should be safe even if it contains control
characters.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs
Guido van Rossum added the comment:
> >>> import sys
> >>> x = input()
> Hi:
> >>> repr(x)
> "'Hi:'"
> >>> x = sys.stdin.readline()
> Hi:
> >>> repr(x)
> "'Hi:\\n'"
Hm, that works a
Guido van Rossum added the comment:
Ah d'oh!
The 3.0 code was wrong. I've fixed it now in SVN by going back to input()
Committed revision 59583.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.p
Guido van Rossum added the comment:
I think we can fix this as follows: whenever it calls os.mkdir() and an
error is returned, check if that is EISDIR or EEXISTS, and if so, check
that indeed it now exists as a directory, and then ignore the error.
Moreover, I'd like to do this for the ult
Guido van Rossum added the comment:
Can you rephrase this as svn diff output?
Also, mkdir() is a confusing name for the helper -- I'd call it
forgiving_mkdir() or something like that.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python
Changes by Guido van Rossum:
--
assignee: -> gvanrossum
nosy: +gvanrossum
priority: -> normal
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1679>
__
Changes by Guido van Rossum:
--
assignee: -> gvanrossum
nosy: +gvanrossum
priority: -> normal
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1682>
__
Guido van Rossum added the comment:
On the one hand, if you want to format a float like that, you should use
"%.0f". On the other hand, this should either consistently fail or
succeed, not depending on how large the float is.
I think in 2.6 we can't change this, but in 3.0, I p
Guido van Rossum added the comment:
Committed revision 59636.
--
resolution: -> fixed
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
Committed revision 59646.
--
nosy: +gvanrossum
resolution: -> accepted
status: open -> closed
type: -> behavior
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.py
New submission from Guido van Rossum:
Should I apply this? There are more places that reference __cmp__ in
the library. OTOH there are some folks who would like to see __cmp__
make a come-back as a shorthand for defining 6 comparison operators, for
totally-ordered types. (I'm still wa
Guido van Rossum added the comment:
Go ahead then!
--
assignee: -> tiran
nosy: +gvanrossum
resolution: -> accepted
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
This should be fixed. Maarten, can you create a patch that works for
you? (Since few of us have access to the platform where it breaks.)
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.p
Guido van Rossum added the comment:
Maarten, can you build Python 2.6 from svn? (See
http://www.python.org/dev/faq/#subversion-svn for help.)
It looks like this has been fixed there, per r53434. Perhaps the same
changes can be backported.
__
Tracker <[EM
Changes by Guido van Rossum:
--
resolution: -> invalid
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1718>
__
___
Python
Guido van Rossum added the comment:
I don't see a bug here. It is as Amaury explains.
--
nosy: +gvanrossum
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pyt
Guido van Rossum added the comment:
Looks like a doc change that Georg could easily apply.
--
assignee: nnorwitz -> georg.brandl
nosy: +georg.brandl, gvanrossum
priority: -> low
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
I see this too. Maybe Fredrik understands?
--
assignee: -> effbot
nosy: +effbot, gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
Guido van Rossum added the comment:
Good catch!
Committed revision 59659.
I wonder how many other places have the same problem...
--
nosy: +gvanrossum
resolution: -> fixed
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://
Changes by Guido van Rossum:
--
assignee: -> vsajip
nosy: +vsajip
priority: -> low
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1709>
__
__
Guido van Rossum added the comment:
Actually, only the f.close() call should be conditionalized.
Does this patch fix the issue for you?
--
assignee: -> gvanrossum
nosy: +gvanrossum
priority: -> normal
Added file: http://bugs.python.org/file9040/distutil
Guido van Rossum added the comment:
site.py is already fixed in svn; it actually did cause problems, see
issue #1667.
I fixed pydoc.py.
Committed revision 59660.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
Guido van Rossum added the comment:
I'm not a big distutils expert myself, I can't quite interpret your
feedback. In dry-run mode, it shouldn't try to copy the script, correct?
--
keywords: +patch
__
Tracker <[EMAIL PROTECTED]>
&l
Guido van Rossum added the comment:
How about 2.6?
--
assignee: -> loewis
nosy: +gvanrossum, loewis
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.o
Guido van Rossum added the comment:
So that's correct behavior right? (I looked at the code for copy_file()
in cmd.py and it passes the dry_run parameter to the file utility
function that does the actual copying.
__
Tracker <[EMAIL PROTECTED
Changes by Guido van Rossum:
--
assignee: -> georg.brandl
nosy: +georg.brandl
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1722>
__
___
Python-bu
Guido van Rossum added the comment:
Committed revision 59668. (2.5 branch)
Committed revision 59669. (2.6 trunk)
--
resolution: -> fixed
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Guido van Rossum added the comment:
So what's the correct fix? Skip the whole block if size==0?
--
nosy: +gvanrossum
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python
Guido van Rossum added the comment:
I'll look at this when I have time. If you find someone else interested
in reviewing, please give them the patch!
--
assignee: -> gvanrossum
keywords: +patch
nosy: +gvanrossum
priority: -> low
__
Trac
Guido van Rossum added the comment:
Yup. Same if the exception comes in a file. I'll try to free up some
cycles to look into this. Thanks for the report!
--
assignee: -> gvanrossum
nosy: +gvanrossum
__
Tracker <[EMAIL PROTEC
Guido van Rossum added the comment:
Could you please add a unit test for this? Then I can check it in.
I'm thinking this could safely be fixed in 2.5.2.
--
assignee: -> gvanrossum
nosy: +gvanrossum
resolution: -> accepted
__
Tracker <[E
Changes by Guido van Rossum:
--
resolution: -> wont fix
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1205>
__
___
Python
Changes by Guido van Rossum:
--
nosy: -gagenellina
resolution: -> invalid
status: open -> closed
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1723>
__
New submission from Guido van Rossum:
Since 1e-1000 == 0.0 (on an IEEE-754 platform anyway), I would expect
-1e-1000 to be -0.0. But it does not appear that way:
>>> 1e-1000
0.0
>>> -0.0
-0.0
>>> -1e-1000
0.0
However (correctly):
>>> (-1.0) * 1e-1000
-0.
Guido van Rossum added the comment:
BTW this was on x86 Linux. On x86 OSX (10.4, Tiger) it works correctly:
>>> -1e-1000
-0.0
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.py
Guido van Rossum added the comment:
One nit: you added a blank line to the end of test_math.py.
This will cause the checkin to fail. :-)
One bigger issue: the sign() function doesn't seem to work properly for
nans. E.g. on Linux I get:
>>> inf = 1e1000
>>> nan =
Guido van Rossum added the comment:
Can you produce a version of the patch that applies cleanly to 2.6? Then
we can apply it there first and merge into 3.0 later.
But I wonder. It seems this changes every single call to
PyImport_ImportModule() in the core to _PyImport_ImportModuleNoLock
Guido van Rossum added the comment:
I'm just following the last post's suggestion "Should we close this then?"
My message (somebody "should" submit a patch) was sarcastic --- it was
in reference to the comment that Python "should" be more practical.
Sinc
Guido van Rossum added the comment:
I think I understand what's going on. The trail leads from the last "if
(tty) {" block in builtin_input() to PyOS_Readline() which in turn ends
up calling PyOS_StdioReadline() (because that's the most likely
initialization of PyOS_Rea
Guido van Rossum added the comment:
> > One nit: you added a blank line to the end of test_math.py.
> > This will cause the checkin to fail. :-)
>
> *grr* stupid white space check hook
No, you edited a line that didn't need editing. :-)
> > One bigger issue: the si
Guido van Rossum added the comment:
I see. I think there's still something uncomfortable with this function
though -- it looks in sys.modules, but if it doesn't find it there, it
invokes the full-blown import machinery, which calls the (possibly
overridden) __import__ builtin, whi
1201 - 1300 of 5563 matches
Mail list logo