Re: [PATCH] D19064: clang-format: Allow include of clangFormat.h in managed context

2016-04-14 Thread Marianne Mailhot-Sarrasin via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL266319: clang-format: Allow include of clangFormat.h in managed context (authored by mamai). Changed prior to commit: http://reviews.llvm.org/D19064?vs=53608&id=53717#toc Repository: rL LLVM http://

Re: [PATCH] D19064: clang-format: Allow include of clangFormat.h in managed context

2016-04-13 Thread Maxime Beaulieu via cfe-commits
mxbOctasic updated this revision to Diff 53608. mxbOctasic added a comment. Removed empty line. http://reviews.llvm.org/D19064 Files: include/clang/Format/Format.h lib/Format/Format.cpp Index: lib/Format/Format.cpp === --- lib

Re: [PATCH] D19064: clang-format: Allow include of clangFormat.h in managed context

2016-04-13 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Comment at: include/clang/Format/Format.h:30 @@ +29,3 @@ +namespace vfs { + +class FileSystem; nit: Remove this empty line. http://reviews.llv

[PATCH] D19064: clang-format: Allow include of clangFormat.h in managed context

2016-04-13 Thread Maxime Beaulieu via cfe-commits
mxbOctasic created this revision. mxbOctasic added a reviewer: djasper. mxbOctasic added subscribers: cameron314, cfe-commits. Herald added a subscriber: klimek. Including `VirtualFileSystem.h` in the clangFormat.h indirectly includes ``. This header is blocked when compiling with /clr. I added a