<div dir="ltr">We already have tests that rely on CRLF line endings in Sema/warn-documentation-crlf.c and FixIt/fixit-newline-style.c. They should be fine once they're in the repo, I'm just concerned that commit process and syncing with svn might mess it up. If the changes look good I could give it a try?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 8, 2014 at 10:05 PM, 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">I don't think we can rely on both git and svn deal correctly with line-endings on all platforms =\ Can we generate a test file with needed line endings during a test run?<span class=""><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 6:09 AM, 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">This patch should have mixed newlines, I just hope Git doesn't convert them on commit.</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 1: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><div><div><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></span></blockquote></div></div></div></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote></div>
</div></div></span></div>
</blockquote></div><br></div>