Bug 210429 - Fix various build warnings
Summary: Fix various build warnings
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-04-13 08:43 PDT by Michael Catanzaro
Modified: 2020-08-17 10:14 PDT (History)
18 users (show)

See Also:


Attachments
Patch (6.42 KB, patch)
2020-04-13 08:48 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 2020-04-13 08:43:04 PDT
Fix various build warnings:

[269/1615] Building CXX object Source/JavaScriptCore/CMak...ScriptCore/unified-sources/UnifiedSource-bfc896e1-2.cpp.o
In file included from DerivedSources/JavaScriptCore/unified-sources/UnifiedSource-bfc896e1-2.cpp:2:
/home/mcatanzaro/Projects/WebKit/Source/JavaScriptCore/dfg/DFGArrayMode.cpp: In member function ‘bool JSC::DFG::ArrayMode::alreadyChecked(JSC::DFG::Graph&, JSC::DFG::Node*, const JSC::DFG::AbstractValue&) const’:
/home/mcatanzaro/Projects/WebKit/Source/JavaScriptCore/dfg/DFGArrayMode.cpp:565:9: warning: this statement may fall through [-Wimplicit-fallthrough=]
  565 |         }
      |         ^
/home/mcatanzaro/Projects/WebKit/Source/JavaScriptCore/dfg/DFGArrayMode.cpp:567:5: note: here
  567 |     case Array::DirectArguments:
      |     ^~~~

[877/1615] Building CXX object Source/WebCore/CMakeFiles/...s/WebCore/unified-sources/UnifiedSource-950a39b6-27.cpp.o
In file included from DerivedSources/WebCore/unified-sources/UnifiedSource-950a39b6-27.cpp:7:
/home/mcatanzaro/Projects/WebKit/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp: In member function ‘void WebCore::WebGLRenderingContextBase::texImage2DBase(GCGLenum, GCGLint, GCGLenum, GCGLsizei, GCGLsizei, GCGLint, GCGLenum, GCGLenum, GCGLsizei, const void*)’:
/home/mcatanzaro/Projects/WebKit/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp:4196:198: warning: unused parameter ‘byteLength’ [-Wunused-parameter]
 4196 | void WebGLRenderingContextBase::texImage2DBase(GCGLenum target, GCGLint level, GCGLenum internalFormat, GCGLsizei width, GCGLsizei height, GCGLint border, GCGLenum format, GCGLenum type, GCGLsizei byteLength, const void* pixels)
      |                                                                                                                                                                                            ~~~~~~~~~~^~~~~~~~~~

[1072/1615] Generating ../../DerivedSources/webkit2gtk/WebKitWaylandClientProtocol.c
Using "code" is deprecated - use private-code or public-code.
See the help page for details.
[1073/1615] Generating ../../DerivedSources/webkit2gtk/pointer-constraints-unstable-v1-protocol.c
Using "code" is deprecated - use private-code or public-code.
See the help page for details.
[1074/1615] Generating ../../DerivedSources/webkit2gtk/relative-pointer-unstable-v1-protocol.c
Using "code" is deprecated - use private-code or public-code.
See the help page for details.
Comment 1 Michael Catanzaro 2020-04-13 08:48:28 PDT
Created attachment 396289 [details]
Patch
Comment 2 Mark Lam 2020-04-13 09:05:59 PDT
Comment on attachment 396289 [details]
Patch

r=me
Comment 3 EWS 2020-04-13 09:33:38 PDT
Committed r260012: <https://trac.webkit.org/changeset/260012>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 396289 [details].
Comment 4 Radar WebKit Bug Importer 2020-04-13 09:34:13 PDT
<rdar://problem/61719974>
Comment 5 Mike Gorse 2020-08-17 10:14:08 PDT
FYI, --private-code was added in wayland-scanner 1.16.0, so the build now fails if wayland is older than that.