[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Christoph Zwerschke
Christoph Zwerschke <[EMAIL PROTECTED]> added the comment: The unspecific error message is thrown by the vgetargskeywords() function, so changing this behavior would be a larger, more difficult and general issue (similar to #1283289). We could go another path and require an object for 'reverse' i

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Andrii V. Mishkovskyi
Changes by Andrii V. Mishkovskyi <[EMAIL PROTECTED]>: -- nosy: +mishok13 ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Patches are welcome. -- priority: -> low resolution: wont fix -> status: closed -> open ___ Python tracker <[EMAIL PROTECTED]> __

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Christoph Zwerschke
Christoph Zwerschke <[EMAIL PROTECTED]> added the comment: The problem is not only that the error message "TypeError: an integer is required" has "integer" instead of "boolean", but it does not mention the attribute name "reverse", i.e. it does not even say *where* the integer is required. I firm

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Benjamin Peterson
Changes by Benjamin Peterson <[EMAIL PROTECTED]>: -- resolution: -> wont fix status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Well, booleans technically are integers. -- nosy: +benjamin.peterson ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue3354] sort(reverse=None) prints misleading error message

2008-07-14 Thread Christoph Zwerschke
New submission from Christoph Zwerschke <[EMAIL PROTECTED]>: When you sort a list with list.sort() or sorted(list), and set the reverse parameter to None, then you get the following misleading error message: TypeError: an integer is required I would expect a more proper error message for th