<div dir="ltr">Two thoughts:<div>a) I think it might be "better" (in some regards) to entirely disable these tests on MSVC. Changing them to use escape sequences makes them somewhat harder to maintain. As we run these tests frequently on other platforms the chances of regressions are slim.</div>
<div>b) We want to split up the tests and have a significant portion of them in FileCheck(-like) tests in the near future. Thus, the formatting input-data will be in plain files that aren't compiled and this problem should go away, making the argument for (a) stronger.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 2, 2013 at 11:02 AM, Curdeius Curdeius <span dir="ltr"><<a href="mailto:curdeius@gmail.com" target="_blank">curdeius@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div>I've prepared a patch for clang-format unit test.</div><div>There were encoding problems with Unicode characters on MSVC.</div>
<div>I've escaped them.</div><div>However, the 4-byte characteres don't work as expected, so the corresponding test is still turned off on MSVC (I've tried using escape sequences like "\U0001abcd" but that doesn't work neither).</div>


<div>The patch is attached.</div><div><br></div><div>The formatting do not stick to the coding style rules, but I didn't know how to handle escape sequences (yes, I could have used clang-format...).</div>

<div><br></div><div>Cheers,<div>Marek Kurdej</div>

</div>





</div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>