diff options
author | Jan Engelhardt <[email protected]> | 2024-05-21 13:41:50 +0000 |
---|---|---|
committer | GitHub <[email protected]> | 2024-05-21 13:41:50 +0000 |
commit | 15f3b945800962adc808a2138360b3fd2b6b0806 (patch) | |
tree | aad66128cd73359bf36a415c99bb936de56ded17 /tests/parser/wordEncoderTest.cpp | |
parent | Fix OpenSSL initialization #299 (#300) (diff) | |
download | vmime-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