diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2019-02-01 14:17:42 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2019-02-01 19:03:36 +0100 |
| commit | 07d04fd3ae72dc1c9998f83a4131407ecc39cda9 (patch) | |
| tree | 631096befc3ac2c7da3ce8af84cce3c604ff21ce /io | |
| parent | remove unnecessary SAL_WARN (diff) | |
| download | core-07d04fd3ae72dc1c9998f83a4131407ecc39cda9.tar.gz core-07d04fd3ae72dc1c9998f83a4131407ecc39cda9.zip | |
Better fix for missing [[noreturn]] of CPPUNIT_FAIL
This includes a revert of commit 9808486a89c6368f836579f8d8c0dda63fd0063c "Avoid
-Werror,-Wimplicit-fallthrough with clang-cl", and also undoes older MSVC
workarounds (that would start to cause "warning C4702: unreachable code" now).
Change-Id: If72bf336e12e0a2db589857e8875003dae861977
Reviewed-on: https://gerrit.libreoffice.org/67248
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'io')
| -rw-r--r-- | io/qa/textinputstream.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/io/qa/textinputstream.cxx b/io/qa/textinputstream.cxx index 75371a0d3863..6e67f5e986ed 100644 --- a/io/qa/textinputstream.cxx +++ b/io/qa/textinputstream.cxx @@ -43,7 +43,7 @@ private: sal_Int32 SAL_CALL readBytes(css::uno::Sequence<sal_Int8> &, sal_Int32) override - { CPPUNIT_FAIL("readLine is supposed to call readSomeBytes instead"); return 0;} + { CPPUNIT_FAIL("readLine is supposed to call readSomeBytes instead"); } sal_Int32 SAL_CALL readSomeBytes( css::uno::Sequence<sal_Int8 > & aData, sal_Int32 nMaxBytesToRead) override |