From 5d93d153d92c30a9c0c2089bcc185ee41b34a8fe Mon Sep 17 00:00:00 2001 From: Arjen Hiemstra Date: Thu, 20 Mar 2025 14:19:21 +0100 Subject: [PATCH] [PATCH] Tweak compiler warning flags to reduce unneeded output -Wno-gnu-statement-expression-from-macro-expansion and -Wno-c99-extensions are non-existent options that add more noise than they fix. Additionally, -Wpedantic is enabled by default which causes a lot of noise, so disable that. Finally, add -Wno-missing-field-initializers to quiet down the final bits of PipeWire API. Gbp-Pq: Name upstream_686df1e3_Tweak-compiler-warning-flags-to-reduce-unneeded-output.patch --- src/CMakeLists.txt | 2 +- src/libwebpencoder.cpp | 7 ------- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 10d1a11..1ef842e 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -66,7 +66,7 @@ target_sources(KPipeWire PRIVATE ) # these are features pipewire use extensively and make it hard to look at compile logs -target_compile_options(KPipeWire PUBLIC -Wno-gnu-statement-expression-from-macro-expansion -Wno-zero-as-null-pointer-constant -Wno-c99-extensions -Wno-cast-align) +target_compile_options(KPipeWire PUBLIC -Wno-zero-as-null-pointer-constant -Wno-missing-field-initializers -Wno-cast-align -Wno-pedantic) target_link_libraries(KPipeWire PRIVATE Qt::Quick Qt::GuiPrivate KF6::I18n KF6::CoreAddons PkgConfig::PipeWire epoxy::epoxy PkgConfig::GBM Libdrm::Libdrm PkgConfig::LIBVA PkgConfig::LIBVA-drm) diff --git a/src/libwebpencoder.cpp b/src/libwebpencoder.cpp index 0006441..c62cecb 100644 --- a/src/libwebpencoder.cpp +++ b/src/libwebpencoder.cpp @@ -51,9 +51,6 @@ bool LibWebPEncoder::initialize(const QSize &size) m_avCodecContext->time_base = AVRational{1, 1000}; AVDictionary *options = nullptr; - - applyEncodingPreference(options); - if (int result = avcodec_open2(m_avCodecContext, codec, &options); result < 0) { qCWarning(PIPEWIRERECORD_LOGGING) << "Could not open codec" << av_err2str(result); return false; @@ -66,7 +63,3 @@ int LibWebPEncoder::percentageToAbsoluteQuality(const std::optional &qua { return quality.value_or(-1); // Already 0-100. -1 resets to default. } - -void LibWebPEncoder::applyEncodingPreference([[maybe_unused]] AVDictionary *options) -{ -} -- 2.30.2