This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug c++/69237] [6 Regression] strange -Wmisleading-indentation warning when building Chromium
- From: "dmalcolm at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Tue, 12 Jan 2016 14:06:54 +0000
- Subject: [Bug c++/69237] [6 Regression] strange -Wmisleading-indentation warning when building Chromium
- Auto-submitted: auto-generated
- References: <bug-69237-4 at http dot gcc dot gnu dot org/bugzilla/>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69237
David Malcolm <dmalcolm at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |UNCONFIRMED
Resolution|WONTFIX |---
--- Comment #6 from David Malcolm <dmalcolm at gcc dot gnu.org> ---
(In reply to Markus Trippelsdorf from comment #0)
[...snip...]
> In file included from ../../third_party/skia/include/core/SkPathRef.h:17:0,
> from ../../third_party/skia/include/core/SkPath.h:12,
> from
> ../../third_party/WebKit/Source/platform/graphics/GraphicsTypes.h:32,
> from
> ../../third_party/WebKit/Source/platform/graphics/Image.h:33,
> from
> ../../third_party/WebKit/Source/core/style/StyleImage.h:31,
> from
> ../../third_party/WebKit/Source/core/style/NinePieceImage.h:30,
> from
> ../../third_party/WebKit/Source/core/style/ComputedStyle.h:38,
> from
> ../../third_party/WebKit/Source/core/layout/LayoutObject.h:46,
> from
> ../../third_party/WebKit/Source/core/animation/css/CSSAnimationUpdate.h:15,
> from
> ../../third_party/WebKit/Source/core/animation/css/CSSAnimations.h:37,
> from
> ../../third_party/WebKit/Source/core/animation/ElementAnimations.h:35,
> from
> ../../third_party/WebKit/Source/core/animation/ElementAnimation.h:37,
> from gen/blink/bindings/core/v8/V8Element.cpp:35,
> from
> gen/blink/bindings/core/v8/V8GeneratedCoreBindings03.cpp:37:
> ../../third_party/skia/include/core/SkTDArray.h: In member function âvoid
> SkTDArray<T>::pop(T*)â:
> ../../third_party/skia/include/core/SkTDArray.h:270:0: warning: statement is
> indented as if it were guarded by... [-Wmisleading-indentation]
> void pop(T* elem) { SkASSERT(fCount > 0); if (elem) *elem =
> (*this)[fCount - 1]; --fCount; }
>
>
> ../../third_party/skia/include/core/SkTDArray.h:270:0: note: ...this âifâ
> clause, but it is not
>
> Unfortunately when adding --save-temps, the issue vanishes.
Thanks for reporting this.
Note that many of the locations are reported as being in column 0: in
particular, for the location of the diagnostic itself. This suggests that
location_t values are >= LINE_MAP_MAX_LOCATION_WITH_COLS.
Hence I believe that this is a duplicate of PR preprocessor/69177 and PR
c++/68819. This would be consistent with the issue going away with
--save-temps.