[Libreoffice-commits] core.git: 2 commits - compilerplugins/clang compilerplugins/Makefile-clang.mk
Luboš Luňák (via logerrit)
logerrit at kemper.freedesktop.org
Sun Oct 6 14:58:30 UTC 2019
compilerplugins/Makefile-clang.mk | 2 +-
compilerplugins/clang/flatten.cxx | 10 ++++++++--
2 files changed, 9 insertions(+), 3 deletions(-)
New commits:
commit 3e5d4ecbde512efe169536544489635f7f076fd2
Author: Luboš Luňák <l.lunak at collabora.com>
AuthorDate: Sun Oct 6 15:01:23 2019 +0200
Commit: Luboš Luňák <l.lunak at collabora.com>
CommitDate: Sun Oct 6 16:57:34 2019 +0200
the PCH in compilerplugins should also depend on the Makefile
Otherwise it may not get rebuilt in Jenkins when flags in the Makefile
change.
Change-Id: I9a06c5bbfd4af2714497c5658d8c523a6c929f68
Reviewed-on: https://gerrit.libreoffice.org/80318
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak at collabora.com>
diff --git a/compilerplugins/Makefile-clang.mk b/compilerplugins/Makefile-clang.mk
index 1a2ceef369df..6e18ee55487a 100644
--- a/compilerplugins/Makefile-clang.mk
+++ b/compilerplugins/Makefile-clang.mk
@@ -306,7 +306,7 @@ LO_CLANG_ANALYZER_PCH_CXXFLAGS := -I$(BUILDDIR)/config_host -I$(CLANGDIR)/includ
-D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
$(CLANGOUTDIR)/sharedvisitor/clang.pch: $(CLANGINDIR)/sharedvisitor/precompiled_clang.hxx \
- $(CLANGOUTDIR)/clang-timestamp \
+ $(SRCDIR)/compilerplugins/Makefile-clang.mk $(CLANGOUTDIR)/clang-timestamp \
| $(CLANGOUTDIR)/sharedvisitor
$(call gb_Output_announce,$(subst $(BUILDDIR)/,,$@),$(true),PCH,1)
$(QUIET)$(CLANGDIR)/bin/clang -x c++-header $(LO_CLANG_ANALYZER_PCH_CXXFLAGS) \
commit 1cccad3dd24e4a5cf5d7299326ae302e71980971
Author: Luboš Luňák <l.lunak at collabora.com>
AuthorDate: Sat Oct 5 13:27:55 2019 +0200
Commit: Luboš Luňák <l.lunak at collabora.com>
CommitDate: Sun Oct 6 16:57:23 2019 +0200
replace throw with abort
There is nothing catching the exception anyway, and this fails
to compile if compiling the plugin with Clang's CXXFLAGS (which
include -fno-exceptions).
Change-Id: Iad9316ad9613e4fd66d0e5a16fd71bbb8066cc2b
Reviewed-on: https://gerrit.libreoffice.org/80299
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak at collabora.com>
diff --git a/compilerplugins/clang/flatten.cxx b/compilerplugins/clang/flatten.cxx
index e56d21603774..81f897cb0ee3 100644
--- a/compilerplugins/clang/flatten.cxx
+++ b/compilerplugins/clang/flatten.cxx
@@ -474,7 +474,10 @@ std::string stripOpenAndCloseBrace(std::string s)
{
size_t i = s.find("{");
if (i == std::string::npos)
- throw "did not find {";
+ {
+ assert( !"did not find {" );
+ abort();
+ }
++i;
// strip to line end
@@ -486,7 +489,10 @@ std::string stripOpenAndCloseBrace(std::string s)
i = s.rfind("}");
if (i == std::string::npos)
- throw "did not find }";
+ {
+ assert( !"did not find }" );
+ abort();
+ }
--i;
while (s[i] == ' ')
--i;
More information about the Libreoffice-commits
mailing list