The value is: - currently unused, and useless without [[nodiscard]] - confusing, because it is always true, unless a num-string is set Instead of adding [[nodiscard]], throw when setting is not possible.
Backport of core#25736.
Depends on D16283.
Differential D16284
univalue: Remove unused and confusing set*() return value Fabien on Jun 5 2024, 11:58. Authored by Tags None Subscribers None
Details
The value is: - currently unused, and useless without [[nodiscard]] - confusing, because it is always true, unless a num-string is set Instead of adding [[nodiscard]], throw when setting is not possible. Backport of core#25736. Depends on D16283. ninja all check-all
Diff Detail
|