<div>Hi Mikael,</div><div><br></div><div>Just to be sure, did you build clang-format before running this test?</div><div><br></div><div>Possibly related, one of the tests I added was just removed for being redundant:</div><div><a href="https://reviews.llvm.org/D28943">https://reviews.llvm.org/D28943</a><br></div><div><br></div><div><br></div><div><div class="gmail_quote"><div>On Fri, Jan 20, 2017 at 7:50 AM Mikael Holmén <<a href="mailto:mikael.holmen@ericsson.com">mikael.holmen@ericsson.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Antonio,<br class="gmail_msg"><br><br class="gmail_msg"><br>The test case<br class="gmail_msg"><br><br class="gmail_msg"><br>Clang :: Format/style-on-command-line.cpp<br class="gmail_msg"><br><br class="gmail_msg"><br>fails for me since this commit.<br class="gmail_msg"><br><br class="gmail_msg"><br>test/Format/style-on-command-line.cpp:40:13: error: expected string not<br class="gmail_msg"><br>found in input<br class="gmail_msg"><br>// CHECK10: {{^    int\* i;$}}<br class="gmail_msg"><br>             ^<br class="gmail_msg"><br><stdin>:1:1: note: scanning from here<br class="gmail_msg"><br>// RUN: clang-format -style="{BasedOnStyle: Google, IndentWidth: 8}" %s |<br class="gmail_msg"><br>^<br class="gmail_msg"><br><stdin>:61:16: note: possible intended match here<br class="gmail_msg"><br>   // CHECK10: {{^    int\* i;$}}<br class="gmail_msg"><br>                ^<br class="gmail_msg"><br><br class="gmail_msg"><br>I suppose it works for you... Any idea what the problem can be?<br class="gmail_msg"><br><br class="gmail_msg"><br>Regards,<br class="gmail_msg"><br>Mikael<br class="gmail_msg"><br><br>On 01/20/2017 02:22 AM, Antonio Maiorano via cfe-commits wrote:<br class="gmail_msg"><br>> Author: amaiorano<br class="gmail_msg"><br>> Date: Thu Jan 19 19:22:42 2017<br class="gmail_msg"><br>> New Revision: 292562<br class="gmail_msg"><br>><br class="gmail_msg"><br>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=292562&view=rev" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project?rev=292562&view=rev</a><br class="gmail_msg"><br>> Log:<br class="gmail_msg"><br>> clang-format: fix fallback style set to "none" not always formatting<br class="gmail_msg"><br>><br class="gmail_msg"><br>> This fixes clang-format not formatting if fallback-style is explicitly set to<br class="gmail_msg"><br>> "none", and either a config file is found or YAML is passed in without a<br class="gmail_msg"><br>> "BasedOnStyle". With this change, passing "none" in these cases will have no<br class="gmail_msg"><br>> affect, and LLVM style will be used as the base style.<br class="gmail_msg"><br>><br class="gmail_msg"><br>> Differential Revision: <a href="https://reviews.llvm.org/D28844" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28844</a><br class="gmail_msg"><br>><br class="gmail_msg"><br>> Modified:<br class="gmail_msg"><br>>     cfe/trunk/lib/Format/Format.cpp<br class="gmail_msg"><br>>     cfe/trunk/test/Format/style-on-command-line.cpp<br class="gmail_msg"><br>>     cfe/trunk/unittests/Format/FormatTest.cpp<br class="gmail_msg"><br>><br class="gmail_msg"><br>> Modified: cfe/trunk/lib/Format/Format.cpp<br class="gmail_msg"><br>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=292562&r1=292561&r2=292562&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=292562&r1=292561&r2=292562&view=diff</a><br class="gmail_msg"><br>> ==============================================================================<br class="gmail_msg"><br>> --- cfe/trunk/lib/Format/Format.cpp (original)<br class="gmail_msg"><br>> +++ cfe/trunk/lib/Format/Format.cpp Thu Jan 19 19:22:42 2017<br class="gmail_msg"><br>> @@ -1888,8 +1888,8 @@ static FormatStyle::LanguageKind getLang<br class="gmail_msg"><br>>  }<br class="gmail_msg"><br>><br class="gmail_msg"><br>>  llvm::Expected<FormatStyle> getStyle(StringRef StyleName, StringRef FileName,<br class="gmail_msg"><br>> -                                     StringRef FallbackStyle, StringRef Code,<br class="gmail_msg"><br>> -                                     vfs::FileSystem *FS) {<br class="gmail_msg"><br>> +                                     StringRef FallbackStyleName,<br class="gmail_msg"><br>> +                                     StringRef Code, vfs::FileSystem *FS) {<br class="gmail_msg"><br>>    if (!FS) {<br class="gmail_msg"><br>>      FS = vfs::getRealFileSystem().get();<br class="gmail_msg"><br>>    }<br class="gmail_msg"><br>> @@ -1903,9 +1903,9 @@ llvm::Expected<FormatStyle> getStyle(Str<br class="gmail_msg"><br>>        (Code.contains("\n- (") || Code.contains("\n+ (")))<br class="gmail_msg"><br>>      Style.Language = FormatStyle::LK_ObjC;<br class="gmail_msg"><br>><br class="gmail_msg"><br>> -  // FIXME: If FallbackStyle is explicitly "none", format is disabled.<br class="gmail_msg"><br>> -  if (!getPredefinedStyle(FallbackStyle, Style.Language, &Style))<br class="gmail_msg"><br>> -    return make_string_error("Invalid fallback style \"" + FallbackStyle.str());<br class="gmail_msg"><br>> +  FormatStyle FallbackStyle = getNoStyle();<br class="gmail_msg"><br>> +  if (!getPredefinedStyle(FallbackStyleName, Style.Language, &FallbackStyle))<br class="gmail_msg"><br>> +    return make_string_error("Invalid fallback style \"" + FallbackStyleName);<br class="gmail_msg"><br>><br class="gmail_msg"><br>>    if (StyleName.startswith("{")) {<br class="gmail_msg"><br>>      // Parse YAML/JSON style from the command line.<br class="gmail_msg"><br>> @@ -1977,7 +1977,7 @@ llvm::Expected<FormatStyle> getStyle(Str<br class="gmail_msg"><br>>      return make_string_error("Configuration file(s) do(es) not support " +<br class="gmail_msg"><br>>                               getLanguageName(Style.Language) + ": " +<br class="gmail_msg"><br>>                               UnsuitableConfigFiles);<br class="gmail_msg"><br>> -  return Style;<br class="gmail_msg"><br>> +  return FallbackStyle;<br class="gmail_msg"><br>>  }<br class="gmail_msg"><br>><br class="gmail_msg"><br>>  } // namespace format<br class="gmail_msg"><br>><br class="gmail_msg"><br>> Modified: cfe/trunk/test/Format/style-on-command-line.cpp<br class="gmail_msg"><br>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Format/style-on-command-line.cpp?rev=292562&r1=292561&r2=292562&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Format/style-on-command-line.cpp?rev=292562&r1=292561&r2=292562&view=diff</a><br class="gmail_msg"><br>> ==============================================================================<br class="gmail_msg"><br>> --- cfe/trunk/test/Format/style-on-command-line.cpp (original)<br class="gmail_msg"><br>> +++ cfe/trunk/test/Format/style-on-command-line.cpp Thu Jan 19 19:22:42 2017<br class="gmail_msg"><br>> @@ -11,6 +11,21 @@<br class="gmail_msg"><br>>  // RUN: clang-format -style=file -assume-filename=%T/foo.cpp < %s | FileCheck -strict-whitespace -check-prefix=CHECK7 %s<br class="gmail_msg"><br>>  // RUN: clang-format -style="{BasedOnStyle: LLVM, PointerBindsToType: true}" %s | FileCheck -strict-whitespace -check-prefix=CHECK8 %s<br class="gmail_msg"><br>>  // RUN: clang-format -style="{BasedOnStyle: WebKit, PointerBindsToType: false}" %s | FileCheck -strict-whitespace -check-prefix=CHECK9 %s<br class="gmail_msg"><br>> +<br class="gmail_msg"><br>> +// Fallback style tests<br class="gmail_msg"><br>> +// RUN: rm %T/_clang-format<br class="gmail_msg"><br>> +// Test no config file found, WebKit fallback style is applied<br class="gmail_msg"><br>> +// RUN: clang-format -style=file -fallback-style=WebKit -assume-filename=%T/foo.cpp < %s 2>&1 | FileCheck -strict-whitespace -check-prefix=CHECK10 %s<br class="gmail_msg"><br>> +// Test no config file and no fallback style, LLVM style is applied<br class="gmail_msg"><br>> +// RUN: clang-format -style=file -assume-filename=%T/foo.cpp < %s 2>&1 | FileCheck -strict-whitespace -check-prefix=CHECK11 %s<br class="gmail_msg"><br>> +// Test no config file and fallback style "none", no formatting is applied<br class="gmail_msg"><br>> +// RUN: clang-format -style=file -fallback-style=none -assume-filename=%T/foo.cpp < %s 2>&1 | FileCheck -strict-whitespace -check-prefix=CHECK12 %s<br class="gmail_msg"><br>> +// Test config file with no based style, and fallback style "none", formatting is applied<br class="gmail_msg"><br>> +// RUN: printf "IndentWidth: 6\n" > %T/_clang-format<br class="gmail_msg"><br>> +// RUN: clang-format -style=file -fallback-style=none -assume-filename=%T/foo.cpp < %s 2>&1 | FileCheck -strict-whitespace -check-prefix=CHECK13 %s<br class="gmail_msg"><br>> +// Test yaml with no based style, and fallback style "none", LLVM formatting applied<br class="gmail_msg"><br>> +// RUN: clang-format -style="{IndentWidth: 7}" -fallback-style=none %s | FileCheck -strict-whitespace -check-prefix=CHECK14 %s<br class="gmail_msg"><br>> +<br class="gmail_msg"><br>>  void f() {<br class="gmail_msg"><br>>  // CHECK1: {{^        int\* i;$}}<br class="gmail_msg"><br>>  // CHECK2: {{^       int \*i;$}}<br class="gmail_msg"><br>> @@ -22,6 +37,11 @@ void f() {<br class="gmail_msg"><br>>  // CHECK7: {{^      int\* i;$}}<br class="gmail_msg"><br>>  // CHECK8: {{^  int\* i;$}}<br class="gmail_msg"><br>>  // CHECK9: {{^    int \*i;$}}<br class="gmail_msg"><br>> +// CHECK10: {{^    int\* i;$}}<br class="gmail_msg"><br>> +// CHECK11: {{^  int \*i;$}}<br class="gmail_msg"><br>> +// CHECK12: {{^int\*i;$}}<br class="gmail_msg"><br>> +// CHECK13: {{^      int \*i;$}}<br class="gmail_msg"><br>> +// CHECK14: {{^       int \*i;$}}<br class="gmail_msg"><br>>  int*i;<br class="gmail_msg"><br>>  int j;<br class="gmail_msg"><br>>  }<br class="gmail_msg"><br>><br class="gmail_msg"><br>> Modified: cfe/trunk/unittests/Format/FormatTest.cpp<br class="gmail_msg"><br>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Format/FormatTest.cpp?rev=292562&r1=292561&r2=292562&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Format/FormatTest.cpp?rev=292562&r1=292561&r2=292562&view=diff</a><br class="gmail_msg"><br>> ==============================================================================<br class="gmail_msg"><br>> --- cfe/trunk/unittests/Format/FormatTest.cpp (original)<br class="gmail_msg"><br>> +++ cfe/trunk/unittests/Format/FormatTest.cpp Thu Jan 19 19:22:42 2017<br class="gmail_msg"><br>> @@ -10978,13 +10978,31 @@ TEST(FormatStyle, GetStyleOfFile) {<br class="gmail_msg"><br>>    ASSERT_TRUE((bool)Style1);<br class="gmail_msg"><br>>    ASSERT_EQ(*Style1, getLLVMStyle());<br class="gmail_msg"><br>><br class="gmail_msg"><br>> -  // Test 2: fallback to default.<br class="gmail_msg"><br>> +  // Test 2.1: fallback to default.<br class="gmail_msg"><br>>    ASSERT_TRUE(<br class="gmail_msg"><br>>        FS.addFile("/b/test.cpp", 0, llvm::MemoryBuffer::getMemBuffer("int i;")));<br class="gmail_msg"><br>>    auto Style2 = getStyle("file", "/b/test.cpp", "Mozilla", "", &FS);<br class="gmail_msg"><br>>    ASSERT_TRUE((bool)Style2);<br class="gmail_msg"><br>>    ASSERT_EQ(*Style2, getMozillaStyle());<br class="gmail_msg"><br>><br class="gmail_msg"><br>> +  // Test 2.2: no format on 'none' fallback style.<br class="gmail_msg"><br>> +  Style2 = getStyle("file", "/b/test.cpp", "none", "", &FS);<br class="gmail_msg"><br>> +  ASSERT_TRUE((bool)Style2);<br class="gmail_msg"><br>> +  ASSERT_EQ(*Style2, getNoStyle());<br class="gmail_msg"><br>> +<br class="gmail_msg"><br>> +  // Test 2.3: format if config is found with no based style while fallback is<br class="gmail_msg"><br>> +  // 'none'.<br class="gmail_msg"><br>> +  ASSERT_TRUE(FS.addFile("/b/.clang-format", 0,<br class="gmail_msg"><br>> +                         llvm::MemoryBuffer::getMemBuffer("IndentWidth: 2")));<br class="gmail_msg"><br>> +  Style2 = getStyle("file", "/b/test.cpp", "none", "", &FS);<br class="gmail_msg"><br>> +  ASSERT_TRUE((bool)Style2);<br class="gmail_msg"><br>> +  ASSERT_EQ(*Style2, getLLVMStyle());<br class="gmail_msg"><br>> +<br class="gmail_msg"><br>> +  // Test 2.4: format if yaml with no based style, while fallback is 'none'.<br class="gmail_msg"><br>> +  Style2 = getStyle("{}", "a.h", "none", "", &FS);<br class="gmail_msg"><br>> +  ASSERT_TRUE((bool)Style2);<br class="gmail_msg"><br>> +  ASSERT_EQ(*Style2, getLLVMStyle());<br class="gmail_msg"><br>> +<br class="gmail_msg"><br>>    // Test 3: format file in parent directory.<br class="gmail_msg"><br>>    ASSERT_TRUE(<br class="gmail_msg"><br>>        FS.addFile("/c/.clang-format", 0,<br class="gmail_msg"><br>><br class="gmail_msg"><br>><br class="gmail_msg"><br>> _______________________________________________<br class="gmail_msg"><br>> cfe-commits mailing list<br class="gmail_msg"><br>> <a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a><br class="gmail_msg"><br>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="gmail_msg"><br>><br class="gmail_msg"><br></blockquote></div></div>