Bug 197412 - WebCore::StyleColorScheme should not have explicitly-declared copy constructor
Summary: WebCore::StyleColorScheme should not have explicitly-declared copy constructor
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-04-30 06:17 PDT by Michael Catanzaro
Modified: 2019-04-30 07:33 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.76 KB, patch)
2019-04-30 06:19 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2019-04-30 06:17:05 PDT
Since we enabled dark mode CSS for GTK, I can't see any real warnings past this warning spam:

[1135/1984] Building CXX object Source/WebCore/CMakeFiles...s/WebCore/unified-sources/UnifiedSource-043dd90b-29.cpp.o
In file included from ../../Source/WebCore/rendering/RenderObject.h:35,
                 from ../../Source/WebCore/rendering/RenderElement.h:26,
                 from ../../Source/WebCore/rendering/RenderLayerModelObject.h:25,
                 from ../../Source/WebCore/rendering/RenderBoxModelObject.h:28,
                 from ../../Source/WebCore/rendering/RenderBox.h:26,
                 from ../../Source/WebCore/rendering/RenderLayer.h:51,
                 from ../../Source/WebCore/rendering/svg/SVGRenderingContext.cpp:32,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-043dd90b-29.cpp:1:
../../Source/WebCore/rendering/style/RenderStyle.h: In member function ‘void WebCore::RenderStyle::setColorScheme(WebCore::StyleColorScheme)’:
../../Source/WebCore/rendering/style/RenderStyle.h:1182:97: warning: implicitly-declared ‘constexpr WebCore::StyleColorScheme& WebCore::StyleColorScheme::operator=(const WebCore::StyleColorScheme&)’ is deprecated [-Wdeprecated-copy]
 1182 |     void setColorScheme(StyleColorScheme supported) { SET_VAR(m_rareInheritedData, colorScheme, supported); }
      |                                                                                                 ^~~~~~~~~
../../Source/WebCore/rendering/style/RenderStyle.h:94:39: note: in definition of macro ‘SET_VAR’
   94 |             group.access().variable = value; \
      |                                       ^~~~~
In file included from ../../Source/WebCore/rendering/style/StyleRareInheritedData.h:41,
                 from ../../Source/WebCore/rendering/style/RenderStyle.h:59,
                 from ../../Source/WebCore/rendering/RenderObject.h:35,
                 from ../../Source/WebCore/rendering/RenderElement.h:26,
                 from ../../Source/WebCore/rendering/RenderLayerModelObject.h:25,
                 from ../../Source/WebCore/rendering/RenderBoxModelObject.h:28,
                 from ../../Source/WebCore/rendering/RenderBox.h:26,
                 from ../../Source/WebCore/rendering/RenderLayer.h:51,
                 from ../../Source/WebCore/rendering/svg/SVGRenderingContext.cpp:32,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-043dd90b-29.cpp:1:
../../Source/WebCore/rendering/style/StyleColorScheme.h:39:5: note: because ‘WebCore::StyleColorScheme’ has user-provided ‘WebCore::StyleColorScheme::StyleColorScheme(const WebCore::StyleColorScheme&)’
   39 |     StyleColorScheme(const StyleColorScheme& other)
      |     ^~~~~~~~~~~~~~~~

The copy constructor should be removed.
Comment 1 Michael Catanzaro 2019-04-30 06:19:45 PDT
Created attachment 368555 [details]
Patch
Comment 2 Don Olmstead 2019-04-30 06:28:59 PDT
Comment on attachment 368555 [details]
Patch

r=me assuming bots are happy
Comment 3 WebKit Commit Bot 2019-04-30 07:32:48 PDT
Comment on attachment 368555 [details]
Patch

Clearing flags on attachment: 368555

Committed r244768: <https://trac.webkit.org/changeset/244768>
Comment 4 WebKit Commit Bot 2019-04-30 07:32:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2019-04-30 07:33:14 PDT
<rdar://problem/50334395>