summaryrefslogtreecommitdiffstats
path: root/idlc/source
Commit message (Collapse)AuthorAgeFilesLines
* boost::unordered_map->std::unordered_mapCaolán McNamara2015-01-021-0/+2
| | | | | | you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
* fdo#39440 reduce scope of local variablesMichael Weghorn2014-12-185-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>
* idlc: Use appropriate OUString functions on string constantsStephan Bergmann2014-12-161-1/+1
| | | | Change-Id: Ia537f5d5d573d24f5bbed7a20b9e9268181faa5f
* add link to mkstemp coverity warning wrt threading fiascoCaolán McNamara2014-12-151-0/+1
| | | | Change-Id: Ic8fea32a163ca5e85ac3e2a34d04e4fa1a1943f9
* sal: clean up public headers with include-what-you-useMichael Stahl2014-11-176-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
* fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava2014-11-142-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>
* Revert "use the new OUString::fromUtf8 method"Stephan Bergmann2014-11-0611-50/+73
| | | | | | | | This reverts commit 05050cdb23de586870bf479a9df5ced06828d498, not all places that use e.g. OStringToOUString to convert potential UTF-8 are guaranteed to fulfil the prerequisites necessary to use fromUtf8 (and some places like e.g. in codemaker are happy with the best-effort effect of OStringToOUString's OSTRING_TO_OUSTRING_CVTFLAGS).
* use the new OUString::fromUtf8 methodNoel Grandin2014-11-0611-73/+50
| | | | Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
* markup with event type not checker typeCaolán McNamara2014-11-051-1/+1
| | | | Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
* fdo#38835 strip out OUString globalsNoel Grandin2014-11-051-4/+4
| | | | | | | they are largely unnecessary these days, since our OUString infrastructure gained optimised handling for static char constants. Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
* coverity#982268 silence Copy-paste errorCaolán McNamara2014-10-301-0/+1
| | | | Change-Id: I10e5dd8ea4097892d2571304d982ab20ee230988
* remove unnecessary 'using namespace rtl' declarationsNoel Grandin2014-10-2915-28/+0
| | | | | | | | | It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* coverity#982167 Unchecked return valueCaolán McNamara2014-10-281-1/+5
| | | | Change-Id: I14c306132a8f12c6f7ecebe5b012984d04bf0d91
* coverity#441992 Uncaught exceptionCaolán McNamara2014-10-241-105/+106
| | | | Change-Id: Ibd07b854bbcf9d9e83236669addfd61fe639882a
* coverity#982268 Copy-paste errorCaolán McNamara2014-10-031-3/+3
| | | | Change-Id: I68e8e6a564356bae620647b385e16279ff212d32
* remove HAVE_GCC_PRAGMA_DIAGNOSTIC_MODIFY check and macroMichael Stahl2014-10-021-1/+1
| | | | | | This has been supported by GCC and clang for a very long time. Change-Id: I410a2b39004c932003f8cbefe935aedb109b1163
* idlc: std::auto_ptr -> std::unique_ptrStephan Bergmann2014-09-301-7/+3
| | | | Change-Id: I0d64c9b2b7aeddfd70541a14c8a325fea9944c39
* create clang plugin to warn about C-style castsNoel Grandin2014-09-079-43/+42
| | | | | | | | | | We don't like C-style casts in our nice C++ code Change-Id: I94e7ec90de9275cd6e20c4146d4f3a74bed93c9d Reviewed-on: https://gerrit.libreoffice.org/10367 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>
* improve the inlinesimplememberfunctions clang pluginNoel Grandin2014-06-171-4/+0
| | | | Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
* Resolves: #i122368# convert inc paths into short form on windows...Jürgen Schmidt2014-06-101-1/+51
| | | | | | | | | | | to avoid problem with ucpp (cherry picked from commit 6e370ddd6b532e7bf12df3aa52f93913f4ea8348) Conflicts: idlc/source/options.cxx Change-Id: Ia8c7dc3a9a7df68a483dd106cd47c8624952aeae
* loplugin: inlinesimplememberfunctionsNoel Grandin2014-06-091-4/+0
| | | | Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
* fix-includes.pl: idlcThomas Arnhold2014-05-1911-18/+18
| | | | Change-Id: I62da508151614277d59e831c7506c3ec715348f4
* Revert "CID#705985 ensure umask for mkstemp"Stephan Bergmann2014-05-131-4/+0
| | | | | | | | | This reverts commit 5f5a981dd6df406b7bbddd88fa0b701b5a8d2246. Coverity warning about mkstemp without umask appears to be bogus (cf. <https://communities.coverity.com/message/6516> "Why are uses of mkstemp 'without securely setting umask first' being flagged?) and calling umask is not MT-safe, see fdo#60338 "FILESAVE: Saved files have incorrect permissions on linux."
* fixincludeguards.sh: idl and idlcThomas Arnhold2014-04-191-2/+2
| | | | Change-Id: I665e901eb6f65308b8f130da1c6da95bc3255490
* callcatcher: update unused codeCaolán McNamara2014-03-241-33/+0
| | | | Change-Id: I48990c044e4583e835f3e995527ba423e8c459fb
* loplugin:literaltoboolconversionStephan Bergmann2014-03-121-1/+1
| | | | Change-Id: I513aeeabbb4d6fc48b6a95e034c311a454dd5ca7
* coverity#707935 Uninitialized scalar fieldCaolán McNamara2014-03-081-0/+1
| | | | Change-Id: I7c957419b72cc97dbec819c51acb521bcd716624
* idlc: sal_Bool -> boolStephan Bergmann2014-02-1714-72/+72
| | | | Change-Id: Ibeed903d73eb8b3fce7b8c6021c9107437a8c813
* Fix spelling errrorTor Lillqvist2014-02-061-1/+1
| | | | Change-Id: Ie5fc19fa223942e31899408324beb2e15c1d7bef
* Remove UNOIDL "array" and "union" vaporware from idlcStephan Bergmann2014-01-309-1005/+7
| | | | Change-Id: Iccd5a78b54620796cfde672388b70ad97d17b7a5
* bool improvementsStephan Bergmann2014-01-281-2/+2
| | | | Change-Id: I0ec9d6ed41eddbc0177c26755b621f404cfd4703
* coverity#440064 Logically dead codeCaolán McNamara2014-01-281-6/+2
| | | | Change-Id: I52c9e37ca4d3238b9018264b6542a9dcebca13f6
* coverity#440240 Dereference after null checkNorbert Thiebaud2014-01-251-1/+1
| | | | Change-Id: I1ca5e4d31ab203a0ddd5e98928f5046f5fffb485
* fdo#72598 Remove SunStudio cruft from code baseJelle van der Waa2013-12-132-5/+1
| | | | | | | Change-Id: Ia6799c852eb95d496fbc8dcfdabde62dffc263a6 Reviewed-on: https://gerrit.libreoffice.org/7066 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Unwind occurrences of deprecated sal_sChar, sal_uCharStephan Bergmann2013-11-271-8/+8
| | | | Change-Id: I76be464200d486efef9c8a7e957c310c9adae3b8
* remove most use of RTL_CONSTASCII_USTRINGPARAM macroNoel Grandin2013-11-191-1/+1
| | | | | | This is largely unnecessary when working with OUString Change-Id: I3cf4d68357a43665d01162ef4a2d5346a45da9be
* remove unnecessary use of OUString constructorNoel Grandin2013-11-111-1/+1
| | | | Change-Id: Ifb220af71857ddacd64e8204fb6d3e4aad8eef71
* Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin2013-11-041-2/+2
| | | | | | This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
* Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin2013-10-312-7/+3
| | | | | | | | | | | | | | | | | | | | | | This is both an optimisation and a cleanup. This converts code like aStr.indexOf("XX") == 0 to aStr.startsWith("XX") and converts code like aStr.lastIndexOf("XXX") == aStr.getLength() - 3 to aStr.endsWith("XXX") Note that in general aStr.lastIndexOf("X") == aStr.getLength() - 1 converts to aStr.isEmpty() || aStr.endsWith("X") so I used the surrounding context to determine if aStr could be empty when modifying the code. Change-Id: I22cb8ca7c2a4d0288b001f72adb27fd63af87669
* windows: warning 4706 is disabled globally in com_MSC_defs.mkThomas Arnhold2013-10-281-1/+1
| | | | | | So there is no need to do this locally, too. Change-Id: Ibc623235cae07a86dfd0dbc1d7672ca4273facec
* convert code to use OUString::endsWithNoel Grandin2013-10-231-2/+2
| | | | | | | | | Convert places that call aStr[aStr.getLength()-1] == 'x' to use the shorter form aStr.endsWith("x") Change-Id: I1b3a19c0e89b8989cdbeed440f95fc76f9a4b6b6
* Bin comments that claim to say why some header is includedTor Lillqvist2013-10-221-1/+1
| | | | | | They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
* CID#705985 ensure umask for mkstempCaolán McNamara2013-09-121-0/+4
| | | | Change-Id: I861b85ac38a2881ff4b993047c1a15335308ba62
* CID#440614 use after freeCaolán McNamara2013-09-121-2/+2
| | | | | | a bit of guesswork here I admit Change-Id: I774da6f585e4b81a25660b7823d6bb150630f44f
* Minor clean upStephan Bergmann2013-09-041-2/+1
| | | | | | | Remove an odd {1} repeat count, and remove the "."{DIGIT}+ pattern that is already covered by the following {DIGIT}*"."{DIGIT}+ pattern. Change-Id: If99dfe10a5e37225355472bf4deceb34c0a92eb9
* Do not leave $$ uninitialized in case of errorStephan Bergmann2013-08-281-0/+1
| | | | Change-Id: Id28fb53a98ff164cd14f8f9dcb4ebee7a71bfd1e
* parser.y: untabify, remove trailing whitespace, mode linesStephan Bergmann2013-08-281-2120/+2123
| | | | Change-Id: Ief0109289c6dc70b51193fa5f4432efe2b48c3fb
* downcast of address which does not point to an object of type 'AstInterface'Stephan Bergmann2013-08-222-4/+4
| | | | | | (note: object is of type 'AstTypeDef'); Clang -fsanitize=undefined Change-Id: I1b11a7678f18557c8040af2118c1c0101913f086
* downcast of address which does not point to an object of type 'AstType'Stephan Bergmann2013-08-221-1/+1
| | | | | | (note: object is of type 'AstDeclaration'); Clang -fsanitize=undefined Change-Id: I08eef7d3e8b298abd16b0c6d72cfb72b75190cd5
* finish deprecation of O(U)String::valueOf()Luboš Luňák2013-08-212-11/+11
| | | | | | | Compiler plugin to replace with matching number(), boolean() or OUString ctor, ran it, few manual tweaks, mark as really deprecated. Change-Id: I4a79bdbcf4c460d21e73b635d2bd3725c22876b2