summaryrefslogtreecommitdiffstats
path: root/idlc
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* Get rid of unnecessary directory levels $D/inc/$DTor Lillqvist2016-05-2752-232/+232
| | | | | | | | Change-Id: Ibf313b8948a493043006ebf3a8281487c1f67b48 Reviewed-on: https://gerrit.libreoffice.org/25532 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Tor Lillqvist <tml@collabora.com>
* Fix typo in codeAndrea Gelmini2016-05-201-1/+1
| | | | | | | Change-Id: I71ede9bf7d3f7de236dd161f30b36f52efec2c56 Reviewed-on: https://gerrit.libreoffice.org/25185 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* Add SAL_FALLTHROUGH, where apparently appropriateStephan Bergmann2016-05-101-0/+1
| | | | Change-Id: I07f247f003a5522e7d655f30f7c68d1b8f7225f2
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-101-1/+1
| | | | | | | | | | | | | | | | ...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
* loplugin:nullptrStephan Bergmann2016-04-261-103/+107
| | | | Change-Id: Id58f3320ff152286fa6b3268cdc9c2af2d7e068b
* loplugin:passstuffbyref in variousNoel Grandin2016-04-141-1/+1
| | | | Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
* use consistent #define checks for the Windows platformNoel Grandin2016-02-171-3/+3
| | | | | | | | | | | | stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Formatting changes across all modulesChris Sherlock2016-02-091-4/+4
| | | | | | | | | | | | | | + Removed comment cruft + Tab formatting in number of files + Some commented out code removed + Tab characters replaced with spaces + Newline cleanup in quite a few files + Tweak header guard #endifs Change-Id: I3208ff2f047da890edcc49b73389aca22442f5fc Reviewed-on: https://gerrit.libreoffice.org/22221 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* loplugin:unusedmethodsNoel Grandin2016-01-202-53/+0
| | | | | | | Change-Id: Ib2dc804f55d1a96ae01f2cf31b7d55956040a72e Reviewed-on: https://gerrit.libreoffice.org/21603 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedfieldsNoel Grandin2015-12-242-2/+0
| | | | | | | Change-Id: I979592adb978c3757a1e54615021ee424a2e02bf Reviewed-on: https://gerrit.libreoffice.org/20892 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* FD_complex is unusedStephan Bergmann2015-11-303-28/+11
| | | | | | | ...since 8e71d9400bd350d63723c5489ddaadb3d3610baf "Remove UNOIDL 'array' and 'union' vaporware from idlc" Change-Id: Id4cdef900667cdac466a41e47c189e2d25a62065
* loplugin:unusedfields idl,idlcNoel Grandin2015-11-201-1/+0
| | | | Change-Id: I39816fc613aadb5250eed39fd0c6830abe1cde2c
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-1021-151/+151
| | | | Change-Id: Ic42387288c96436915956ede2eb510b9c3fbc49b
* loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann2015-11-061-2/+2
| | | | Change-Id: I106ec424d4a2b32b91d9c4c70eb8a6670c20ede9
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-129-13/+13
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* clang-analyzer-deadcode.DeadStoresStephan Bergmann2015-10-071-2/+2
| | | | Change-Id: I9c409cb2d7898ac6f395618cd950385cf036b002
* loplugin:unusedmethodsNoel Grandin2015-10-011-1/+0
| | | | | | | | | | | | - improvements to the plugin to find more method calls - improvements to python script to remove more false+ - fix the FORCE_COMPILE_ALL build flag to include code in the $WORKDIR Change-Id: I4d6015dcb9b9d60c26f0bcee8abad807177a7836 Reviewed-on: https://gerrit.libreoffice.org/19064 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Return unique_ptr from AstExpression::eval_* functionsStephan Bergmann2015-09-282-12/+16
| | | | | | | | | | | ...generally, idlc leaks most memory, and e.g. trying to turn AstExpression::m_exprValue into a unique_ptr would fail because AstExpression::eval_symbol returns a pointer to an object "owned" elsewhere (but never actually deleted by its owner). So improvement of idlc ends here---for one, it does not improve anything to try and make idlc not leak memory, and for another, idlc is doomed to go away anyway. Change-Id: I36f54130c8bfd1933126ae7f8a982e50d9bc616e
* m_exprValue cannot be non-null hereStephan Bergmann2015-09-281-3/+0
| | | | | | | ...as, like the other private AstExpression::eval_* functions, eval_un_op is only called from AstExpression::evaluate if m_exprValue is null Change-Id: I812ea249d33a72f95b9692dc72ba6631ffbd4884
* AstExpression::coerce always returned m_exprValue, so simplify its signatureStephan Bergmann2015-09-282-19/+11
| | | | Change-Id: I4fa380bc7e8d5b3581468cb0e6417b99587a1e9f
* Remove redundant AstExpression::eval_internalStephan Bergmann2015-09-282-15/+8
| | | | Change-Id: I27c067adb2b09ad65a8449f2d88a9cdde149d4be
* AstExpression::eval_internal always returns m_exprValueStephan Bergmann2015-09-282-18/+17
| | | | Change-Id: I214fb3d14cef07357860a88fae1652a7d1a282b0
* AstExpression::coerce is always called with bAssign = trueStephan Bergmann2015-09-282-4/+3
| | | | Change-Id: I883b8319fde0bd3cc65efe64eb2faf53e2344cfc
* Simplify coerce_value signatureStephan Bergmann2015-09-281-161/+163
| | | | Change-Id: I0791600a5cc8600ea80233afba07724dd7f565e1
* The only EvalKind actually used is EK_constStephan Bergmann2015-09-263-66/+34
| | | | Change-Id: Ide451d1e287eeb8666b9ddcce312b9a753761c50
* tdf#39440 reduce scope of local variablesMichael Weghorn2015-08-221-4/+2
| | | | | | | | | This addresses some cppcheck warnings. Change-Id: I6ab170c4426494ab9fc53017f88e47fbfdbd9aad Reviewed-on: https://gerrit.libreoffice.org/17920 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* loplugin:unusedmethodsNoel Grandin2015-08-051-4/+0
| | | | | | | Change-Id: I6801618efb5a66d24156fa429e026acb6ca03aba Reviewed-on: https://gerrit.libreoffice.org/17506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedmethodsNoel Grandin2015-07-3010-67/+0
| | | | | | | Change-Id: Ib4d77ee01e7362f5951f81fceeca3c489872d971 Reviewed-on: https://gerrit.libreoffice.org/17378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2015-07-031-4/+4
| | | | | | | Change-Id: Ie2bbe020fc6e3a4a4f913208c245f395849bb9ee Reviewed-on: https://gerrit.libreoffice.org/16708 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-152-23/+10
| | | | | | | | Idea originally from caolan. Found using the following command: find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;' Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
* idlc: fix warningsStephan Bergmann2015-05-184-37/+24
| | | | Change-Id: I2ef51fae0bc4425c5d2b6707d1df381888b427e7
* More loplugin:simplifyboolStephan Bergmann2015-04-271-2/+2
| | | | Change-Id: Iddfa035ac4b41d203ed6864f83dab69dcec33386
* loplugin:simplifyboolStephan Bergmann2015-04-241-20/+20
| | | | Change-Id: I858ab62a2659ae122beff1300387aefc3aa64189
* Improved loplugin:literaltoboolconversion looking into cond. exprs.Stephan Bergmann2015-04-231-20/+20
| | | | | | ...automatic rewriter fixes Change-Id: I6b04ca80f08f8a71ff94e309fd52f44d736751ee
* fix android buildNoel Grandin2015-04-091-1/+1
| | | | | | | | since after my converting stuff in include/registry/types.h to use 'enum class', it can obviously no longer be "extern C", so drop the "extern C" and rename the file to reflect that. Change-Id: Ia30f5731316525e48531c4785ab7471a428bcf6f
* convert RTReferenceType to scoped enumNoel Grandin2015-04-093-6/+6
| | | | Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
* convert RTMethodMode to scoped enumNoel Grandin2015-04-092-4/+4
| | | | Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
* convert RT_ACCESS fields to scoped enumNoel Grandin2015-04-094-20/+20
| | | | Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
* convert RegError to scoped enumNoel Grandin2015-04-095-17/+17
| | | | Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
* convert RegValueType to scoped enumNoel Grandin2015-04-094-7/+7
| | | | Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
* Clean up remaining C-style casts among void pointersStephan Bergmann2015-03-294-6/+6
| | | | Change-Id: I1b49c020d597b569e330482f4dbf20c15ccdae3f
* Clean up C-style casts from pointers to voidStephan Bergmann2015-03-282-3/+3
| | | | Change-Id: Iacc2cfe28571d7fffc375ad9b7f3b0da735e093f
* loplugin:staticfunctionNoel Grandin2015-03-276-92/+88
| | | | Change-Id: Id796b799f8e2fcc3eae98d43800c5e31fec27fef
* const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann2015-03-261-2/+2
| | | | Change-Id: I20cd0d2f3b38fdec90022844de2ac6bdb9978ee7
* loplugin:constantfunction: variousNoel Grandin2015-03-241-1/+0
| | | | Change-Id: I6eddda9f4b31c7ce413c328b6a857a81bd222eed
* V668 no sense in testing the result of new against nullCaolán McNamara2015-03-111-2/+1
| | | | Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
* remove unnecessary parenthesis in return statementsNoel Grandin2015-02-232-10/+10
| | | | | | | found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
generated by cgit (git 2.34.1) at 2024-07-04 21:13:17 +0000