Patches item #1515361, was opened at 2006-06-30 16:16
Message generated for change (Comment added) made by nnorwitz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1515361&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core (C code)
Group: Python 2.5
>Status: Closed
>Resolution: Rejected
Priority: 5
Submitted By: Shane Hathaway (hathawsh)
>Assigned to: Neal Norwitz (nnorwitz)
Summary: 1515169 fix (ImportWarning flood)

Initial Comment:
This patch to import.c only generates ImportWarnings if
the import fails.  It also fixes what appear to be
memory leaks involving the "copy" variable.

This patch applies to Python 2.5b1.


----------------------------------------------------------------------

>Comment By: Neal Norwitz (nnorwitz)
Date: 2006-07-05 21:29

Message:
Logged In: YES 
user_id=33168

Shane, thanks for the patch.

The decision was made to simply ignore the warning by
default.  I'm closing it because we are taking a different
approach at this time.  We will revisit the decision for
2.6.  I hope you continue reading python-dev and help us fix
this better for 2.6, whatever that may be.  Perhaps this
patch will be revived at that time.

I have added a link to this patch in PEP 361.

I agree with the memory leaks wrt to copy and have checked
in changes that should address all the leaks.  Let me know
if you spot other ones (or I made a mistake).

----------------------------------------------------------------------

Comment By: Sergey (sergeyli)
Date: 2006-07-01 11:10

Message:
Logged In: YES 
user_id=141494

See also an alternative python.org/sf/1515609. Mine doesn't
have the memory leak fix.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1515361&group_id=5470
_______________________________________________
Patches mailing list
[email protected]
http://mail.python.org/mailman/listinfo/patches

Reply via email to