Re: [PATCH] D16524: [clang-format-vs] Fix sort include main include: Use current path for the '-assume-filename'

2016-02-10 Thread Daniel Jasper via cfe-commits
djasper added a subscriber: djasper. djasper closed this revision. djasper added a comment. Submitted as r260378. http://reviews.llvm.org/D16524 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D16524: [clang-format-vs] Fix sort include main include: Use current path for the '-assume-filename'

2016-02-09 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D16524 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D16524: [clang-format-vs] Fix sort include main include: Use current path for the '-assume-filename'

2016-02-09 Thread Jean-Philippe Dufraigne via cfe-commits
jeanphilippeD added a comment. Hi Manuel, Thank you very much for the review. I do not have commit access, would you be able to commit this patch for me. Kind Regards, Jean-Philippe http://reviews.llvm.org/D16524 ___ cfe-commits mailing list

Re: [PATCH] D16524: [clang-format-vs] Fix sort include main include: Use current path for the '-assume-filename'

2016-02-08 Thread Jean-Philippe Dufraigne via cfe-commits
jeanphilippeD added a comment. Ping, Hi Daniel, Sorry, should I have put someone else as reviewer, I see the previous committers for this file where Manuel Klimek, Hans Wennborg and Marek Kurdej. Should have i added any or all of them. I thought it made sense to put you originally since you