Hi Balaji,
Dne 29. března 2012 10:21 Balaji Rao <[email protected]> napsal(a):
> On Thursday 29 March 2012 01:18 PM, Vojtech Horky wrote:
>>
>> Hi Balaji,
>
>
> Hi Vojtech,
>
>> thanks for the patch.
>>
>
> Thanks for the review!
No problem. I already applied your patch to my branch [1] and it will
be merged soon.

>> I hope someone else will carry-on and add support for national characters.
>>
>
> Oh you're talking about the problem of creating SFN when the given filename
> has
> unicode characters ?
Yes. The problem is that
dir_entry.name = dir_entry.name.upper().encode('ascii')
may not work for all characters. You have to add the 'ignore' option
but then you may end with empty filename. I haven't look yet into doc
how the algorithm for mangling shall deal with this.

Again, thanks for the patch.

- Vojta

[1] https://code.launchpad.net/~vojtech-horky/helenos/misc

>
>>
>> Below are the patch excerpts with comments
>>>
>>>
>>> === modified file 'tools/mkfat.py'
>>> --- tools/mkfat.py    2011-07-24 21:18:16 +0000
>>> +++ tools/mkfat.py    2012-03-26 18:44:46 +0000
>>> +
>>> +def get_utf16(name, l) :
>>> +    "Create a int array out of a string which we can
>>> +    store in uint16_t arrays"
>>
>> This causes parsing error. For multiline comments you shall use """.
>>
>>> +    csum = 0
>>> +    for i in range(0, 11) :
>>> +        csum = ((sum&  1)<<  7) + (sum>>  1) + ord(fname11[i])
>>> +        csum = sum&  0xFF
>>>
>>> +
>>> +    for e in entries :
>>> +        e.csum = sum;
>>
>> Few typos here (sum ->  csum). I wonder which version of Python you are
>> using that it worked for you ;-).
>
>
> Lol! Sorry :(
>
> Best,
> Balaji
>
>
>
> _______________________________________________
> HelenOS-devel mailing list
> [email protected]
> http://lists.modry.cz/cgi-bin/listinfo/helenos-devel

_______________________________________________
HelenOS-devel mailing list
[email protected]
http://lists.modry.cz/cgi-bin/listinfo/helenos-devel

Reply via email to