[issue18468] re.group() should never return a bytearray

2013-07-25 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
nosy: +serhiy.storchaka

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18468
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18549] Dead code in socket_ntohl()

2013-07-25 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Agree, it can be removed.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18549
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18418] Thread.isAlive() sometimes True after fork

2013-07-25 Thread Charles-François Natali

Charles-François Natali added the comment:

 Bump.

Did you see my review at http://bugs.python.org/review/18418/#ps8668 ?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18418
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18549] Dead code in socket_ntohl()

2013-07-25 Thread Christian Heimes

Christian Heimes added the comment:

Thanks for the review!

--
resolution:  - fixed
stage: needs patch - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18549
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18549] Dead code in socket_ntohl()

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset b7ea3f94f6ca by Christian Heimes in branch '3.3':
Issue #18549: Eliminate dead code in socket_ntohl().
http://hg.python.org/cpython/rev/b7ea3f94f6ca

New changeset 2a59428dbff5 by Christian Heimes in branch 'default':
Issue #18549: Eliminate dead code in socket_ntohl().
http://hg.python.org/cpython/rev/2a59428dbff5

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18549
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18542] httpd memory consuption increasing continously due to mod_python

2013-07-25 Thread Ronald Oussoren

Changes by Ronald Oussoren ronaldousso...@mac.com:


--
stage:  - committed/rejected
status: pending - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18542
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18491] Add exe wrapper functionality to Windows launcher

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 4123e002a1af by Vinay Sajip in branch 'default':
Closes #18491: Added script-wrapper functionality to launcher source (but not 
to executable).
http://hg.python.org/cpython/rev/4123e002a1af

--
nosy: +python-dev
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18491
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18387] Add 'symbols' link to pydoc's html menu bar.

2013-07-25 Thread Ronald Oussoren

Changes by Ronald Oussoren ronaldousso...@mac.com:


--
nosy: +ronaldoussoren

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18387
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18418] Thread.isAlive() sometimes True after fork

2013-07-25 Thread A. Jesse Jiryu Davis

A. Jesse Jiryu Davis added the comment:

Oh, no I didn't. I'm learning how this system works. I'll respond there
soon.

Thanks

On Thursday, July 25, 2013, Charles-François Natali wrote:


 Charles-François Natali added the comment:

  Bump.

 Did you see my review at http://bugs.python.org/review/18418/#ps8668 ?

 --

 ___
 Python tracker rep...@bugs.python.org javascript:;
 http://bugs.python.org/issue18418
 ___


--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18418
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18553] os.isatty() is not Unix only

2013-07-25 Thread anatoly techtonik

New submission from anatoly techtonik:

It seems like os.isatty(0) works on Windows too. Documentation says Unix only:

  http://docs.python.org/2/library/os.html#os.isatty
  http://docs.python.org/3.4/library/os.html#os.isatty


C:\py -c import os; print os.isatty(0)
True

C:\echo x | py -c import os; print os.isatty(0)
False

C:\py -c import os; print os.isatty(0) | more
True

--
components: Library (Lib)
messages: 193694
nosy: techtonik
priority: normal
severity: normal
status: open
title: os.isatty() is not Unix only
versions: Python 2.7, Python 3.1, Python 3.2, Python 3.3, Python 3.4, Python 3.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18553
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18553] os.isatty() is not Unix only

2013-07-25 Thread anatoly techtonik

Changes by anatoly techtonik techto...@gmail.com:


--
assignee:  - docs@python
components: +Documentation
nosy: +docs@python

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18553
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18534] State clearly that open() 'file' param is name attr of the result

2013-07-25 Thread Nick Coghlan

Nick Coghlan added the comment:

Switching this to a docs bugs, since 
http://docs.python.org/3/library/functions.html#open doesn't mention this 
behaviour at all, and http://docs.python.org/3/library/io.html#io.FileIO.name 
only notes the fact it may be a file descriptor without mentioning the 
str/bytes discrepancy.

So, at a bare minimum, we need to clearly describe this behaviour in the docs. 
We may also want to explicitly point out that using os.fsdecode(name) before 
passing it to open() will ensure that the name attribute is set to a string 
rather than a bytes object.

--
assignee:  - docs@python
components: +Documentation
nosy: +docs@python
title: File name attribute should always be a text string - State clearly 
that open() 'file' param is name attr of the result
versions: +Python 3.3, Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18534
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18554] os.__all__ is incomplete

2013-07-25 Thread Ronald Oussoren

New submission from Ronald Oussoren:

On Unix the __all__ of the os module doesn't list the names imported from the 
posix module. 

The attached patch ensures that exports from the posix module are added to 
__all__, just like they are for NT and for Python 2.7.

--
components: Library (Lib)
files: incomplete-os-all.txt
keywords: needs review, patch
messages: 193696
nosy: ronaldoussoren
priority: normal
severity: normal
stage: patch review
status: open
title: os.__all__ is incomplete
type: behavior
versions: Python 3.3, Python 3.4
Added file: http://bugs.python.org/file31033/incomplete-os-all.txt

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18554
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18538] `python -m dis ` relying on argparse

2013-07-25 Thread Antoine Pitrou

Changes by Antoine Pitrou pit...@free.fr:


--
nosy: +ncoghlan

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18538
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue4079] new urllib2.Request 'timeout' attribute needs to have a default

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Thanks, but the new test doesn't fail.  With your test patch, 
test_default_values fails, but that is explicitly checking for the 'timeout' 
attribute, which isn't actually part of the public API.  The new test passes 
even without the fix.

Reading the original issue, it seems like the problem is a bit more subtle.  To 
quote that issue:

 * API weirdness: Only some, special, urllib2.Request objects may be
passed to handlers, because the constructor does not create objects with
a .timeout.

The original report here says the problem occurs if a subclass overrides 'open' 
(like Mechanize does).  So that problem will only occur if a request gets 
passed to a handler without having passed through 'open'.  But there is a 
question whether there is a way for the code that assumes the timeout attribute 
exists to fail that doesn't involve an override of 'open'.  The answer may be 
no, but I haven't traced the logic or reviewed the API, so I don't know.

However, it is clear that if open is overridden, and the code doing so doesn't 
care about timeouts and so doesn't do the attribute set itself, subsequent code 
will sometimes fail because it assumes that the attribute does exist.  We could 
write a test that does that kind of override on 'open', and causes one of those 
code paths that assumes timeout exists to be called.

But...I have some question, now, just how real an issue this is...if you 
override open, you really should be either calling it via super or replicating 
its functionality...so if there is not in fact any way to cause the lack of a 
timeout attribute to cause a failure without overriding open, there may not be 
a real issue here.  Presumably mechanize ran into it because a keyword 
argument (new feature) was added.  So the mechanize bug was really that there 
was a backward compatibility issue between 2.6 and previous versions.  Is that 
still an issue for 2.7?  Perhaps: there could still be code someone will 
forward port to 2.7 that overrides open and doesn't know about timeout.  So it 
is worth fixing, since the fix is simple.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4079
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18503] No assertion in test_del_param_on_nonexistent_header function and unused variables in some places in test_email.py

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Thanks, Vajrasky.

--
resolution:  - fixed
stage:  - committed/rejected
status: open - closed
type:  - behavior
versions: +Python 3.3

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18503
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18503] No assertion in test_del_param_on_nonexistent_header function and unused variables in some places in test_email.py

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 61d9c561b63d by R David Murray in branch '3.3':
#18503: small cleanups in test_email.
http://hg.python.org/cpython/rev/61d9c561b63d

New changeset be5f1f0bea09 by R David Murray in branch 'default':
#18503: small cleanups in test_email.
http://hg.python.org/cpython/rev/be5f1f0bea09

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18503
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

The patch looks good to me.  Can you add a pickling test?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18036] How do I create a .pyc file? FAQ entry is out of date

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Well, the FAQ entry is why didn't a .pyc file get created, so presumably the 
querant knows enough to be looking for one to get created, and so presumably 
knows more-or-less what it is.

It seems to me that your rewording loses the sense of answering the question, 
and instead becomes more of a mini-tutorial on what a .pyc file is.

The changes needed here, it seems to me, are to talk about the __pycache__ 
directory instead of the .pyc file.  In python2, the issue would be that there 
was no write access to the source directory, and thus the .pyc file could not 
be created.  Now that same lack of access will first of all result in the 
__pycache__ directory not being created.

There is also then a second level: if the __pycache__ directory already exists 
but is not writable when a new module is first run, the .pyc file will not get 
created.  But I bet almost no one will notice that, and thus it won't be a FAQ 
:)  

Still, that issue could be mentioned...but then you also need to mention the 
full, two-part, version-specific extension, since one has to know which .pyc 
file to expect, if there are multiple versions of python on the system.  That 
mention could just be a pointer to the relevant docs, though, since most users 
will only be dealing with one python3 version.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18555] type_set_bases() doesn't check return value of PyArg_UnpackTuple()

2013-07-25 Thread Christian Heimes

New submission from Christian Heimes:

http://hg.python.org/cpython/file/be5f1f0bea09/Objects/typeobject.c#l554

CID 486265 (#1 of 1): Unchecked return value (CHECKED_RETURN)
19. check_return: Calling function PyArg_UnpackTuple(PyObject *, char const *, 
Py_ssize_t, Py_ssize_t, ...) without checking return value (as is done 
elsewhere 64 out of 65 times).
PyArg_UnpackTuple(PyList_GET_ITEM(temp, i),
  , 2, 2, cls, mro);

--
components: Interpreter Core
messages: 193702
nosy: christian.heimes
priority: normal
severity: normal
stage: needs patch
status: open
title: type_set_bases() doesn't check return value of  PyArg_UnpackTuple()
type: behavior
versions: Python 3.3, Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18555
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18556] ctypes' U_set() doesn't check return value of PyUnicode_AsWideChar()

2013-07-25 Thread Christian Heimes

New submission from Christian Heimes:

http://hg.python.org/cpython/file/be5f1f0bea09/Modules/_ctypes/cfield.c#l1235

CID 486657 (#1 of 1): Unchecked return value (CHECKED_RETURN)
5. check_return: Calling function PyUnicode_AsWideChar(PyObject *, wchar_t *, 
Py_ssize_t) without checking return value (as is done elsewhere 7 out of 8 
times).
1263PyUnicode_AsWideChar(value, (wchar_t *)ptr, size);

--
components: Extension Modules
messages: 193703
nosy: christian.heimes
priority: low
severity: normal
stage: needs patch
status: open
title: ctypes' U_set() doesn't check return value of PyUnicode_AsWideChar()
type: behavior
versions: Python 3.3, Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18556
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17545] os.listdir and os.path.join inconsistent on empty path

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Is your patch backward?

Treat the path naturally assume everyone agrees with you as to what is 
natural.  It would be better, I think, to be explicit about it.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17545
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue15130] remove redundant paragraph in socket howto

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 68fce7587f72 by R David Murray in branch '3.3':
#15130: remove repeat of abstract paragraph from socket howto body.
http://hg.python.org/cpython/rev/68fce7587f72

New changeset be4cc946b9df by R David Murray in branch 'default':
Merge #15130: remove repeat of abstract paragraph from socket howto body.
http://hg.python.org/cpython/rev/be4cc946b9df

New changeset a8d8bfb61874 by R David Murray in branch '2.7':
#15130: remove repeat of abstract paragraph from socket howto body.
http://hg.python.org/cpython/rev/a8d8bfb61874

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15130
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue15130] remove redundant paragraph in socket howto

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Thanks, Tshepang.

--
nosy: +r.david.murray
resolution:  - fixed
stage:  - committed/rejected
status: open - closed
type:  - behavior

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15130
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18557] email address parser desired

2013-07-25 Thread Lukas Pirl

New submission from Lukas Pirl:

I am missing a non-hidden equivalent of email._parseaddr.AddressList.
AddressList was retained from the former rfc822 module and is still used 
internally.

Yes, there is email.utils but for what I know, with the provided functionality 
it is not possible to parse a comma separated list of email addresses (ex. 
a...@example.com, b...@example.com).

Since RFC5322 allow commas in the local part [1], one cannot simply split a 
list of email addresses by comma and use email.utils.getaddresses.

[1] http://tools.ietf.org/html/rfc5322#section-3.2.3

--
components: email
messages: 193707
nosy: barry, lpirl, r.david.murray
priority: normal
severity: normal
status: open
title: email address parser desired
type: enhancement
versions: Python 3.3

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18557
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread Claudiu.Popa

Claudiu.Popa added the comment:

Here's the new patch. Also, I noticed a test failing when running ./python -m 
test, pyclbr, complaining about _Aifc_params not present in some dict, but I 
don't really know how to fix it..

--
Added file: http://bugs.python.org/file31034/aifc_4.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18418] Thread.isAlive() sometimes True after fork

2013-07-25 Thread A. Jesse Jiryu Davis

A. Jesse Jiryu Davis added the comment:

New patch for 3.3 branch after Charles-François's critique: instead of changing 
startup sequence in Thread._bootstrap_inner, stop all threads in _limbo after a 
fork.

--
Added file: http://bugs.python.org/file31035/issue18418-2.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18418
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18418] Thread.isAlive() sometimes True after fork

2013-07-25 Thread A. Jesse Jiryu Davis

A. Jesse Jiryu Davis added the comment:

(Sorry about the extraneous XML file from my IDE, I made a mistake and allowed 
the diff to include it.)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18418
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18557] email address parser desired

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

No, but you can do this:

   utils.getaddresses(['a...@example.com, b...@example.com'])
  [('', 'a...@example.com'), ('', 'b...@example.com')]

Not the worlds most intuitive API, I know...which is why we have implemented a 
new improved API.  In 3.3, by using the new (provisional) policies, you can get 
a parsed list of addresses right from the Message object, without having to do 
any further processing yourself:

   import email.policy
   m = email.message_from_string('To: a...@example.com, 
b...@example.com\n\n', policy=email.policy.default)
   m['to']
  'a...@example.com, b...@example.com'
   m['to'].addresses
  (Address(display_name='', username='a', domain='example.com'), 
Address(display_name='', username='b', domain='example.com'))

This new code is not as battle tested as the old code, so I need people to try 
using it in order to find the bugs :)

--
resolution:  - out of date
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18557
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18476] No way to pass custom arguments for loggers and formatters.

2013-07-25 Thread Piotr Dobrogost

Changes by Piotr Dobrogost p...@bugs.python.dobrogost.net:


--
status: pending - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18476
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18557] email address parser desired

2013-07-25 Thread Lukas Pirl

Lukas Pirl added the comment:

Ah, obviously! :)

Thanks for your rapid and enlightening answer. Now, at least, this feature is 
documented and can be found by others.

Best!

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18557
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

pyclbr is parsing the source code, and since _Aifc_params is not a class, it 
does not get detected.  So we just need to add it to the ignore list in the 
pyclbr test.

I'm also getting this when I run the test with your patch applied:

/home/rdmurray/python/p34/Lib/unittest/case.py:496: ResourceWarning: unclosed 
file _io.BufferedReader name='@test_9764_tmp'
  testMethod()

Looks like you are missing a close or two.

Oh, and it occurs to me that _Aifc_params doesn't follow pep8.  aifc should 
either be all lower case or, if you are viewing it as a class, it should be 
_AIFCParams.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread Claudiu.Popa

Claudiu.Popa added the comment:

Here's the new modifications.

--
Added file: http://bugs.python.org/file31036/aifc_5.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18556] ctypes' U_set() doesn't check return value of PyUnicode_AsWideChar()

2013-07-25 Thread Matthew Barnett

Matthew Barnett added the comment:

Re msg193703: A little before that, 'value' is INCREF'ed, and then:

wstr = PyUnicode_AsUnicodeAndSize(value, size);
if (wstr == NULL)
return NULL;

Shouldn't 'value' be DECREF'ed before returning?

--
nosy: +mrabarnett

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18556
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 560c6e9d1beb by R David Murray in branch 'default':
#17818: aifc.getparams now returns a namedtuple.
http://hg.python.org/cpython/rev/560c6e9d1beb

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17899] os.listdir() leaks FDs if invoked on FD pointing to a non-directory

2013-07-25 Thread Larry Hastings

Larry Hastings added the comment:

Okay, this bug has dragged on long enough.  Serhiy already said they looked 
good to him, and I am now declaring that that's good enough for me.

I'll check in my patches Saturday-ish unless someone pipes up.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17899
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17818] aifc.Aifc_read/Aifc_write.getparams can return a namedtuple

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

Thanks, Claudiu.

--
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18545] enum always runs member_type when use_args is True

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 95e1d0efd896 by Ethan Furman in branch 'default':
Close #18545: now only executes member_type if no _value_ is assigned in 
__new__.
http://hg.python.org/cpython/rev/95e1d0efd896

--
nosy: +python-dev
resolution:  - fixed
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18545
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18556] ctypes' U_set() doesn't check return value of PyUnicode_AsWideChar()

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 78e8980ec9f7 by Brett Cannon in branch '3.3':
Issue #18556: Check the return value for PyUnicode_AsWideChar() in
http://hg.python.org/cpython/rev/78e8980ec9f7

New changeset 2f4c4db9aee5 by Brett Cannon in branch 'default':
merge for issue #18556
http://hg.python.org/cpython/rev/2f4c4db9aee5

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18556
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue486657] small edit in ext/ownershipRules.html

2013-07-25 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 78e8980ec9f7 by Brett Cannon in branch '3.3':
Issue #18556: Check the return value for PyUnicode_AsWideChar() in
http://hg.python.org/cpython/rev/78e8980ec9f7

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue486657
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18556] ctypes' U_set() doesn't check return value of PyUnicode_AsWideChar()

2013-07-25 Thread Brett Cannon

Brett Cannon added the comment:

Thanks for the check, Matthew, but if you look at the other *_set() methods 
they do not necessarily DECREF their 'value' argument. Plus if 
PyUnicode_AsWideChar() fails it means memory is full or a fatal error has 
triggered, so there are bigger worries. =)

--
nosy: +brett.cannon
resolution:  - fixed
stage: needs patch - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18556
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18558] Iterables not detected correctly

2013-07-25 Thread Stephen Paul Chappell

New submission from Stephen Paul Chappell:

The following interactive session shows that iterables are not detected 
properly by the `collections.abc.Iterable` class.

 class IsIterable:
def __init__(self, data):
self.data = data
def __getitem__(self, key):
return self.data[key]

 is_iterable = IsIterable(range(5))
 for value in is_iterable:
value


0
1
2
3
4
 from collections.abc import Iterable
 isinstance(is_iterable, Iterable)
False

--
components: Library (Lib)
messages: 193723
nosy: Zero
priority: normal
severity: normal
status: open
title: Iterables not detected correctly
type: behavior
versions: Python 3.3

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18558
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18558] Iterable glossary entry needs clarification

2013-07-25 Thread R. David Murray

R. David Murray added the comment:

The definition of an Iterable is a class that defines an __iter__  method.  
Your class does not, so the behavior you site is correct.

The glossary entry for 'iterable' could use a little clarification.  A class 
that defines __getitem__ is an iterable if and only if it returns results when 
passed integers.  Since the documentation for Iterable references that glossary 
entry, it should probably also be explicit that defining __getitem__ does not 
(because of the forgoing limitation) cause isinstance(x, Iterable) to be True.  
For a class that does not define __iter__, you must explicitly register it with 
Iterable.

To see why this must be so, consider this:

   y = IsIterable({'a': 'b', 'c': 'd'})
   [x for x in y]
  Traceback (most recent call last):
File stdin, line 1, in module
File stdin, line 1, in listcomp
File stdin, line 5, in __getitem__
  KeyError: 0

--
assignee:  - docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python, r.david.murray
stage:  - needs patch
title: Iterables not detected correctly - Iterable glossary entry needs 
clarification
versions: +Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18558
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8079] make install fails with -j8 with python2.6/config on FreeBSD

2013-07-25 Thread Garrett Cooper

Garrett Cooper added the comment:

Feel free to close this. I don't have any interest in pursuing it further.

--
nosy: +Garrett.Cooper

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8079
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com