aboutsummaryrefslogtreecommitdiffstats
path: root/tests/parser/wordEncoderTest.cpp
diff options
context:
space:
mode:
authorJan Engelhardt <[email protected]>2024-05-21 13:41:50 +0000
committerGitHub <[email protected]>2024-05-21 13:41:50 +0000
commit15f3b945800962adc808a2138360b3fd2b6b0806 (patch)
treeaad66128cd73359bf36a415c99bb936de56ded17 /tests/parser/wordEncoderTest.cpp
parentFix OpenSSL initialization #299 (#300) (diff)
downloadvmime-15f3b945800962adc808a2138360b3fd2b6b0806.tar.gz
vmime-15f3b945800962adc808a2138360b3fd2b6b0806.zip
Resolve compiler warnings (#302)
* build: replace class noncopyable by C++11 deleted function declaration C++11 is mandatory since commit v0.9.2-48-g8564b2f8, therefore we can exercise the =delete keyword in class declarations to prohibit copying. * build: resolve -Woverloaded-virtual warnings context.hpp:109:26: warning: "virtual vmime::context& vmime::context::operator=(const vmime::context&)’ was hidden [-Woverloaded-virtual=] 109 | virtual context& operator=(const context& ctx); | ^~~~~~~~ generationContext.hpp:153:28: note: by ‘vmime::generationContext& vmime::generationContext::operator=(const vmime::generationContext&)’ 153 | generationContext& operator=(const generationContext& ctx); | ^~~~~~~~ AFAICS, there is no point in having "virtual" on an assignment operator. Any derived classes' operator= has different signature anyway. It is also the only class with a virtual operator=, so that's an indicator for oddness as well. * build: resolve -Wdeprecated-declarations warnings encoding.cpp: In static member function "static const vmime::encoding vmime::encoding::decideImpl(std::__cxx11::basic_string<char>::const_iterator, std::__cxx11::basic_string<char>::const_iterator)": encoding.cpp:161:29: warning: "std::binder2nd<_Operation> std::bind2nd(const _Operation&, const _Tp&) [with _Operation = less<unsigned char>; _Tp = int]" is deprecated: use "std::bind" instead [-Wdeprecated-declarations] 161 | std::bind2nd(std::less<unsigned char>(), 127) | ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++11 is mandatory, so just use a lambda already.
Diffstat (limited to 'tests/parser/wordEncoderTest.cpp')
0 files changed, 0 insertions, 0 deletions