>
>
> +  for (int i = 0; i != argc; ++i)
> +  {
> +    // check lenght
>
> May want to correct the typo.
>
>
Fixed


>
> I think this should be formatted more like this to meet our coding
> standards (but am not 100% sure, so I will defer to others):
>
> if (ch < (UTF32)0x80)
>   bytesToWrite = 1;
> else if (ch < (UTF32)0x800)
>  bytesToWrite = 2;
> ...
> else
>  ch = UNI_REPLACEMENT_CHAR;
>
>
I haven't touched this code. I only took the function out of the section
that was commented out.

Attachment: clang.patch
Description: Binary data

Attachment: llvm.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to