Re: [PATCH] Use fully qualified os.path.join()
- From: Vincent Legoll <vincent legoll gmail com>
- To: Stephen Kennedy <stevek gnome org>
- Cc: meld-list <meld-list gnome org>
- Subject: Re: [PATCH] Use fully qualified os.path.join()
- Date: Sun, 5 Apr 2009 14:50:40 +0200
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]