include/comphelper/sequence.hxx | 2 ++
1 file changed, 2 insertions(+)
New commits:
commit 509b2039f72ff35c79bb6ff2f49525b9d1e9226c
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Mon Nov 1 18:48:04 2021 +0200
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Nov 2 06:49:21 2021 +0100
Use braces to workaround a strange GCC bug
Using gcc version 9.3.0 (Ubuntu 9.3.0-17ubuntu1~20.04):
[CXX] ucb/source/ucp/cmis/certvalidation_handler.cxx
In file included from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/detail/converter_lexical_streams.hpp:62,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/detail/converter_lexical.hpp:54,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/try_lexical_convert.hpp:44,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast.hpp:32,
from
/home/user/core.git/external/boost/include/boost/lexical_cast.hpp:30,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/format_date_parser.hpp:14,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/date_generator_parser.hpp:20,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/date_facet.hpp:25,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/gregorian/gregorian_io.hpp:16,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/gregorian/gregorian.hpp:31,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/posix_time/time_formatters.hpp:12,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/posix_time/posix_time.hpp:24,
from
/home/user/core.git/external/boost/include/boost/date_time/posix_time/posix_time.hpp:30,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time/local_time/local_time.hpp:11,
from
/home/user/core.git/workdir/UnpackedTarball/boost/boost/date_time.hpp:15,
from
/home/user/core.git/external/boost/include/boost/date_time.hpp:30,
from
/home/user/core.git/workdir/UnpackedTarball/libcmis/inc/libcmis/object.hxx:40,
from
/home/user/core.git/workdir/UnpackedTarball/libcmis/inc/libcmis/document.hxx:39,
from
/home/user/core.git/workdir/UnpackedTarball/libcmis/inc/libcmis/libcmis.hxx:34,
from
/home/user/core.git/ucb/source/ucp/cmis/certvalidation_handler.hxx:19,
from
/home/user/core.git/ucb/source/ucp/cmis/certvalidation_handler.cxx:23:
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/detail/lcast_unsigned_converters.hpp:
In member function ‘CharT* boost::detail::lcast_put_unsigned<Traits, T,
CharT>::main_convert_loop()’:
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/detail/lcast_unsigned_converters.hpp:149:17:
error: this ‘while’ clause does not guard... [-Werror=misleading-indentation]
149 | inline CharT* main_convert_loop() BOOST_NOEXCEPT {
| ^~~~~
/home/user/core.git/workdir/UnpackedTarball/boost/boost/lexical_cast/detail/lcast_unsigned_converters.hpp:150:17:
note: ...this statement, but the latter is misleadingly indented as if it were
guarded by the ‘while’
150 | while (main_convert_iteration());
| ^~~~~~
It surfaced after commit 00a76942e423589e3fdd4e059be753eab3393344,
apparently showing some version-specific compiler bug.
Change-Id: I92a989726cea5eafd762f724cfc0b3c1f986824c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124474
Reviewed-by: Aron Budea <aron.bu...@collabora.com>
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
Tested-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/include/comphelper/sequence.hxx b/include/comphelper/sequence.hxx
index 1aa3fbea803d..bc67559dfbee 100644
--- a/include/comphelper/sequence.hxx
+++ b/include/comphelper/sequence.hxx
@@ -122,7 +122,9 @@ namespace comphelper
inline css::uno::Sequence< DstType > arrayToSequence( const SrcType*
i_pArray, sal_Int32 nNum )
{
if constexpr (std::is_same_v< DstType, SrcType >)
+ {
return css::uno::Sequence< DstType >( i_pArray, nNum );
+ }
else
{
css::uno::Sequence< DstType > result( nNum );
