[Mono-dev] [PATCH] Boost speed of UnicodeEncoding
Atsushi Eno
atsushi at ximian.com
Sun Mar 19 21:04:05 EST 2006
Hi,
Kornél Pál wrote:
> I already sent this to the list but I didn't receive it. Now I changed
> to GMail so now everything should be better.:)
>
>> Cool, thanks Kornél. Can you please commit the patch with a few
>> coding style fixes?
>>
>> foo [i] instead of foo[i]
>> Blah () instead of Blah()
>> (byte) x instead of (byte)x
>
> As previously I followed the style of existing code I modified the style of
> the entire code. In addition I moved UnicodeDecoder to the top of
> UnicodeEncoding to have a more common style.
>
> Previously I forgot to update UnicodeDecoder.GetChars but now it is updated
> as well.
>
> Lupus pointed out the problem with zero-length arrays so now it is fixed as
> well.
>
> Please have another look at the patch. Now I consider it to be complete.
>
> Kornél
Please don't mix different kind of changes: indentation and actual
code fixes. The new patch is not reviewable. The previous one is better.
Atsushi Eno
More information about the Mono-devel-list
mailing list