Bug 227348

Summary: [GStreamer] Build warnings in AudioFileReader since r279123
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: PlatformAssignee: Philippe Normand <pnormand>
Status: RESOLVED FIXED    
Severity: Normal CC: calvaris, eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio, vjaquez, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=227208
Attachments:
Description Flags
Patch
none
Patch none

Philippe Normand
Reported 2021-06-24 04:49:17 PDT
[2156/3527] Building CXX object Source/WebCore/CMakeFiles/WebCore.dir/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp.o ../../Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:351:9: warning: variable 'source' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] if (m_data) { ^~~~~~ ../../Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:361:131: note: uninitialized use occurs here gst_bin_add_many(((((GstBin*) g_type_check_instance_cast ((GTypeInstance*) ((m_pipeline.get())), ((gst_bin_get_type ())))))), source, m_decodebin.get(), nullptr); ^~~~~~ ../../Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:351:5: note: remove the 'if' if its condition is always true if (m_data) { ^~~~~~~~~~~~ ../../Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:350:23: note: initialize the variable 'source' to silence this warning GstElement* source; ^ = nullptr 1 warning generated.
Attachments
Patch (2.13 KB, patch)
2021-06-24 04:51 PDT, Philippe Normand
no flags
Patch (2.59 KB, patch)
2021-06-24 04:55 PDT, Philippe Normand
no flags
Philippe Normand
Comment 1 2021-06-24 04:51:41 PDT
Philippe Normand
Comment 2 2021-06-24 04:55:29 PDT
EWS
Comment 3 2021-06-25 08:12:36 PDT
Committed r279283 (239162@main): <https://commits.webkit.org/239162@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 432151 [details].
Radar WebKit Bug Importer
Comment 4 2021-06-25 08:13:18 PDT
Note You need to log in before you can comment on or make changes to this bug.