commit | 202702b0b20d0499877b50c5733879e409d43395 | [log] [tgz] |
---|---|---|
author | Jonathan Doman <jonathan.doman@intel.com> | Fri Sep 15 16:21:22 2023 -0700 |
committer | Jonathan Doman <jonathan.doman@intel.com> | Fri Sep 15 16:21:22 2023 -0700 |
tree | 91bb92ac5c0e4a1c84a188fc4278a050aa1cd483 | |
parent | 1b42a9c58f4bb6fc6c4f306e42c0544018064892 [diff] |
Fix std::optional maybe-uninitialized (maybe?) Attempt to fix the GCC waybe-uninitialized warning for unpackArgs. It only triggers on compilation of one command handler (ipmiSensorGetDeviceSdrInfo) with a single std::optional<uint8_t> param. The warning is a false positive (I think) but GCC seems to like it if we get rid of the empty emplace() call in the underlying std::optional unpack function. Tested: Compiles in the unit-test docker environment when configured with `meson setup --reconfigure build -Dwerror=true -Dwarning_level=3 --buildtype=debugoptimized` Change-Id: Ida8b82dbf2227d3b5339cd4b5756729eeeea9e1d Signed-off-by: Jonathan Doman <jonathan.doman@intel.com>
meson builddir ninja -C builddir
meson builddir -Dbuildtype=minsize -Db_lto=true -Dtests=disabled ninja -C builddir
If any of the dependencies are not found on the host system during configuration, meson automatically gets them via its wrap dependencies mentioned in ipmid/subprojects
.
meson builddir -Dwrap_mode=nofallback ninja -C builddir
meson builddir -Dbuildtype=debug ninja -C builddir
meson builddir -Db_coverage=true -Dtests=enabled ninja -C builddir test ninja -C builddir coverage