summaryrefslogtreecommitdiffstats
path: root/idlc
Commit message (Collapse)AuthorAgeFilesLines
* Use even more WIN32_LEAN_AND_MEANMike Kaganski2017-09-181-15/+3
| | | | | | | Change-Id: I538fe5b41156366e0e87b3a93e58a3947afd18f5 Reviewed-on: https://gerrit.libreoffice.org/42398 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* clang-tidy modernize-use-emplace in hwpfilter..lotuswordproNoel Grandin2017-09-113-8/+7
| | | | | | | Change-Id: Ic11634ca28396fd156390c511087bae03bd5fb70 Reviewed-on: https://gerrit.libreoffice.org/42156 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryparen include c++ castsNoel Grandin2017-09-041-1/+1
| | | | | | | Change-Id: I132d3c66f0562e2c37a02eaf4c168d06c2b473eb Reviewed-on: https://gerrit.libreoffice.org/41874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert std::map::insert to std::map::emplace IINoel Grandin2017-08-111-13/+7
| | | | | | | Change-Id: Ief8bd59c903625ba65b75114b7b52c3b7ecbd331 Reviewed-on: https://gerrit.libreoffice.org/41019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:constparams in sw part6Noel Grandin2017-08-063-5/+5
| | | | | | | Change-Id: Ic82946cf9be50d9d9c43338b86dfb700ce5b354c Reviewed-on: https://gerrit.libreoffice.org/40786 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* no copy neededJochen Nitschke2017-07-181-52/+2
| | | | | | | | | coerce_value(..) is a local static function without side effects Change-Id: Id153ad26973b5724b33f144ae9eba49562b15067 Reviewed-on: https://gerrit.libreoffice.org/40040 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
* loplugin:constparams codemaker,idl,idlcNoel Grandin2017-07-187-12/+12
| | | | | | | Change-Id: Ic72b7525cfbfbd3c5afb12fe9ea814176e6c8c9d Reviewed-on: https://gerrit.libreoffice.org/40126 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* extend loplugin useuniqueptr to OUString pointersNoel Grandin2017-07-172-2/+2
| | | | | | | Change-Id: Ieb5bab3895e1edaff497c4a1a88303ccac097edc Reviewed-on: https://gerrit.libreoffice.org/39948 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix lifecycle issueStephan Bergmann2017-07-162-4/+6
| | | | | | | | | ...as shown by ASan/UBSan build use-after-free report at <https://ci.libreoffice.org/job/lo_ubsan/602/> after 8e39ef66928a3e37c618d3a70a631e71266db274 "extend loplugin useuniqueptr to POD types" Change-Id: Ic42e408c2c4b2901cdf5d681b332543ec8f445ef
* extend loplugin useuniqueptr to POD typesNoel Grandin2017-07-142-13/+13
| | | | | | | Change-Id: I6ff24f048bd8f75bf87a78b718f37b57855d4781 Reviewed-on: https://gerrit.libreoffice.org/39932 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryparenStephan Bergmann2017-07-051-3/+3
| | | | Change-Id: I37833e0e19162790f4ae37d74eb63cb84434cf2e
* loplugin:oncevar in helpcompiler..jvmfwkNoel Grandin2017-06-252-4/+3
| | | | | | | Change-Id: Ia9b20a8ca95684cbeb21e3425972c43ba50df3cd Reviewed-on: https://gerrit.libreoffice.org/39187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedfields in forms..idlcNoel Grandin2017-06-232-4/+0
| | | | | | | Change-Id: If250ff47c1f375fe24c61b5bf271da9a9c330822 Reviewed-on: https://gerrit.libreoffice.org/39133 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* More tests to suppress (all .PHONY test targets should be covered now)Stephan Bergmann2017-06-091-0/+4
| | | | Change-Id: Ib948547e5c177ac62506fc42a564368a8d31fa4e
* remove some unnecessary OString constructor callsNoel Grandin2017-05-291-12/+12
| | | | | | | Change-Id: I11358f17770cacb2a33011797f4be82dc02340b7 Reviewed-on: https://gerrit.libreoffice.org/38130 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove unnecessary use of OString::getStrNoel Grandin2017-05-283-8/+6
| | | | | | | Change-Id: I0490efedf459190521f4339854b3394d57765fdb Reviewed-on: https://gerrit.libreoffice.org/38058 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove unnecessary empty OUString fields and varsNoel Grandin2017-05-094-24/+18
| | | | | | | Change-Id: I940120087a0bc6b1b0abc30a3e7727ce22b7d9a7 Reviewed-on: https://gerrit.libreoffice.org/37394 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:redundantcast check for c-style casts to voidNoel Grandin2017-04-061-1/+1
| | | | | | | Change-Id: Ic8b99f590436f94825e471bc61411c69fd768862 Reviewed-on: https://gerrit.libreoffice.org/36208 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:redundantcast find cstyle double castsNoel Grandin2017-04-061-1/+1
| | | | | | | Change-Id: I5507be190dac781e5cdb545a60acf3d50056c9f8 Reviewed-on: https://gerrit.libreoffice.org/36187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2017-03-251-1/+1
| | | | | | | Change-Id: I389542e18fcf71011f22919743b5280779f449fd Reviewed-on: https://gerrit.libreoffice.org/35632 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* Remove unused #include <ctype.h>Stephan Bergmann2017-03-231-1/+0
| | | | Change-Id: I8bf3e30687e20151a9e1936e69362abfe9b3a99d
* Use rtl::isAscii* instead of ctype.h is* (and fix passing plain char)Stephan Bergmann2017-03-221-1/+3
| | | | Change-Id: I9d3cf6363e1caf66a9d4cc3a1560c93e17038c2d
* Remove redundant 'inline' keywordStephan Bergmann2017-03-031-1/+1
| | | | | | | | | | ...from function definitions occurring within class definitions. Done with a rewriting Clang plugin (to be pushed later). Change-Id: I9c6f2818a57ccdb361548895a7743107cbacdff8 Reviewed-on: https://gerrit.libreoffice.org/34874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Idlc::m_pErrorHandler is unusedStephan Bergmann2017-02-212-3/+0
| | | | Change-Id: I41fea5e08381e21dff12f61142a4d54f07a502f4
* loplugin:subtlezeroinit: idlcStephan Bergmann2017-02-213-13/+13
| | | | Change-Id: Ifec509c4d96a932cfe591b5ffa77027b7b372d52
* convert ExprComb to scoped enumNoel Grandin2017-02-163-83/+73
| | | | | | and drop unused EC_bit_neg enumerator Change-Id: I8a93310cd849cee56fcf628424d96005f89ba799
* convert ErrorCode to scoped enumNoel Grandin2017-02-164-162/+132
| | | | | | and drop unused enumerators Change-Id: I7330ebf8feead7b67e67f73ba942979e4756269d
* convert WarningCode to scoped enumNoel Grandin2017-02-163-16/+4
| | | | | | and drop unused enumerators Change-Id: I212cf132c2406e85b44b1af552b0989cecbec162
* remove unused PS_OpCompleted from ParseState enumNoel Grandin2017-02-162-3/+0
| | | | Change-Id: I3a738e464051d0b810553bc62c8465d37505bc3c
* Remove dynamic exception specificationsStephan Bergmann2017-01-262-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | ...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* Remove junkStephan Bergmann2017-01-251-5/+0
| | | | | | | left over from 232c0fb700efd31eff8cfee896437c481582ed0f "#115784# idlc: fixed commandline option processing." Change-Id: I41f81b1223cf72c0fcf8e84cdc7016e3aebf0c9d
* loplugin: unnecessary destructor forms..idlcNoel Grandin2017-01-2515-21/+0
| | | | | | | Change-Id: I765e0fa61f8134a60e5ea24452c6bbcb3fa8b054 Reviewed-on: https://gerrit.libreoffice.org/33492 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* New loplugin:dynexcspec: Add @throws documentation, idlcStephan Bergmann2017-01-191-0/+3
| | | | Change-Id: I3c269d84906cc49d79d5f32f0dd3f15afd2f588e
* no special support for MACOSX PPC in master for a while nowCaolán McNamara2017-01-171-2/+0
| | | | Change-Id: I8d5dc878f41939e55ef8995d1da74548faa4711f
* New loplugin:externvar: idlcStephan Bergmann2017-01-092-4/+1
| | | | Change-Id: I7b892efd6a5267c228c4cf17e80fac82628fbb64
* bootstrap building with --disable-dynloading on standard linuxCaolán McNamara2016-12-201-0/+1
| | | | | | | Change-Id: I5baf70f0053612cba8b74f54aff11ce25cdeb95a Reviewed-on: https://gerrit.libreoffice.org/32202 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Rewrite some (trivial) assignments inside if/while conditions: idlcStephan Bergmann2016-11-292-8/+7
| | | | Change-Id: I6be338f54dd8e6912e48258c0a2e075dc69b2893
* loplugin:expandablemethods in hwpfilter..linguisticNoel Grandin2016-11-082-3/+1
| | | | | | | Change-Id: I62ae20ab4a47b3b7e2b0d503cedcad3319cc9c85 Reviewed-on: https://gerrit.libreoffice.org/30683 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryvirtual in hwpfilter..rscNoel Grandin2016-11-084-11/+11
| | | | | | | Change-Id: I2e9df8223d503b813b4c458747e8c44bb8ef8868 Reviewed-on: https://gerrit.libreoffice.org/30661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:oncevar in helpcompiler..reportdesignNoel Grandin2016-11-021-4/+2
| | | | | | | Change-Id: I2dc57931fb230953c285aeb18f57c0a41fedafcb Reviewed-on: https://gerrit.libreoffice.org/30463 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:expandablemethodds in hwpfilter..linguisticNoel Grandin2016-10-194-6/+2
| | | | | | | Change-Id: If20fd4a6cf8a8e005804dbb7caf41cce73d587a5 Reviewed-on: https://gerrit.libreoffice.org/30016 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* clang-cl loplugin: idlcStephan Bergmann2016-10-161-1/+1
| | | | | | | Change-Id: Idcd117aa896779bb5ebdfa33c24d5e374dd943cf Reviewed-on: https://gerrit.libreoffice.org/29861 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#96099 Reduce no of typedefs used for trivial containersAsela Dasanayaka2016-10-136-23/+20
| | | | | | | | | Removed various pinter typedefs in idlc module Change-Id: Id850558a4550085d47a5e1122b1ce871f15956f2 Reviewed-on: https://gerrit.libreoffice.org/29754 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Simplify AstExpression::compareStephan Bergmann2016-10-063-31/+4
| | | | | | | | | ...to only work with LONG values, as it is only used to compare enumerator values. (But keep its general defensive-programming air that's so prevalent across the idlc code base; not sure whether it could rightfully be tightened, or whether this might be needed in parser's error recovery.) Change-Id: I15f1700834f9397f3c2e0ffdb00e2abeecb734f9
* we shouldn't need explicit lt_rtl_OUString, etc things anymoreCaolán McNamara2016-09-241-9/+1
| | | | Change-Id: Ibeec3fcc353e07e61fb2c838b318e0a04081ce2c
* loplugin:unusedenumvaluesNoel Grandin2016-09-191-4/+3
| | | | | | | Change-Id: I9dcc9f73af6db5b4f7cc946bc28931e5c230b34c Reviewed-on: https://gerrit.libreoffice.org/29012 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-1320-20/+20
| | | | | | | | | | | | | | | | | The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
* loplugin:unusedenumconstants in forms..sotNoel Grandin2016-08-121-1/+0
| | | | | | | Change-Id: Ic445e1bdd012c32ef7d84aec9df908467c3296a7 Reviewed-on: https://gerrit.libreoffice.org/28055 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unnecessaryvirtualNoel Grandin2016-08-081-1/+1
| | | | | | | Change-Id: If25d9307efda5f57b0f80a0cf5c2c5cab6a752d6 Reviewed-on: https://gerrit.libreoffice.org/27981 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin unnecessary overrideNoel Grandin2016-07-154-13/+0
| | | | | | | Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
generated by cgit (git 2.34.1) at 2024-08-05 23:31:15 +0000