On Wed, Nov 24, 2010 at 01:43:34PM -0500, John Wiegley wrote:
On Nov 24, 2010, at 12:29 PM, John Piasetzki wrote:

I updated acprep to work with Python 3 (tested with Python 3.1.2).

Very cool!  Just a few questions:

-        lib64_dirs = map(lambda x: join(x, 'lib64'), search_prefixes)
-        lib_dirs = map(lambda x: join(x, 'lib'), search_prefixes)
+        lib64_dirs = [search_prefix + '/lib64' for search_prefix in  
search_prefixes]
+        lib_dirs = [search_prefix + '/lib' for search_prefix in  
search_prefixes]

Why is this an improvement?  Is '/' now a universal directory separator in
Python?  Will this work on Windows?  Is there a problem with map and
lambda?

'/' has always worked in Windows.  Various windows programs may not
understand a forward slash, but programmatic APIs have always worked
just fine with either slash.

David

Reply via email to