gnucash master: Multiple changes pushed
John Ralls
jralls at code.gnucash.org
Sat Mar 18 16:55:26 EDT 2023
Updated via https://github.com/Gnucash/gnucash/commit/00c560d8 (commit)
via https://github.com/Gnucash/gnucash/commit/352ab8a0 (commit)
via https://github.com/Gnucash/gnucash/commit/24f52d9b (commit)
via https://github.com/Gnucash/gnucash/commit/b5f3f990 (commit)
from https://github.com/Gnucash/gnucash/commit/f6e5d209 (commit)
commit 00c560d8f5fddb2516baf75d9371db743bbe5dcb
Author: John Ralls <jralls at ceridwen.us>
Date: Sat Mar 18 13:53:49 2023 -0700
Move operator==(const GncGUID&, const GncGUID&) to guid.[ch]pp.
commit 352ab8a0e56c343640394208cf6d8dba282edf3c
Merge: f6e5d209cd 24f52d9b17
Author: John Ralls <jralls at ceridwen.us>
Date: Sat Mar 18 12:19:50 2023 -0700
Merge Richard Cohen's 'minor-options-fixes' into master.
commit 24f52d9b171f5474a4d1e8c2f10a198dc462ae65
Author: Richard Cohen <richard at daijobu.co.uk>
Date: Tue Mar 7 11:44:20 2023 +0000
Remove incorrect operator!= for GncItem
It's better to use the default std::pair implementation
- spotted by clang-tidy bugprone-suspicious-string-compare
- will always return false, because type is the same
Also,
- remove redundant "using GncItem ..."
commit b5f3f99035a763e41217cfd7b792e4669ce17ba7
Author: Richard Cohen <richard at daijobu.co.uk>
Date: Tue Mar 7 11:27:21 2023 +0000
Fix incorrect use of remove_if()
- spotted by clang-tidy bugprone
- probably no visible effects because of uniqueness, and other checks
libgnucash/engine/gnc-optiondb.cpp:149:5: warning: this call will remove at most one item even when multiple items should be removed [bugprone-inaccurate-erase]
m_options.erase(std::remove_if(m_options.begin(), m_options.end(),
^
libgnucash/engine/gnc-optiondb.cpp:358:5: warning: the value returned by this function should be used [bugprone-unused-return-value]
std::remove_if(m_callbacks.begin(), m_callbacks.end(),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
libgnucash/engine/gnc-optiondb.cpp:358:5: note: cast the expression to void to silence this warning
Summary of changes:
libgnucash/engine/gnc-option-impl.cpp | 16 ----------------
libgnucash/engine/gnc-optiondb.cpp | 7 ++++---
libgnucash/engine/guid.cpp | 7 +++++++
libgnucash/engine/guid.hpp | 4 +++-
4 files changed, 14 insertions(+), 20 deletions(-)
More information about the gnucash-patches
mailing list