Re: [PATCH] Use fully qualified os.path.join()



On Sun, Apr 5, 2009 at 2:15 PM, Stephen Kennedy <stevek gnome org> wrote:
> On Sat, Apr 4, 2009 at 2:01 PM, Vincent Legoll <vincent legoll gmail com> wrote:
>> Is there really a demonstrated speed benefit here,
>> caching namespace lookups for os.path.join()
>>
>> If not, I'll apply the attached patch.
>
> Nope, it was just for brevity.

r1328

> On a related note I've been using
> Jason Orendorff's path module in other projects and think it might
> be a good fit for meld. While purists decry its kitchen-sink interface,
> I've found it makes code more readable and terse in a good way.
> http://vvtools.googlecode.com/svn/trunk/ahcm/path.py

Looks nice for a new project, why not, but I don't think we want
to move meld code to that API, just stick to python's stdlib.

-- 
Vincent Legoll


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]