[MERGE] bzrlib.osutils fixes
Dmitry Vasiliev
dima at hlabs.spb.ru
Thu Mar 8 08:53:04 GMT 2007
Alexander Belchenko wrote:
> Dmitry Vasiliev пишет:
>> The attached bundle contains small fixes/optimizations for bzrlib.osutils.
>
> Can you explain this change, please?
>
>> @@ -704,7 +711,8 @@
>> for dirdetail, dirblock in osutils._walkdirs_utf8('.'):
>> self.assertIsInstance(dirdetail[0], str)
>> if isinstance(dirdetail[1], unicode):
>> - dirdetail[1] = dirdetail[1].encode('utf8')
>> + dirdetail = (dirdetail[0], dirdetail[1].encode('utf8'))
>> + dirblock = [list(info) for info in dirblock]
>> for info in dirblock:
>> self.assertIsInstance(info[4], unicode)
>> info[4] = info[4].encode('utf8')
The changes is a bit straightforward. The old code tried to mutate
immutable 'dirdetail' and 'info' tuples. The new code recreate
'dirdetail' tuple and converts all 'info' tuples (there is no real
difference for them to be lists instead of tuples in the latter code) to
lists if needed.
--
Dmitry Vasiliev <dima at hlabs.spb.ru>
http://hlabs.spb.ru
More information about the bazaar
mailing list