On 20 August 2013 20:35, Eli Friedman <[email protected]> wrote:

> On Tue, Aug 20, 2013 at 8:08 PM, Nick Lewycky <[email protected]> wrote:
>
>> The attached patch adds a fixit for invalid characters to replace them
>> with the equivalent \xNN code. Please review!
>>
>
> You might want to put a warning on top of the test that this file isn't
> valid UTF-8, despite appearing to be a text file.
>

Good idea. Done, r188863.

Otherwise, looks good; nice idea!
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to