Bug 174511

Summary: -Wreorder warning caused by GraphicsContext3D cleanup
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: WebCore Misc.Assignee: Michael Catanzaro <mcatanzaro>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, buildbot, cgarcia, commit-queue, dino, graouts, kondapallykalyan, magomez, mcatanzaro, noam, zan
Priority: P2    
Version: Other   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch
none
Patch none

Description Michael Catanzaro 2017-07-14 10:38:59 PDT
Building CXX object Source...ics/cairo/GraphicsContext3DCairo.cpp.o
In file included from ../../Source/WebCore/platform/graphics/cairo/GraphicsContext3DCairo.cpp:33:0:
../../Source/WebCore/platform/graphics/GraphicsContext3D.h: In constructor ‘WebCore::GraphicsContext3D::GraphicsContext3D(WebCore::GraphicsContext3DAttributes, WebCore::HostWindow*, WebCore::GraphicsContext3D::RenderStyle)’:
../../Source/WebCore/platform/graphics/GraphicsContext3D.h:1394:38: warning: ‘WebCore::GraphicsContext3D::m_compositorTexture’ will be initialized after [-Wreorder]
     GC3Duint m_compositorTexture { 0 };
                                      ^
../../Source/WebCore/platform/graphics/GraphicsContext3D.h:1392:24: warning:   ‘GC3Duint WebCore::GraphicsContext3D::m_fbo’ [-Wreorder]
     GC3Duint m_fbo { 0 };
                        ^
../../Source/WebCore/platform/graphics/cairo/GraphicsContext3DCairo.cpp:88:1: warning:   when initialized here [-Wreorder]
 GraphicsContext3D::GraphicsContext3D(GraphicsContext3DAttributes attributes, HostWindow*, GraphicsContext3D::RenderStyle renderStyle)
 ^~~~~~~~~~~~~~~~~
Comment 1 Michael Catanzaro 2017-07-14 10:54:58 PDT
Created attachment 315448 [details]
Patch
Comment 2 Michael Catanzaro 2017-07-14 11:04:05 PDT
Created attachment 315451 [details]
Patch
Comment 3 WebKit Commit Bot 2017-07-16 23:58:51 PDT
Comment on attachment 315451 [details]
Patch

Clearing flags on attachment: 315451

Committed r219553: <http://trac.webkit.org/changeset/219553>
Comment 4 WebKit Commit Bot 2017-07-16 23:58:53 PDT
All reviewed patches have been landed.  Closing bug.