Jason Tackaberry wrote:
> On Thu, 2007-02-15 at 17:17 +0100, Dirk Meyer wrote:
>> elif line.startswith("ID_") and line.find("=") != -1:
>
> This is more pythonic:
>
> elif line.startswith('ID_') and '=' in line:
>
>
>> - attr, value = line.split("=")
>> - attr = attr[3:]
>> + # attr, value = line.split("=") could crash if the filename
>> + # contains a '=' (which could happen very easy in an http url)
>> + attr = line[3:line.find('=')]
>> + value = line[line.find('=')+1]
>
> Don't you just want to split on the first equal sign? Shouldn't this
> work:
>
> attr, value = line.split('=', 1)
> attr = attr[3:]
>
> It's much cleaner.It is much cleaner, but it had the disadvantage against the other solution that I did not know that I get give an extra parameter to split :) Changed now. > P.S. Sorry I haven't been around much. I've been unusually busy with > work. Too bad. Any change to work on candy the next weeks? Dischi -- Linux vs. Windows is a no-WIN situation.
pgpbe6CRuxYv9.pgp
Description: PGP signature
------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________ Freevo-cvslog mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/freevo-cvslog
