diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2021-09-14 17:26:17 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2022-06-10 17:15:49 +0200 |
commit | a8485d558fab53291e2530fd9a1be581c1628deb (patch) | |
tree | dff1f2885f954b9da2d3d2dbb9adc20b5b69c69c /compilerplugins | |
parent | 611fe67394820e589e86f543c614632fc273eb5e (diff) |
[API CHANGE] Remove deprecated idlc and regmerge from the SDK
* Client code must replace uses of idlc and regmerge with uses of unoidl-write,
see the changes to odk/examples/ and ure/source/uretext/ in
40f2aee6584eafcf4cd1d95fcf1f775e5435440d "Provide unoidl-write also for the
SDK" for examples.
* The new types.rdb format is not compatible with LibreOffice < 4.1. Clients
generating extensions containing such files are advised to use appropriate
LibreOffice-minimal-version elements.
* For compatibility with old extensions, reading the legacy types.rdb format is
still supported.
* The SDK no longer ships an idl/ sub-directory containing the udkap and offapi
.idl files (as, unlike idlc, unoidl-write does not need them).
odk/config/cfgWin.js had to be adapted to look (somewhat arbitrarily) for an
examples/ sub-directory instead of idl/ when checking for "an sdk folder".
gb_UnoApi_package_idlfiles became unused and has been removed.
* The idlc and regmerge executables have been removed. Module idlc has been
removed except for idlc/test/parser/, which is also used by
CustomTarget_unoidl/unoidl-write_test, and which may eventually be moved into
module unoidl. Module external/ucpp and the corresponding configure options
have also been removed.
Change-Id: I42a0231699b863b5ebe2bee63bc32c8f79278cc1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122363
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins')
-rw-r--r-- | compilerplugins/clang/shouldreturnbool.cxx | 3 | ||||
-rw-r--r-- | compilerplugins/clang/unnecessaryparen.cxx | 2 | ||||
-rw-r--r-- | compilerplugins/clang/useuniqueptr.cxx | 12 | ||||
-rw-r--r-- | compilerplugins/clang/writeonlyvars.cxx | 3 |
4 files changed, 0 insertions, 20 deletions
diff --git a/compilerplugins/clang/shouldreturnbool.cxx b/compilerplugins/clang/shouldreturnbool.cxx index bc232b472e5e..fa1bd4cbdfb3 100644 --- a/compilerplugins/clang/shouldreturnbool.cxx +++ b/compilerplugins/clang/shouldreturnbool.cxx @@ -47,9 +47,6 @@ public: // false +, slightly odd usage, but not wrong if (loplugin::isSamePathname(fn, SRCDIR "/libreofficekit/qa/tilebench/tilebench.cxx")) return; - // uses the Unix convention of "non-zero return indicates error" - if (loplugin::isSamePathname(fn, SRCDIR "/idlc/source/idlcproduce.cxx")) - return; // template magic if (loplugin::isSamePathname(fn, SRCDIR "/vcl/source/gdi/bmpfast.cxx")) return; diff --git a/compilerplugins/clang/unnecessaryparen.cxx b/compilerplugins/clang/unnecessaryparen.cxx index 29a790702c66..3b9e5f14bbbb 100644 --- a/compilerplugins/clang/unnecessaryparen.cxx +++ b/compilerplugins/clang/unnecessaryparen.cxx @@ -83,8 +83,6 @@ public: // fixing this, makes the source in the .y files look horrible if (loplugin::isSamePathname(fn, WORKDIR "/YaccTarget/unoidl/source/sourceprovider-parser.cxx")) return false; - if (loplugin::isSamePathname(fn, WORKDIR "/YaccTarget/idlc/source/parser.cxx")) - return false; return true; } virtual void run() override diff --git a/compilerplugins/clang/useuniqueptr.cxx b/compilerplugins/clang/useuniqueptr.cxx index c0ef34b2823a..cf4853f03b2b 100644 --- a/compilerplugins/clang/useuniqueptr.cxx +++ b/compilerplugins/clang/useuniqueptr.cxx @@ -463,9 +463,6 @@ void UseUniquePtr::CheckDeleteLocalVar(const FunctionDecl* functionDecl, const C // complicated if (fn == SRCDIR "/svx/source/sdr/contact/objectcontact.cxx") return; - // memory management in this module is a mess - if (fn == SRCDIR "/idlc/source/aststack.cxx") - return; // complicated if (fn == SRCDIR "/cui/source/customize/cfg.cxx") return; @@ -728,9 +725,6 @@ void UseUniquePtr::CheckLoopDelete(const FunctionDecl* functionDecl, const CXXDe if (loplugin::hasPathnamePrefix(fn, SRCDIR "/vcl/qa/")) return; // linked list - if (fn == SRCDIR "/registry/source/reflwrit.cxx") - return; - // linked list if (fn == SRCDIR "/tools/source/generic/config.cxx") return; // linked lists @@ -848,9 +842,6 @@ void UseUniquePtr::CheckCXXForRangeStmt(const FunctionDecl* functionDecl, const if (!fieldDecl) return; - // appears to just randomly leak stuff, and it involves some lex/yacc stuff - if (fn == SRCDIR "/idlc/source/aststack.cxx") - return; // complicated if (fn == SRCDIR "/vcl/source/gdi/print.cxx") return; @@ -1092,9 +1083,6 @@ bool UseUniquePtr::TraverseConstructorInitializer(CXXCtorInitializer * ctorInit) return true; StringRef fn = getFilenameOfLocation(compiler.getSourceManager().getSpellingLoc(ctorInit->getSourceLocation())); - // don't feel like fiddling with the yacc parser - if (loplugin::hasPathnamePrefix(fn, SRCDIR "/idlc/")) - return true; // cannot change URE if (loplugin::hasPathnamePrefix(fn, SRCDIR "/cppu/source/helper/purpenv/helper_purpenv_Environment.cxx")) return true; diff --git a/compilerplugins/clang/writeonlyvars.cxx b/compilerplugins/clang/writeonlyvars.cxx index f5051c99e128..3ea5356e4071 100644 --- a/compilerplugins/clang/writeonlyvars.cxx +++ b/compilerplugins/clang/writeonlyvars.cxx @@ -185,9 +185,6 @@ void WriteOnlyVars::run() // false+ if (loplugin::isSamePathname(fn, SRCDIR "/store/source/storpage.cxx")) return; - // yydebug? - if (loplugin::isSamePathname(fn, SRCDIR "/idlc/source/idlccompile.cxx")) - return; if (fn.contains("/qa/")) return; if (fn.contains("/vcl/workben/")) |