[haiku-bugs] [Haiku] #12741: [Patch] LocaleRoster.cpp: fix gcc6 build

  • From: "mt" <trac@xxxxxxxxxxxx>
  • Date: Mon, 02 May 2016 08:03:27 -0000

#12741: [Patch] LocaleRoster.cpp: fix gcc6 build
-----------------------------+------------------------------
 Reporter:  mt               |        Owner:  pulkomandy
     Type:  bug              |       Status:  new
 Priority:  normal           |    Milestone:  Unscheduled
Component:  Kits/Locale Kit  |      Version:  R1/Development
 Keywords:                   |   Blocked By:
 Blocking:                   |  Has a Patch:  0
 Platform:  All              |
-----------------------------+------------------------------
 In _InitializeCatalog(), gcc6 warns '-Werror=misleading-indentation'.
 I think we don't need ";" after if() at line 544.


 {{{
 C++ /home/haiku/haiku/haiku/generated-
 gcc6/objects/haiku/x86/release/kits/locale/LocaleRoster.o
 /home/haiku/haiku/haiku/src/kits/locale/LocaleRoster.cpp: In function
 'status_t _InitializeCatalog(void*)':
 /home/haiku/haiku/haiku/src/kits/locale/LocaleRoster.cpp:544:2: error:
 this 'if' clause does not guard... [-Werror=misleading-indentation]
   if (BEntry(info.name).GetRef(&ref) == B_OK && catalog->SetTo(ref) ==
 B_OK);
   ^~
 /home/haiku/haiku/haiku/src/kits/locale/LocaleRoster.cpp:545:3: note:
 ...this statement, but the latter is misleadingly indented as if it is
 guarded by the 'if'
    return B_OK;
    ^~~~~~
 cc1plus: all warnings being treated as errors

 }}}

--
Ticket URL: <https://dev.haiku-os.org/ticket/12741>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: