[PATCH] D79265: Let normalize() for posix style convert backslash to slash unconditionally.

[Nico Weber via Phabricator via cfe-commits]

thakis added a comment.

In D79265#2018770 <https://reviews.llvm.org/D79265#2018770>, @compnerd wrote:

> What happens if you encounter a `"\t"` as a string?  This would convert that to a `"/t"` would it not?  Although, I suppose that in practice the treatment of escaped characters is not important.  I think I still prefer the `} else {` here over the early return.


`"\t"` is converted to `"/t"`. That's also the behavior before this change though – only '\' followed by '\' was handled specially before, not '\' followed by anything else.

I restored the else and removed the early return.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79265/new/

https://reviews.llvm.org/D79265