summaryrefslogtreecommitdiff
path: root/idlc
AgeCommit message (Collapse)AuthorFilesLines
2016-04-14loplugin:passstuffbyref in variousNoel Grandin1-1/+1
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
2016-02-17use consistent #define checks for the Windows platformNoel Grandin1-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>
2016-02-09Formatting changes across all modulesChris Sherlock1-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>
2016-01-20loplugin:unusedmethodsNoel Grandin2-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>
2015-12-24loplugin:unusedfieldsNoel Grandin2-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>
2015-11-30FD_complex is unusedStephan Bergmann3-28/+11
...since 8e71d9400bd350d63723c5489ddaadb3d3610baf "Remove UNOIDL 'array' and 'union' vaporware from idlc" Change-Id: Id4cdef900667cdac466a41e47c189e2d25a62065
2015-11-20loplugin:unusedfields idl,idlcNoel Grandin1-1/+0
Change-Id: I39816fc613aadb5250eed39fd0c6830abe1cde2c
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann21-151/+151
Change-Id: Ic42387288c96436915956ede2eb510b9c3fbc49b
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann1-2/+2
Change-Id: I106ec424d4a2b32b91d9c4c70eb8a6670c20ede9
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann9-13/+13
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-07clang-analyzer-deadcode.DeadStoresStephan Bergmann1-2/+2
Change-Id: I9c409cb2d7898ac6f395618cd950385cf036b002
2015-10-01loplugin:unusedmethodsNoel Grandin1-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>
2015-09-28Return unique_ptr from AstExpression::eval_* functionsStephan Bergmann2-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
2015-09-28m_exprValue cannot be non-null hereStephan Bergmann1-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
2015-09-28AstExpression::coerce always returned m_exprValue, so simplify its signatureStephan Bergmann2-19/+11
Change-Id: I4fa380bc7e8d5b3581468cb0e6417b99587a1e9f
2015-09-28Remove redundant AstExpression::eval_internalStephan Bergmann2-15/+8
Change-Id: I27c067adb2b09ad65a8449f2d88a9cdde149d4be
2015-09-28AstExpression::eval_internal always returns m_exprValueStephan Bergmann2-18/+17
Change-Id: I214fb3d14cef07357860a88fae1652a7d1a282b0
2015-09-28AstExpression::coerce is always called with bAssign = trueStephan Bergmann2-4/+3
Change-Id: I883b8319fde0bd3cc65efe64eb2faf53e2344cfc
2015-09-28Simplify coerce_value signatureStephan Bergmann1-161/+163
Change-Id: I0791600a5cc8600ea80233afba07724dd7f565e1
2015-09-26The only EvalKind actually used is EK_constStephan Bergmann3-66/+34
Change-Id: Ide451d1e287eeb8666b9ddcce312b9a753761c50
2015-08-22tdf#39440 reduce scope of local variablesMichael Weghorn1-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>
2015-08-05loplugin:unusedmethodsNoel Grandin1-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>
2015-07-30loplugin:unusedmethodsNoel Grandin10-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>
2015-07-03Fix typosAndrea Gelmini1-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>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin2-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
2015-05-18idlc: fix warningsStephan Bergmann4-37/+24
Change-Id: I2ef51fae0bc4425c5d2b6707d1df381888b427e7
2015-04-27More loplugin:simplifyboolStephan Bergmann1-2/+2
Change-Id: Iddfa035ac4b41d203ed6864f83dab69dcec33386
2015-04-24loplugin:simplifyboolStephan Bergmann1-20/+20
Change-Id: I858ab62a2659ae122beff1300387aefc3aa64189
2015-04-23Improved loplugin:literaltoboolconversion looking into cond. exprs.Stephan Bergmann1-20/+20
...automatic rewriter fixes Change-Id: I6b04ca80f08f8a71ff94e309fd52f44d736751ee
2015-04-09fix android buildNoel Grandin1-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
2015-04-09convert RTReferenceType to scoped enumNoel Grandin3-6/+6
Change-Id: I8320f6f42d5579fbd09450ddca61c4c066de98e4
2015-04-09convert RTMethodMode to scoped enumNoel Grandin2-4/+4
Change-Id: I81599570698eb92abf14fa6386d8545c2031e863
2015-04-09convert RT_ACCESS fields to scoped enumNoel Grandin4-20/+20
Change-Id: Ifef0aaae6208f18c1352fef7c471deb60a97820d
2015-04-09convert RegError to scoped enumNoel Grandin5-17/+17
Change-Id: I55977c38578cb59deb195d91f8948da2e0905b6a
2015-04-09convert RegValueType to scoped enumNoel Grandin4-7/+7
Change-Id: Ic672e75db4d7323760577b19490ffa28d38965b6
2015-03-29Clean up remaining C-style casts among void pointersStephan Bergmann4-6/+6
Change-Id: I1b49c020d597b569e330482f4dbf20c15ccdae3f
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann2-3/+3
Change-Id: Iacc2cfe28571d7fffc375ad9b7f3b0da735e093f
2015-03-27loplugin:staticfunctionNoel Grandin6-92/+88
Change-Id: Id796b799f8e2fcc3eae98d43800c5e31fec27fef
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann1-2/+2
Change-Id: I20cd0d2f3b38fdec90022844de2ac6bdb9978ee7
2015-03-24loplugin:constantfunction: variousNoel Grandin1-1/+0
Change-Id: I6eddda9f4b31c7ce413c328b6a857a81bd222eed
2015-03-11V668 no sense in testing the result of new against nullCaolán McNamara1-2/+1
Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
2015-02-23remove unnecessary parenthesis in return statementsNoel Grandin2-10/+10
found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
2015-01-28remove unused typedefsNoel Grandin2-2/+0
found with some minor modifications to find/find-unused-defines.sh Change-Id: I18cc479adedc7a0dada68a4aeef08300e62631dd Reviewed-on: https://gerrit.libreoffice.org/14194 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara3-6/+8
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-18fdo#39440 reduce scope of local variablesMichael Weghorn5-13/+7
This addresses some cppcheck warnings. Change-Id: Ib16e4ecc0e0c43b2c1fb527eb0668a960ae9756e Reviewed-on: https://gerrit.libreoffice.org/13513 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-16idlc: Use appropriate OUString functions on string constantsStephan Bergmann1-1/+1
Change-Id: Ia537f5d5d573d24f5bbed7a20b9e9268181faa5f
2014-12-15add link to mkstemp coverity warning wrt threading fiascoCaolán McNamara1-0/+1
Change-Id: Ic8fea32a163ca5e85ac3e2a34d04e4fa1a1943f9
2014-11-18More iwyu suggested headers removalRiccardo Magliocchetti1-1/+0
Signed-off-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com> Signed-off-by: Stephan Bergmann <sbergman@redhat.com>, undid one remove that was detrimental to loplugin:unreffun Change-Id: I18d8252084d828f94ef7a954e1dbfb45743d7970
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl6-0/+9
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava2-5/+5
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>