[MERGE] Allow appending path segments to the :<name> style aliases.

John Arbash Meinel john at arbash-meinel.com
Thu Aug 14 21:35:29 BST 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Michael Hudson wrote:
> John Arbash Meinel wrote:
>> Michael Hudson wrote:
>>
>> ...

...

>> Perhaps. But I also think the code is *clearer* and a lot more obvious
>> about getting the edge conditions correct.
> 
> That's a more reasonable objection!  I don't particularly agree, but
> it's an opinion I can respect.
> 
> Cheers,
> mwh
> 

Actually, the clearest, IMO would be:
name = url[1:]
if '/' in name:
  name, extra = name.split('/', 1)
else:
  extra = None

Though it is certainly *way* too much discussion for a tiny bit of code.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkiklxEACgkQJdeBCYSNAANMAwCfS1mT8eMq1DPRVUNrt4kxnyNG
8mwAoNC231xxWi3O+FibmtKLlxxEMk3V
=XaZ0
-----END PGP SIGNATURE-----



More information about the bazaar mailing list