This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug c++/69237] [6 Regression] strange -Wmisleading-indentation warning when building Chromium


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.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]