[ https://issues.apache.org/jira/browse/AVRO-3616?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17584084#comment-17584084 ]
ASF subversion and git services commented on AVRO-3616: ------------------------------------------------------- Commit d336c6c7b91f9f39de2a5e34ae55c1d7c118507b in avro's branch refs/heads/master from Martin Grigorov [ https://gitbox.apache.org/repos/asf?p=avro.git;h=d336c6c7b ] AVRO-3616: C++ Fix compilation warnings (#1836) * AVRO-3616: [C++]: Fix compilation warnings Replace <boost/test/included/unit_test_framework.hpp> include with <boost/test/included/unit_test.hpp> Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org> * AVRO-3616: [C++]: Fix compilation warnings Fix the order of constructor parameters Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org> * AVRO-3616: Enable -pedantic and -Werror for compiler flags Those should prevent from introducing new warnings. -Wextra reports "unused method arguments" and those could not be fixed without API break. Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org> * AVRO-3616: Use size_t consistently for node->leaves() Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org> Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org> > [C++]: Fix compilation warnings > ------------------------------- > > Key: AVRO-3616 > URL: https://issues.apache.org/jira/browse/AVRO-3616 > Project: Apache Avro > Issue Type: Improvement > Components: c++ > Affects Versions: 1.11.1 > Reporter: Martin Tzvetanov Grigorov > Assignee: Martin Tzvetanov Grigorov > Priority: Minor > Labels: pull-request-available > Time Spent: 1.5h > Remaining Estimate: 0h > > Building the C++ SDK prints two types of warnings: > > 1. Usage of deprecated Boost test related import > {code:java} > In file included from /usr/include/boost/config/header_deprecated.hpp:18, > from /usr/include/boost/test/included/unit_test_framework.hpp:12, > from /home/runner/work/avro/avro/lang/c++/test/SpecificTests.cc:19: > > /usr/include/boost/test/included/unit_test_framework.hpp:13:1: note: > #pragma message: This header is deprecated. Use This header is deprecated. > Please use <boost/test/included/unit_test.hpp> instead. instead. > 13 | BOOST_HEADER_DEPRECATED( "This header is deprecated. Please use > <boost/test/included/unit_test.hpp> instead." ) > | ^~~~~~~~~~~~~~~~~~~~~~~{code} > 2. Wrong order of constructor arguments > {code:java} > In file included from > /home/martin/git/apache/avro/lang/c++/impl/DataFile.cc:19:/home/martin/git/apache/avro/lang/c++/api/DataFile.hh: > In constructor ‘avro::DataFileReaderBase::DataFileReaderBase(const > char*)’:/home/martin/git/apache/avro/lang/c++/api/DataFile.hh:200:11: > warning: ‘avro::DataFileReaderBase::codec_’ will be initialized after > [-Wreorder] 200 | Codec codec_; | > ^~~~~~/home/martin/git/apache/avro/lang/c++/api/DataFile.hh:197:22: warning: > ‘const DecoderPtr avro::DataFileReaderBase::decoder_’ [-Wreorder] 197 | > const DecoderPtr decoder_; | > ^~~~~~~~/home/martin/git/apache/avro/lang/c++/impl/DataFile.cc:262:1: > warning: when initialized here [-Wreorder] 262 | > DataFileReaderBase::DataFileReaderBase(const char *filename) : > filename_(filename), stream_(fileSeekableInputStream(filename)), > codec_(NULL_CODEC), | ^~~~~~~~~~~~~~~~~~In file included from > /home/martin/git/apache/avro/lang/c++/impl/DataFile.cc:19:/home/martin/git/apache/avro/lang/c++/api/DataFile.hh: > In constructor > ‘avro::DataFileReaderBase::DataFileReaderBase(std::unique_ptr<avro::InputStream>)’:/home/martin/git/apache/avro/lang/c++/api/DataFile.hh:200:11: > warning: ‘avro::DataFileReaderBase::codec_’ will be initialized after > [-Wreorder] 200 | Codec codec_; | > ^~~~~~/home/martin/git/apache/avro/lang/c++/api/DataFile.hh:197:22: warning: > ‘const DecoderPtr avro::DataFileReaderBase::decoder_’ [-Wreorder] 197 | > const DecoderPtr decoder_; | > ^~~~~~~~/home/martin/git/apache/avro/lang/c++/impl/DataFile.cc:268:1: > warning: when initialized here [-Wreorder] 268 | > DataFileReaderBase::DataFileReaderBase(std::unique_ptr<InputStream> > inputStream) : stream_(std::move(inputStream)), codec_(NULL_CODEC), {code} > Full example could be seen at > https://github.com/apache/avro/runs/7959849252?check_suite_focus=true -- This message was sent by Atlassian Jira (v8.20.10#820010)