[issue2863] Generator __name__ and better repr()

2008-05-15 Thread svncodereview

svncodereview <[EMAIL PROTECTED]> added the comment:

Dear report,

New code review comments by GvR have been published.
Please go to http://codereview.appspot.com/1046 to read them.

Message:
Looks good to me.

Details:

Issue Description:
http://bugs.python.org/issue2863

Sincerely,

  Your friendly code review daemon (http://codereview.appspot.com/).

--
nosy: +svncodereview

__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue2863>
__
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1736190] asyncore/asynchat patches

2008-05-21 Thread svncodereview

svncodereview <[EMAIL PROTECTED]> added the comment:

Dear GvR, report,

New code review comments by josiah.carlson have been published.
Please go to http://codereview.appspot.com/744 to read them.

Message:

Details:

http://codereview.appspot.com/744/diff/1/22
File Doc/library/asyncore.rst (right):

http://codereview.appspot.com/744/diff/1/22#newcode226
Line 226: A file_dispatcher takes a file descriptor or file object 
along with an optional
On 2008/05/05 22:02:22, GvR wrote:
> Mind keeping the line length under 80 chars?

No problem.  Any other comments?

Issue Description:
http://bugs.python.org/issue1736190

Sincerely,

  Your friendly code review daemon (http://codereview.appspot.com/).

_
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1736190>
_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2753] Test issue (different spelling)

2008-05-04 Thread svncodereview

svncodereview <[EMAIL PROTECTED]> added the comment:

Dear [EMAIL PROTECTED], [EMAIL PROTECTED],

New code review comments by [EMAIL PROTECTED] have been published.
Please go to http://codereview.appspot.com/730 to read them.

Message:

Details:

http://codereview.appspot.com/730/diff/1/21
File Lib/test/regrtest.py (right):

http://codereview.appspot.com/730/diff/1/21#newcode3
Line 3: """Regression test driver.
Comment 6

Issue Description:

Sincerely,

  Your friendly code review daemon (http://codereview.appspot.com/).

--
nosy: +svncodereview
title: Test issue -> Test issue (different spelling)

__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue2753>
__
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1736190] asyncore/asynchat patches

2008-05-05 Thread svncodereview

svncodereview <[EMAIL PROTECTED]> added the comment:

Dear [EMAIL PROTECTED],

New code review comments by [EMAIL PROTECTED] have been published.
Please go to http://codereview.appspot.com/744 to read them.

Message:
(This is mostly a test of the bug/rietveld integration.)

Details:

http://codereview.appspot.com/744/diff/1/22
File Doc/library/asyncore.rst (right):

http://codereview.appspot.com/744/diff/1/22#newcode226
Line 226: A file_dispatcher takes a file descriptor or file object 
along with an optional
Mind keeping the line length under 80 chars?

Issue Description:
http://bugs.python.org/issue1736190

Sincerely,

  Your friendly code review daemon (http://codereview.appspot.com/).

--
nosy: +svncodereview

_
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1736190>
_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com