This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author orsenthil
Recipients belopolsky, daniel.urban, eli.bendersky, eric.araujo, ezio.melotti, georg.brandl, ncoghlan, ned.deily, orsenthil, rhettinger, vinay.sajip, vstinner
Date 2011-02-17.16:12:33
SpamBayes Score 3.260836e-12
Marked as misclassified No
Message-id <1297959154.25.0.62189724586.issue11071@psf.upfronthosting.co.za>
In-reply-to
Content
Here are couple of Review Comments on What's new document.

1. The logging dictConfig example fails if you try directly out-of-text.

The conf file is opened as a 'rb' and json module throws an error as "TypeError: can't use a string pattern on a bytes-like object".  Suggestion - Just reading it as string is fine.
Also, this example throws another error from json, when you read it as string.

ValueError: Expecting property name: line 4 column 17 (char 204)

I was able to fix it by simplifying the conf.json, but I think the current one itself should be fixed and made to work.

Related suggestion - I see another section on logging further down. It would be better to combine the sections.

2.Issue number for functools.lru_cache is not mentioned. If no tracker id/ discussion is available, checkin link can be mentioned.

3. There is section for Popen which says, subprocess.Popen() functions now support with statements for auto-closing of the file descriptors. Not really. Only os.popen() function supports it.

4. In the concurrent.futures example, it is given as:

import threading, shutil
with threading.ThreadPoolExecutor(max_workers=4) as e:

It should be:
import concurrent.futures, shutil
with concurrent.futures.ThreadPoolExector(max_workers=4) as e:


Please correct these before the release on 19th.
History
Date User Action Args
2011-02-17 16:12:34orsenthilsetrecipients: + orsenthil, georg.brandl, rhettinger, vinay.sajip, ncoghlan, belopolsky, vstinner, ned.deily, ezio.melotti, eric.araujo, eli.bendersky, daniel.urban
2011-02-17 16:12:34orsenthilsetmessageid: <1297959154.25.0.62189724586.issue11071@psf.upfronthosting.co.za>
2011-02-17 16:12:33orsenthillinkissue11071 messages
2011-02-17 16:12:33orsenthilcreate