[PATCH] D12407: [clang-format-vs] Add an option to reformat source code when file is saved to disk

[Hans Wennborg via cfe-commits]

hans added inline comments.

================
Comment at: tools/clang-format-vs/ClangFormat/ClangFormatPackage.cs:86
@@ -69,1 +85,3 @@
 
+            IComponentModel componentModel = GetService(typeof(SComponentModel)) as IComponentModel;
+            editorAdaptersFactoryService = componentModel.GetService<IVsEditorAdaptersFactoryService>();
----------------
aaron.ballman wrote:
> berenm wrote:
> > I did more tests on my side, and apparently this line does not work on VS2012, componentModel is null. I don't know at all why and how to fix it, and it works fine starting with VS2013.
> Our minimum supported MSVC version for development is 2013. Do we document supported versions for clang-format? Do we want to support versions older than the development version we're on?
The manifest claims support for 2010 and later. I usually test with 2012 when I build the weekly snapshot.

Our clang-cl VS integration also tries to support 2010 and later, so I think it would be nice if the clang-format plugin does too.


http://reviews.llvm.org/D12407