[issue20525] Got compiler warning when compiling readline module

[Vajrasky Kok]

New submission from Vajrasky Kok:

On Fedora 20, I got this compiler warning:

/home/sky/Code/python/cpython3.4/Modules/readline.c: In function 
‘flex_complete’:
/home/sky/Code/python/cpython3.4/Modules/readline.c:962:5: warning: passing 
argument 1 of ‘completion_matches’ discards ‘const’ qualifier from pointer 
target type [enabled by default]
 result = completion_matches(text, *on_completion);
 ^
/home/sky/Code/python/cpython3.4/Modules/readline.c:40:15: note: expected ‘char 
*’ but argument is of type ‘const char *’
 extern char **completion_matches(char *, rl_compentry_func_t *);
   ^

Here is the patch using Christian Heimes' strategy in silencing compiler 
warning in dbm module.
http://hg.python.org/cpython/rev/3068ff3d9d1e

--
components: Extension Modules
files: fix_compile_warning_readline.patch
keywords: patch
messages: 210364
nosy: vajrasky
priority: normal
severity: normal
status: open
title: Got compiler warning when compiling readline module
type: behavior
versions: Python 3.4
Added file: http://bugs.python.org/file33938/fix_compile_warning_readline.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16074] Bad error message in os.rename, os.link, and os.symlink

[Larry Hastings]

Larry Hastings added the comment:

I probably should have continued with this issue instead of creating a new 
issue for it, sorry.  But there's a new issue tracking this change, where I 
propose to add new error-reporting functions that support two filenames.  
That's happening in #20517.

--
superseder:  -> Support errors with two filenames for errno exceptions

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


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

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Roundup Robot]

Roundup Robot added the comment:

New changeset 305b6a5da852 by Serhiy Storchaka in branch '2.7':
Issue #20520: Fixed readline test in test_codecs.
http://hg.python.org/cpython/rev/305b6a5da852

New changeset 82d374a9bbc7 by Serhiy Storchaka in branch '3.3':
Issue #20520: Fixed readline test in test_codecs.
http://hg.python.org/cpython/rev/82d374a9bbc7

New changeset 1457fa0da200 by Serhiy Storchaka in branch 'default':
Issue #20520: Fixed readline test in test_codecs.
http://hg.python.org/cpython/rev/1457fa0da200

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20167] Exception on IDLE closing

[Terry J. Reedy]

Terry J. Reedy added the comment:

More thoughts: The reason for explicit __del__ is to release memory resources 
connected to other objects or structures at the behest of the Python instance 
being deleted. If __del__ were only called at shutdown, it would not be needed, 
as all memory is released anyway and we could delete __del__. If __del__ is 
called before shutdown (as would happen if the class were unittested) then 
exceptions would indicate a bug and should not be ignored. From this viewpoint, 
not only should the try: except: be within the loop, but the break should be 
conditioned on the exception message.

  if exc.args[0] == 'can't invoke "bind" command:  application has been 
destroyed': break

The modern solution to this dilemma is to use .close instead of .__del__ and 
either explicitly call .close or have it called in the .__exit__ method of a 
context manager. For Idle classes, this would not be a problem for future test 
code, but it would be much trickier finding all places in current code where 
instances of a class with a .__del__ method get deleted. So a compromise patch 
with conditioned break seems most practical.

I tried a few more experiments: run the .py file (F5, to create a Shell window) 
and then close both windows. Whether the closing was shell first or editor 
first, the messages do not appear. Three runs with -n gave the same results. So 
it seems that the problem only appears when there has never been a shell 
window. I agree with Serhiy (his point 2) that this is disconcerting and could 
indicate a 3.4-specific problem with Tk, tkinter, or Idle.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20518] multiple inheritance + C extension = possibly unexpected __base__

[Eric Snow]

Eric Snow added the comment:

So to restate, where some class Spam inherits from multiple classes and at 
least one was written in C, Spam.__base__ may have an unexpected value.

> So, to conclude: it is sometimes not possible to use python
> cooperative multiple inheritance if C base classes are involved. This
> is a bummer.

Be careful not to muddy the waters here by obscuring the problem you are 
describing (regarding __base__) with the challenges of making multiple 
inheritance work.  Multiple inheritance with classes that don't cooperate in 
your hierarchy is tricky, but solveable and is simply the nature of the beast.  
To move this issue forward I recommend simply focusing on how __base__ (and its 
use) could be improved.

I think part of the problem is that the metaclass of PyQt4 uses the __base__ 
class attribute instead of the MRO...


Cooperative multiple inheritance is all about the classes involved cooperating. 
 You already indicated that the QT classes do not cooperate.  To work around 
this you could wrap QT objects with proxies that *do* cooperate in your 
multiple inheritance scheme.  Alternately you could fiddle around in __init__ 
to make it work.  Raymond Hettinger had a great talk on this topic at Pycon 
2012 (or was it 2013) that would be worth checking out.  I remember him 
discussing strategies for fitting uncooperative classes into a multiple 
inheritance hierarchy.


--
components: +Extension Modules -Interpreter Core
nosy: +eric.snow
title: Weird behavior with multiple inheritance when C classes involved -> 
multiple inheritance + C extension = possibly unexpected __base__

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20521] [PATCH] Cleanup for "dis" module documentation

[Antoine Pitrou]

Changes by Antoine Pitrou :


--
versions: +Python 2.7, Python 3.3

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20524] format error messages should provide context information

[Eric V. Smith]

Eric V. Smith added the comment:

int, float, str, and complex are the types formatted by that code.

Notice that Decimal already has a better message:

>>> format(Decimal(42), 'tx')
Traceback (most recent call last):
  ...
ValueError: Invalid format specifier: tx

>>> format(42, 'tx')
Traceback (most recent call last):
  ...
ValueError: Invalid conversion specification

But, look at this:
>>> format(3, '--')
Traceback (most recent call last):
  File "", line 1, in 
ValueError: Unknown format code '-' for object of type 'int'

This is generated in unknown_presentation_type, also in formatter_unicode.c. It 
almost does what you want, but just handles the presentation type, not the 
whole format specifier.

Error handling could be cleaned up in that module. I'd say that the string 
should be:
" with format specifier  for object of type ''"

 might be "Unknown presentation type '-'", or "Cannot specify 
','".

I think that would require some major surgery to the code, but would be worth 
it.

Note that in your original example, you want the error to contain 
"{length:%HH:%MM}". By the time the error is detected, the only thing the code 
knows is the format specifier "%HH:%MM". It doesn't know the "length" part. The 
error is basically in int.__format__. By the time that gets called, the format 
specifier has already been extracted and the argument selection (by indexing, 
by name, including attribute access) has already taken place.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Ned Deily]

Changes by Ned Deily :


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

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Roundup Robot]

Roundup Robot added the comment:

New changeset 74c7a6fd8b45 by Ned Deily in branch '2.7':
Issue #20374: delete spurious empty line
http://hg.python.org/cpython/rev/74c7a6fd8b45

New changeset 6616c94d6149 by Ned Deily in branch '3.3':
Issue #20374: delete spurious empty line
http://hg.python.org/cpython/rev/6616c94d6149

New changeset 0b91e764b889 by Ned Deily in branch 'default':
Issue #20374: merge
http://hg.python.org/cpython/rev/0b91e764b889

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Roundup Robot]

Roundup Robot added the comment:

New changeset 0b5b0bfcc7b1 by Ned Deily in branch '2.7':
Issue #20374: Avoid compiler warnings when compiling readline with libedit.
http://hg.python.org/cpython/rev/0b5b0bfcc7b1

New changeset 9131a9edcac4 by Ned Deily in branch '3.3':
Issue #20374: Avoid compiler warnings when compiling readline with libedit.
http://hg.python.org/cpython/rev/9131a9edcac4

New changeset 0abf103f5559 by Ned Deily in branch 'default':
Issue #20374: merge
http://hg.python.org/cpython/rev/0abf103f5559

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Benjamin Peterson]

Benjamin Peterson added the comment:

That is sure ugly, but okay. :)

--
assignee: benjamin.peterson -> ned.deily

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Ned Deily]

Changes by Ned Deily :


Added file: http://bugs.python.org/file33937/issue20374_no_cast_3x.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20374] Failure to compile with readline-6.3-rc1

[Ned Deily]

Ned Deily added the comment:

Using Serhiy's suggestion, here are patches for 2.7/3.3 and default that 
condition the hook function signatures.  With these patches, all three branches 
compile without warnings and pass test_readline using OS X libedit, readline 
6.3-rc2, and an older readline.

--
resolution: fixed -> 
stage: committed/rejected -> patch review
status: closed -> open
Added file: http://bugs.python.org/file33936/issue20374_no_cast_33.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20524] format error messages should provide context information

[Eric V. Smith]

Eric V. Smith added the comment:

That would be a great improvement. It's in Python/formatter_unicode.c, line 
245, in parse_internal_render_format_spec().

That code knows about the format spec, but not the type being formatted. That 
would be easy enough to pass in.

This fix would only work for the built-in types: int, float, and str, I think. 
Maybe complex. But that's probably good enough.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

[Guido van Rossum]

Guido van Rossum added the comment:

Correct, if you want to work on it, see 
http://code.google.com/p/tulip/issues/detail?id=114

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20524] format error messages should provide context information

[Ezio Melotti]

Changes by Ezio Melotti :


--
nosy: +ezio.melotti
stage:  -> test needed
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

[Yury Selivanov]

Yury Selivanov added the comment:

This one is still not merged in Tulip, right?

--
nosy: +yselivanov

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20524] format error messages should provide context information

[R. David Murray]

New submission from R. David Murray:

Consider the following:

   '{run_time:%H:%M:%S}, 
,COM,DA{id},"{title:.43}",{id},{length:%M:%S}'.format(**mydict)

The error message I got was:

   Invalid format specifier

The problem turned out to be that the value of the 'length' key was an integer 
instead of a datetime.time(), but it sure wasn't easy to figure out which bit 
of the format string or which variable was the problem.

It would be nice for the format error message to include the pattern that it is 
parsing when it hits the error.  The type of the value being substituted would 
also be nice.  Perhaps something like:

   The format specifier in {length:%HH:%MM} is not valid for type int()

--
components: Library (Lib)
messages: 210351
nosy: eric.smith, r.david.murray
priority: normal
severity: normal
status: open
title: format error messages should provide context information

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Zachary Ware added the comment:

Ok, current status of this issue as I see it:

  Objects/typeobject.c:part of conglomerate patch, LGTM
  Objects/longobject.c:awaiting revision based on review
  Objects/listobject.c:part of conglomerate patch, LGTM
  Objects/floatobject.c:   part of conglomerate patch, two minor issues, 
otherwise LGTM
  Modules/resource.c:  part of conglomerate patch, LGTM (could use return 
converter, not essential)
  Modules/_threadmodule.c: untouched, held til 3.5
  Modules/_bz2module.c:untouched, held til 3.5
  Modules/_bisectmodule.c: untouched, held til 3.5
  Python/marshal.c:part of conglomerate patch, LGTM
  Objects/exceptions.c:untouched, held til 3.5
  Modules/nismodule.c: untouched, held til 3.5
  Modules/gcmodule.c:  awaiting revision based on review
  Python/_warnings.c:  untouched, held til 3.5
  Modules/_cryptmodule.c:  done in another issue

I'd like someone with more C experience to have a look at typeobject, 
floatobject, listobject, resource, and marshal to make sure I haven't missed 
anything (or led Vajrasky astray).  If someone else can ok them, I can get them 
committed.

Vajrasky: What's the status on gcmodule and longobject?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20523] global .pdbrc on windows 7 not reachable out of the box

[Ned Deily]

Changes by Ned Deily :


--
nosy: +georg.brandl

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20137] Logging: RotatingFileHandler computes string length instead of byte representation length.

[Vinay Sajip]

Changes by Vinay Sajip :


--
resolution:  -> invalid
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20136] Logging: StreamHandler does not use OS line separator.

[Vinay Sajip]

Changes by Vinay Sajip :


--
resolution:  -> invalid
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20523] global .pdbrc on windows 7 not reachable out of the box

[mbyt]

New submission from mbyt:

The global .pdbrc file is determined by the %HOME% environment variable. 
However, this is not available out of the box on e.g. windows 7 systems. Here 
only %HOMEDRIVE% and %HOMEPATH% are defined.

Thus the usual approach to have a global .pdbrc file on windows is to define a 
%HOME% environment variable by hand. This could be avoided if the global .pdbrc 
would be determined by os.path.expanduser("~/.pdbrc"), which works on current 
windows and does the magic behind.

There are two possible approaches to improve this situation:
* explicitly mention in the docs that on windows a %HOME% varialbe need to be 
created manually
* patch pdb.py to use os.path.expanduser instead (see attached diff)

For reference, see also old discussion 
https://mail.python.org/pipermail/python-list/2005-October/349550.html.

--
files: pdb.diff
keywords: patch
messages: 210349
nosy: mbyt
priority: normal
severity: normal
status: open
title: global .pdbrc on windows 7 not reachable out of the box
type: enhancement
Added file: http://bugs.python.org/file33935/pdb.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20137] Logging: RotatingFileHandler computes string length instead of byte representation length.

[A. Libotean]

A. Libotean added the comment:

> Which encoding are you using, such that the difference in length between 
> encoded and decoded messages is significant?

I agree right off the bat that the error in size is not significant. Only the 
length of the last appended line is computed erroneously.

I'm uploading the patch in case you decide to apply it.

Thanks.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20136] Logging: StreamHandler does not use OS line separator.

[A. Libotean]

A. Libotean added the comment:

You can go ahead and close this.

I ran some tests and concluded that indeed the IO system will take care of the 
line separators.

Sorry to have wasted your time.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] Replace uuid ctypes usage with an extension module.

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

See also issue5885.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue14515] tempfile.TemporaryDirectory documented as returning object but returns name

[R. David Murray]

R. David Murray added the comment:

Just ran into this again :).  Committed a patch with wording clarified based on 
Serhiy's feedback (I say 'target of the as clause of the with statement'.)

--
stage: patch review -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue14515] tempfile.TemporaryDirectory documented as returning object but returns name

[Roundup Robot]

Roundup Robot added the comment:

New changeset b5fe07d39e16 by R David Murray in branch '3.3':
#14515: clarify that TemporaryDirectory's __enter__ returns the name.
http://hg.python.org/cpython/rev/b5fe07d39e16

New changeset 7b7e17723787 by R David Murray in branch 'default':
#14515: clarify that TemporaryDirectory's __enter__ returns the name.
http://hg.python.org/cpython/rev/7b7e17723787

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20505] Remove resolution from selectors and granularity from asyncio

[Charles-François Natali]

Charles-François Natali added the comment:

> To solve a performance issue in asyncio, I added a new resolution
attribute to selectors.BaseSelector and a new _granularity attribute to
asyncio.BaseEventLoop. If I understood correctly, Charles-François (author
and so maintainer of the new selectors module) doesn't like the new
resolution attribute because it is a lie (if I understood correctly
Charles-François complain).

Here are the reasons I don't like this attribute:
- it's a low-level implementation detail, which shouldn't be part of the API
- it's actually impossible to provide a sensible value for the granularity,
because it depends not only on the operating system, but also on the actual
version on the OS, and the hardware, as Victor's extensive tests showed
- what's the definition of this "granularity" anyway?
- but most importantly, it's useless: the performance problem initially
identified was due to the rounding of select/epoll/etc of timeouts towards
zero, which means that it they got passed a timeout smaller than the
syscall resolution, the syscall would end up being called in a tight loop.
Now that the timeouts are rounded away from 0, that's not an issue anymore

Let me state this last point once again: no busy loop can occur now that
timeouts are rounded up.
Sure, some syscalls, on some OS, can sometimes return a little bit earlier
than expected, e.g. epoll can return after 0.98ms instead of 1ms. But
that's not an issue, if that's the case you'll just loop an extra time,
*all* event loops just work this way.
Furthermore, note than due to rounding, the actual probability of having
the syscall return earlier than expected is really low:
for example, if the loop wants to wake up in 5.3ms, it will be rounded to
6ms, and even a slight early wakeup won't be enough to wake up before 5.3ms
(typically it'll return after 5.98ms or so).

 > The patch is a comprise, it solves partially the asyncio performance
issue. With the patch, BaseEventLoop._run_once() may not executed a task
even if there was a scheduled task, just because of timing rounding issues.
So I modified the existing patch to tolerate "useless calls" to _run_once().

If you want to keep the current approach, nothing prevents from using a
fixed "slack" value, independant of the selector (e.g. 1ms seems
reasonable).

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20501] fileinput module will read whole file into memory when using fileinput.hook_encoded

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

As far as the issue is in the codecs module, this is 2.7 only issue, because 
codecs.open is used only in 2.7. And I suppose that replacing codecs.open by 
io.open should fix this issue.

--
keywords: +patch
nosy: +benjamin.peterson, hynek, pitrou, serhiy.storchaka, stutzbach
stage:  -> patch review
versions:  -Python 3.3, Python 3.4
Added file: http://bugs.python.org/file33934/fileinput_hook_encoded.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

[Antoine Pitrou]

Antoine Pitrou added the comment:

Hi Ram,
sorry, I'll take a look at your patch soon.
(as you know, we're in feature freeze right now so your patch must wait for 3.4 
to be released, anyway)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20521] [PATCH] Cleanup for "dis" module documentation

[Berker Peksag]

Changes by Berker Peksag :


--
nosy: +ncoghlan
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


--
Removed message: http://bugs.python.org/msg210340

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33562/clinic_typeobject.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33475/clinic_resource.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33641/clinic_listobject_v2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33612/clinic_listobject.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33773/clinic_floatobject_v2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33778/issue20185_conglomerate.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33887/issue20185_conglomerate_v2.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33769/clinic_marshal_v4.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33643/clinic_floatobject.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


--
nosy: +larry, nadeem.vawda, serhiy.storchaka, vajrasky

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33479/clinic_marshal.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33678/clinic_marshal_v3.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33744/clinic_resource_v2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33644/clinic_marshal_v2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Zachary Ware added the comment:

Clearing nosy for file cleaning, will restore:
larry,nadeem.vawda,serhiy.storchaka,vajrasky,zach.ware

--
nosy:  -larry, nadeem.vawda, serhiy.storchaka, vajrasky

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33746/clinic_typeobject_v2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Changes by Zachary Ware :


Removed file: http://bugs.python.org/file33767/clinic_listobject_v3.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue12691] tokenize.untokenize is broken

[Yury Selivanov]

Yury Selivanov added the comment:

Gareth,

Thanks a lot for such a comprehensive writeup and the patch. Please give me a 
day or two to do the review.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue19920] TarFile.list() fails on some files

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Thank you for your patch Vajrasky.

--
resolution:  -> fixed
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue19920] TarFile.list() fails on some files

[Roundup Robot]

Roundup Robot added the comment:

New changeset a5895fca91f3 by Serhiy Storchaka in branch '3.3':
Issue #19920: TarFile.list() no longer fails when outputs a listing
http://hg.python.org/cpython/rev/a5895fca91f3

New changeset 077aa6d4f3b7 by Serhiy Storchaka in branch 'default':
Issue #19920: TarFile.list() no longer fails when outputs a listing
http://hg.python.org/cpython/rev/077aa6d4f3b7

New changeset 48c5c18110ae by Serhiy Storchaka in branch '2.7':
Issue #19920: Added tests for TarFile.list(). Based on patch by Vajrasky Kok.
http://hg.python.org/cpython/rev/48c5c18110ae

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue12691] tokenize.untokenize is broken

[Gareth Rees]

Changes by Gareth Rees :


Removed file: http://bugs.python.org/file33919/Issue12691.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1028088] Cookies without values are silently ignored (by design?)

[Berker Peksag]

Berker Peksag added the comment:

This was fixed in issue 16611 (for 3.3 and 3.4) and there is a open issue for 
2.7: issue 19870. I'm closing this one as a duplicate of issue 19870, because 
it has a patch.

>>> from http import cookies
>>> C = cookies.SimpleCookie()
>>> C.load("chips=ahoy; vienna=finger; secure")
>>> print(C)
Set-Cookie: chips=ahoy
Set-Cookie: vienna=finger; secure
>>> C['vienna']['secure']
True

--
nosy: +berker.peksag
resolution:  -> duplicate
stage:  -> committed/rejected
status: open -> closed
superseder:  -> Backport Cookie fix to 2.7 (httponly / secure flag)

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20345] Better logging defaults

[Vinay Sajip]

Changes by Vinay Sajip :


--
status: pending -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20485] Enable non-ASCII extension module names

[Stefan Behnel]

Changes by Stefan Behnel :


--
nosy: +scoder

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20485] Enable non-ASCII extension module names

[Jeremy Kloth]

Changes by Jeremy Kloth :


--
nosy: +jkloth

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] Replace uuid ctypes usage with an extension module.

[R. David Murray]

R. David Murray added the comment:

Thanks.  This looks like a good idea, but I'll leave it to someone with more 
experience with this module to review it.

Let's change this issue to an enhancement request for uuid instead.  If someone 
wants to work on the cycle-in-ctypes problem they can open a new issue.

--
nosy: +ned.deily, serhiy.storchaka
stage:  -> patch review
title: ctypes.create_string_buffer creates reference cycles -> Replace uuid 
ctypes usage with an extension module.
type: resource usage -> enhancement
versions: +Python 3.5 -Python 2.7, Python 3.3, Python 3.4

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7718] Build shared libpythonX.Y.so on IRIX

[Berker Peksag]

Berker Peksag added the comment:

According to the python-dev discussion, Irix is not supported anymore:

https://mail.python.org/pipermail/python-dev/2009-February/086111.html

--
nosy: +berker.peksag
resolution:  -> out of date
stage:  -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20485] Enable non-ASCII extension module names

[Arfrever Frehtes Taifersar Arahesis]

Changes by Arfrever Frehtes Taifersar Arahesis :


--
nosy: +Arfrever

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue7717] Compilation fixes for IRIX

[Berker Peksag]

Berker Peksag added the comment:

According to the python-dev discussion, Irix is not supported anymore:

https://mail.python.org/pipermail/python-dev/2009-February/086111.html

--
keywords:  -needs review
nosy: +berker.peksag
resolution:  -> out of date
stage: patch review -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Walter Dörwald]

Walter Dörwald added the comment:

True, here's an updated patch.

--
Added file: http://bugs.python.org/file33933/fix_linetests2.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20485] Enable non-ASCII extension module names

[Brett Cannon]

Changes by Brett Cannon :


--
stage:  -> test needed
title: Enable 'import .pyd' -> Enable non-ASCII extension module 
names

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] ctypes.create_string_buffer creates reference cycles

[Gustavo J. A. M. Carneiro]

Gustavo J. A. M. Carneiro added the comment:

Regardless, if you don't mind, take this patch for Python 3.5 to avoid ctypes, 
at least in the Linux case (I don't have Windows to test).  Creating a proper 
extension module is safer and really not that hard...

--
keywords: +patch
Added file: http://bugs.python.org/file33932/uuid-no-ctypes.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Zachary Ware]

Zachary Ware added the comment:

Vajrasky: Note that you can reply to individual review comments within 
Rietveld; this way context is kept and replies are easier :).  The same people 
get the message either way.

Anyhow, for float point 1: you can use I believe you can use (unaryfunc) for 
nb_int and nb_float, just like nb_positive above.

list point 2: I agree, it's not worth it to try to not use the impl function.

rlimit: I'm not sure what value to give, and I'm not where I can play with it 
until my PC catches fire, either.  Is there some value that makes no sense as a 
legitimate value?  Is it legal for rlim_cur to be greater than rlim_max?  Or is 
there a value that is just exceedingly uncommon?  Or you could simply pick some 
random value; your converter should be written to render with 
self.err_occurred_if("_return_value == RLIMIT_ERROR_VALUE").

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1410680] Add 'surgical editing' to ConfigParser

[Berker Peksag]

Changes by Berker Peksag :


--
versions: +Python 3.5 -Python 3.4

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

> "\n \r\n \r \u2028".split() should have been "\n \r\n \r \u2028".split(" "),
> i.e. a list of different line ends.

Yes, this is obvious fix, but explicit tuple of line ends looks more clear and 
error-proof to me.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] ctypes.create_string_buffer creates reference cycles

[R. David Murray]

R. David Murray added the comment:

Yes, I was pretty sure it was in cytpes, from looking at the UUID code.

--
components: +ctypes -Library (Lib)
nosy: +amaury.forgeotdarc, belopolsky, meador.inge
title: uuid.uuid4().hex generates garbage when ctypes available -> 
ctypes.create_string_buffer creates reference cycles

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Vajrasky Kok]

Vajrasky Kok added the comment:

Zachary, "Looking again, that one is non-trivial, but still doable.  You just 
need a "this means an error happened" value to initialize rl to, and return 
that value instead of NULL."

How do you give "this means an error happened" value to struct rlimit?

struct rlimit {
   rlim_t rlim_cur;  /* Soft limit */
   rlim_t rlim_max;  /* Hard limit (ceiling for rlim_cur) */
};

This is what prevents me to use custom converter.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

[Vajrasky Kok]

Vajrasky Kok added the comment:

Here is the patch addressing Zachary's review (Thanks!). There are some 
Zachary's suggestions that I could not implement:

1. float_conjugate_impl,/* nb_float */
I think this should still be the real function (the parser), not the impl.  The
impl function is really just an implementation detail.

It has to be that way. If I change it to float_conjugate, the compiler will 
complain. But on other places, we can use the real function.

2. v = list_sort_impl((PyListObject *)v, Py_None, 0);
Considering what I said about not using impl functions at the end of
floatobject.c, it would be nice to avoid it here, but I think that would be a
lot more trouble than it would be worth.

I can not use the real function here, otherwise the compiler will throw error.

--
Added file: http://bugs.python.org/file33931/issue20185_conglomerate_v3.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20053] venv and ensurepip are affected by default pip config file

[Paul Moore]

Paul Moore added the comment:

Nothing obvious or Windows-specific from what I can see. It does seem to me 
that EnvironmentVarGuard doesn't monkeypatch os.environ even though it looks 
like it intends to (__exit__ sets it back, but __enter__ doesn't monkeypatch 
it). So maybe that's doing something weird?

I'll try to have a better look later (not much spare time right now).

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20522] pandas version 0.13.0 does not give warning when summing float with object

[Mark Dickinson]

Mark Dickinson added the comment:

Hi Simone,

I'm afraid you've got the wrong bugtracker.  This is the bugtracker for the 
Python core language (which doesn't include Pandas).  Pandas is a 3rd party 
project.  You can report Pandas issues here:

https://github.com/pydata/pandas/issues

--
nosy: +mark.dickinson
resolution:  -> invalid
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20522] pandas version 0.13.0 does not give warning when summing float with object

[Simone Serrajotto]

New submission from Simone Serrajotto:

Good afternoon,
I have just installed pandas 0.13.0 and was observing some strange results
produced by old codes.
One major issue is that I found that when summing two columns of a
dataframe, one of which is a float64 and the other an object, the result
includes just the float64 columns.
This would be not an issue per se, but no warning message is displayed.
I think a warning in this case is of paramount importance.

Regards,
Simone

-- 

Simone Serrajotto

Quaestio Capital Management SGR SpA Unipersonale

Corso Como 15 (8° piano)

20154 Milano

C.F. e P.I. 06803880969

Iscritta all'Albo delle SGR al n. 299

Mobile:  +39 393 919 0217

Tel:   +39 02 3676 5241

E-Mail:s.serrajo...@quaestiocapital.com

-- 

Le informazioni contenute nella comunicazione che precede sono riservate e 
destinate esclusivamente alla persona o all'ente sopraindicati. La 
diffusione, distribuzione e/o copiatura del presente email o di qualsiasi 
documento allegato da parte di qualsiasi soggetto diverso dal destinatario 
e' proibita. La sicurezza e la correttezza dei messaggi di posta 
elettronica non possono essere garantite. Se avete ricevuto questo 
messaggio per errore, Vi preghiamo di contattarci immediatamente. Grazie

The content of this email is confidential and it is intended for the sole 
use of the addressees. The transmission, distribution and/or copy of this 
email or any document attached hereto by any person other than the 
addressees is prohibited. The transmission of this email cannot be 
guaranteed to be secure or error-free. If you have received this 
communication unintentionally, please inform us immediately. Thank you.

--
messages: 210323
nosy: serrajo
priority: normal
severity: normal
status: open
title: pandas version 0.13.0 does not give warning when summing float with 
object

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] uuid.uuid4().hex generates garbage when ctypes available

[Gustavo J. A. M. Carneiro]

Gustavo J. A. M. Carneiro added the comment:

I have narrowed it down to one line of code:
ctypes.create_string_buffer(16)

That is enough to create 7 objects that have reference cycles.

[, {'__module__': 'ctypes', '__doc__': None, 
'__weakref__': , 'raw': 
, '_length_': 16, '_type_': 
, 'value': , '__dict__': }, 
(, , , ), , , 
, ]

So maybe the bug is in ctypes itself, not the uuid module.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4188] test_threading hang when running as verbose

[Berker Peksag]

Berker Peksag added the comment:

The threading._VERBOSE attribute was removed in issue 13550 (see also changeset 
http://hg.python.org/cpython/rev/8ec51b2e57c2).

--
nosy: +berker.peksag
resolution:  -> out of date
stage: needs patch -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] uuid.uuid4().hex generates garbage when ctypes available

[Gustavo J. A. M. Carneiro]

Gustavo J. A. M. Carneiro added the comment:

Well, this isn't a big problem, but I have an application that needs to run 
with the GC disabled, since it causes pauses of a couple of seconds each time a 
full collection runs (we have a few million objects allocated).  I will run the 
GC only once every 3 hours.  So it would be nice if this uuid call didn't 
generate cycles.

No, I didn't identify the cycle.  All I know is that the garbage below is 
produced.  If the ctypes module is unavailable, uuid still works but doesn't 
generate the garbage.

>>> gc.garbage
[(,), {'raw': , '__module__': 'ctypes', '__dict__': , '__weakref__': , '_length_': 16, '_type_': , 
'__doc__': None, 'value': }, 
, , , (, , , 
), , , (,), {'raw': 
, '__module__': 'ctypes', 
'__dict__': , '__weakref__': 
, '_length_': 16, 
'_type_': , '__doc__': None, 'value': }, , , , (, , , ), , , (,), {'raw': , 
'__module__': 'ctypes', '__dict__': , '__weakref__': , '_length_': 16, '_type_': , '__doc__': None, 
'value': }, , , 
, (, , , ), , , 
(,), {'raw': , '__module__': 'ctypes', '__dict__': , '__weakref__': , '_length_': 16, '_type_': , 
'__doc__': None, 'value': }, 
, , , (, , , 
), , ]

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20501] fileinput module will read whole file into memory when using fileinput.hook_encoded

[Gunnar Aastrand Grimnes]

Gunnar Aastrand Grimnes added the comment:

Agreed that a doc-fix is the first step. 

Shall I make another issue for the fix in fileinput?

 I also wonder if the comment in codecs is not too negative - it is not easy to 
get it 100% right, but the method just above readlines is readline, which does 
somehow manage to read a single line. To me it seems like it would be better to 
makes readlines repeatedly call readline. It would also be inefficient, but 
still better than reading the whole (possibly infinite) stream in ...

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20501] fileinput module will read whole file into memory when using fileinput.hook_encoded

[R. David Murray]

R. David Murray added the comment:

Yes, please make another issue for the enhancement request.  That issue can 
start out agnostic as to whether it is codecs or fileinput that could use 
improvement, if you prefer.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20053] venv and ensurepip are affected by default pip config file

[Nick Coghlan]

Nick Coghlan added the comment:

D'oh, still failing even though that new assumption check passes on Windows: 
http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.x/builds/4002/steps/test/logs/stdio

Paul, any ideas?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] uuid.uuid4().hex generates garbage when ctypes available

[R. David Murray]

R. David Murray added the comment:

I'm not sure that this is a real problem, but have you identified what the 
cycle is?

--
nosy: +r.david.murray

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Walter Dörwald]

Walter Dörwald added the comment:

"\n \r\n \r \u2028".split() should have been "\n \r\n \r \u2028".split(" "), 
i.e. a list of different line ends.

> The purpose of these tests is not entirely clear, so I'm not sure that it is 
> properly grasped the idea of the author.

I wrote the tests nearly 10 years ago, so it's no longer entirely clear to me 
either! ;)

Anyway, here's a patch that fixes the resulting test failures.

--
Added file: http://bugs.python.org/file33930/fix_linetests.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20053] venv and ensurepip are affected by default pip config file

[Roundup Robot]

Roundup Robot added the comment:

New changeset ddc82c4d1a44 by Nick Coghlan in branch 'default':
Issue #20053: new test to check an assumption
http://hg.python.org/cpython/rev/ddc82c4d1a44

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20521] [PATCH] Cleanup for "dis" module documentation

[Sven Berkvens-Matthijsse]

New submission from Sven Berkvens-Matthijsse:

The documentation for the "dis" module has a wrong opcode name in it (POP_STACK 
instead of POP_TOP). Furthermore, the patch fixes inconsistent use of markers 
for code and opcodes.

--
assignee: docs@python
components: Documentation
files: dis-doc.patch
keywords: patch
messages: 210314
nosy: docs@python, svenberkvens
priority: normal
severity: normal
status: open
title: [PATCH] Cleanup for "dis" module documentation
type: behavior
versions: Python 3.4
Added file: http://bugs.python.org/file33929/dis-doc.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20167] Exception on IDLE closing

[Terry J. Reedy]

Terry J. Reedy added the comment:

I tried all three versions both installed and recent repository builds and 
verified that the overt issue is limited to 3.4. I agree that the exception 
message suggests stopping with the first exception. Since there is a small cost 
to try: and break, I am inclined to move the try back out of the loop. The only 
thing it could mask is a problem with self.handlerids, and I would expect that 
if that is corrupted or missing, there would be problems before shutdown.

I know that in 3.4, shutdown order has been modified and the wording of the 
warning was discussed on pydev. I do not know (and do not care at the moment) 
if either ignoring __del__ exceptions or warning about them is new in 3.4. What 
I do remember from the discussion is agreement that 1. __del__ exceptions 
should be caught so shutdown can continue; 2. a warning should be given in case 
the exception indicates a bug in __del__; and 3. __del__ writers are 
responsible to catch exceptions that do not indicate a bug, so as to avoid the 
warning. So I am inclined to backport to all versions.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20053] venv and ensurepip are affected by default pip config file

[Nick Coghlan]

Nick Coghlan added the comment:

*sigh*, Windows is not cooperating, suggesting it is still reading the default 
config file:

http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.x/builds/4000/steps/test/logs/stdio

==
FAIL: test_with_pip (test.test_venv.EnsurePipTest)
--
Traceback (most recent call last):
  File "C:\buildbot.python.org\3.x.kloth-win64\build\lib\test\test_venv.py", 
line 342, in test_with_pip
self.assertEqual(err, "")
AssertionError: 'C:\\Users\\Buildbot\\AppData\\Local\\Temp[57 chars]\r\n' != ''
- C:\Users\Buildbot\AppData\Local\Temp\tmpfiz5hrop\Scripts\python_d.exe: No 
module named pip

--
components: +Library (Lib)
keywords: +buildbot
resolution: fixed -> 
stage: committed/rejected -> commit review
status: closed -> open
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1529353] Squeezer - squeeze large output in the interpreter

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Could you please provide a patch which applies to default branch?

There is other proposed solution for the problem of large output: issue1442493.

--
stage: patch review -> needs patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20489] help() fails for zlib Compress and Decompress objects

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


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

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20498] Update StringIO newline tests

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


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

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20520] Readline test in test_codecs is broken

[Serhiy Storchaka]

New submission from Serhiy Storchaka:

The ReadTest.test_readline test in Lib/test/test_codecs.py has two loops on "\n 
\r\n \r \u2028".split(). But as far as "\n \r\n \r \u2028".split() is empty 
list (because newline characters are whitespace characters), the bodies of 
these loops are never executed.

After fixing this bug, a number of other tests were exposed. I tried to fix 
them. The purpose of these tests is not entirely clear, so I'm not sure that it 
is properly grasped the idea of the author.

Test was added in issue1076985 and modified in issue1175396.

--
components: Tests
files: test_codecs_readline.patch
keywords: patch
messages: 210309
nosy: doerwalter, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Readline test in test_codecs is broken
type: behavior
versions: Python 2.7, Python 3.3, Python 3.4
Added file: http://bugs.python.org/file33928/test_codecs_readline.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20167] Exception on IDLE closing

[Tal Einat]

Tal Einat added the comment:

Both previous patches caused an import error. Here's a new patch with that 
fixed, and actually tested to fix the bug.

--
Added file: 
http://bugs.python.org/file33927/taleinat_idle_closing_exception_3.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20167] Exception on IDLE closing

[Tal Einat]

Tal Einat added the comment:

Attaching second patch, to replace the first.

As suggested by Serhiy, I moved the try/except block into the loop.

I also added a comment explaining the try/except block and referencing this 
issue.

--
Added file: 
http://bugs.python.org/file33926/taleinat_idle_closing_exception_2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20519] uuid.uuid4().hex generates garbage when ctypes available

[Gustavo J. A. M. Carneiro]

New submission from Gustavo J. A. M. Carneiro:

If you try the attached program, you will find that for every iteration the 
uuid.uuid4() call generates objects that contain reference cycles and need the 
help of the garbage collector.  This is not nice.  If I make the ctypes module 
not able to import, then no garbage is generated.

This problem appears in 2.7, 3.3, and 3.4, at least.

--
components: Library (Lib)
files: gc.py
messages: 210306
nosy: gustavo
priority: normal
severity: normal
status: open
title: uuid.uuid4().hex generates garbage when ctypes available
type: resource usage
versions: Python 2.7, Python 3.3, Python 3.4
Added file: http://bugs.python.org/file33925/gc.py

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20167] Exception on IDLE closing

[Tal Einat]

Tal Einat added the comment:

I put the try/except outside of the loop on purpose. If calling the widget's 
unbind() method fails once, it will fail forever afterwards.

If you prefer a different spelling, move the try/except into the loop, and 
break out of the loop in case of an exception:

for seq, id in self.handlerids:
try:
self.widget.unbind(self.widgetinst, seq, id)
except _tkinter.TclError:
break


As for avoiding the exception in the first place, I'm sure figuring out how 
IDLE shuts down would be a lot of work, and I honestly don't think it's 
necessary. Note that this problem is triggered in a __del__() method; making 
sure these are called at a proper time is problematic because the timing 
depends on the GC.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3068] IDLE - Add an extension configuration dialog

[Terry J. Reedy]

Terry J. Reedy added the comment:

Thank you! This is a bit of the 'Idle Developer's Guide' that I wish existed. 
It turns out that the exceptional FormatParagraph.py is the one extension file 
I am really familiar with, because I worked with a GSOC student to write tests 
and fix/improve it. I agree that this would be useful now.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20489] help() fails for zlib Compress and Decompress objects

[Roundup Robot]

Roundup Robot added the comment:

New changeset 31ddd45a05ba by Serhiy Storchaka in branch 'default':
Issue #20489: Explicitly qualified expressions for default values in methods.
http://hg.python.org/cpython/rev/31ddd45a05ba

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20498] Update StringIO newline tests

[Roundup Robot]

Roundup Robot added the comment:

New changeset 07e7bb29a2c5 by Serhiy Storchaka in branch '2.7':
Issue #20498: Fixed io.StringIO tests for newline='\n'. Added new tests.
http://hg.python.org/cpython/rev/07e7bb29a2c5

New changeset e23c928b9e39 by Serhiy Storchaka in branch '3.3':
Issue #20498: Fixed io.StringIO tests for newline='\n'. Added new tests.
http://hg.python.org/cpython/rev/e23c928b9e39

New changeset 7ed8a9f9831d by Serhiy Storchaka in branch 'default':
Issue #20498: Fixed io.StringIO tests for newline='\n'. Added new tests.
http://hg.python.org/cpython/rev/7ed8a9f9831d

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue12691] tokenize.untokenize is broken

[Gareth Rees]

Changes by Gareth Rees :


--
assignee:  -> docs@python
components: +Documentation, Tests
nosy: +docs@python

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue20517] Support errors with two filenames for errno exceptions

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

As release manager Larry has the right to add a new feature after feature 
freeze.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com