<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Dec 3, 2014 at 6:43 PM, Nikola Smiljanic <span dir="ltr"><<a href="mailto:popizdeh@gmail.com" target="_blank">popizdeh@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">I think it'll only happen when you run it on Windows. Writing files in text mode will translate each \n into CRLF on this silly platform :) I'm honestly surprised that nobody noticed this before. I filed a bug for clang-modernize for this <a href="https://cpp11-migrate.atlassian.net/browse/CM-171" target="_blank">https://cpp11-migrate.atlassian.net/browse/CM-171</a></div></blockquote><div><br></div><div>Huh, why are clang-modernize bugs in a different bug database?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 1:29 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.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">That may be a good solution, however, could you provide a bit more details: does this problem happen when you run clang-apply-replacements on Windows or on Linux or Mac as well? It would also be nice if we could have a test for this.<div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, Dec 3, 2014 at 4:13 AM, Nikola Smiljanic <span dir="ltr"><<a href="mailto:popizdeh@gmail.com" target="_blank">popizdeh@gmail.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><div dir="ltr">I'm not entirely sure this is the right way to fix the issue, but since SourceManager reads \r \n we should probably write them without newline translation. What happens now is, newline is read as \r \n and when it's written out the \n is translated so we end up with duplicated \r.<div><br></div><div><br></div></div>
<br></span><span>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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></span></blockquote></div><br>
</div></div>
</blockquote></div><br></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></div>