summaryrefslogtreecommitdiff
path: root/idlc/source
AgeCommit message (Collapse)AuthorFilesLines
2014-05-19fix-includes.pl: idlcThomas Arnhold11-18/+18
Change-Id: I62da508151614277d59e831c7506c3ec715348f4
2014-05-13Revert "CID#705985 ensure umask for mkstemp"Stephan Bergmann1-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."
2014-04-19fixincludeguards.sh: idl and idlcThomas Arnhold1-2/+2
Change-Id: I665e901eb6f65308b8f130da1c6da95bc3255490
2014-03-24callcatcher: update unused codeCaolán McNamara1-33/+0
Change-Id: I48990c044e4583e835f3e995527ba423e8c459fb
2014-03-12loplugin:literaltoboolconversionStephan Bergmann1-1/+1
Change-Id: I513aeeabbb4d6fc48b6a95e034c311a454dd5ca7
2014-03-08coverity#707935 Uninitialized scalar fieldCaolán McNamara1-0/+1
Change-Id: I7c957419b72cc97dbec819c51acb521bcd716624
2014-02-17idlc: sal_Bool -> boolStephan Bergmann14-72/+72
Change-Id: Ibeed903d73eb8b3fce7b8c6021c9107437a8c813
2014-02-06Fix spelling errrorTor Lillqvist1-1/+1
Change-Id: Ie5fc19fa223942e31899408324beb2e15c1d7bef
2014-01-30Remove UNOIDL "array" and "union" vaporware from idlcStephan Bergmann9-1005/+7
Change-Id: Iccd5a78b54620796cfde672388b70ad97d17b7a5
2014-01-28bool improvementsStephan Bergmann1-2/+2
Change-Id: I0ec9d6ed41eddbc0177c26755b621f404cfd4703
2014-01-28coverity#440064 Logically dead codeCaolán McNamara1-6/+2
Change-Id: I52c9e37ca4d3238b9018264b6542a9dcebca13f6
2014-01-25coverity#440240 Dereference after null checkNorbert Thiebaud1-1/+1
Change-Id: I1ca5e4d31ab203a0ddd5e98928f5046f5fffb485
2013-12-13fdo#72598 Remove SunStudio cruft from code baseJelle van der Waa2-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>
2013-11-27Unwind occurrences of deprecated sal_sChar, sal_uCharStephan Bergmann1-8/+8
Change-Id: I76be464200d486efef9c8a7e957c310c9adae3b8
2013-11-19remove most use of RTL_CONSTASCII_USTRINGPARAM macroNoel Grandin1-1/+1
This is largely unnecessary when working with OUString Change-Id: I3cf4d68357a43665d01162ef4a2d5346a45da9be
2013-11-11remove unnecessary use of OUString constructorNoel Grandin1-1/+1
Change-Id: Ifb220af71857ddacd64e8204fb6d3e4aad8eef71
2013-11-04Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin1-2/+2
This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
2013-10-31Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin2-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
2013-10-28windows: warning 4706 is disabled globally in com_MSC_defs.mkThomas Arnhold1-1/+1
So there is no need to do this locally, too. Change-Id: Ibc623235cae07a86dfd0dbc1d7672ca4273facec
2013-10-23convert code to use OUString::endsWithNoel Grandin1-2/+2
Convert places that call aStr[aStr.getLength()-1] == 'x' to use the shorter form aStr.endsWith("x") Change-Id: I1b3a19c0e89b8989cdbeed440f95fc76f9a4b6b6
2013-10-22Bin comments that claim to say why some header is includedTor Lillqvist1-1/+1
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-09-12CID#705985 ensure umask for mkstempCaolán McNamara1-0/+4
Change-Id: I861b85ac38a2881ff4b993047c1a15335308ba62
2013-09-12CID#440614 use after freeCaolán McNamara1-2/+2
a bit of guesswork here I admit Change-Id: I774da6f585e4b81a25660b7823d6bb150630f44f
2013-09-04Minor clean upStephan Bergmann1-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
2013-08-28Do not leave $$ uninitialized in case of errorStephan Bergmann1-0/+1
Change-Id: Id28fb53a98ff164cd14f8f9dcb4ebee7a71bfd1e
2013-08-28parser.y: untabify, remove trailing whitespace, mode linesStephan Bergmann1-2120/+2123
Change-Id: Ief0109289c6dc70b51193fa5f4432efe2b48c3fb
2013-08-22downcast of address which does not point to an object of type 'AstInterface'Stephan Bergmann2-4/+4
(note: object is of type 'AstTypeDef'); Clang -fsanitize=undefined Change-Id: I1b11a7678f18557c8040af2118c1c0101913f086
2013-08-22downcast of address which does not point to an object of type 'AstType'Stephan Bergmann1-1/+1
(note: object is of type 'AstDeclaration'); Clang -fsanitize=undefined Change-Id: I08eef7d3e8b298abd16b0c6d72cfb72b75190cd5
2013-08-21finish deprecation of O(U)String::valueOf()Luboš Luňák2-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
2013-06-18fdo#43460 connectivity,extensions,filter,idl,idlc: use isEmpty()Jelle van der Waa2-2/+2
Change-Id: I78ba286be5aa4d731f734e5eb16adb77c686aad9 Reviewed-on: https://gerrit.libreoffice.org/4325 Reviewed-by: Noel Power <noel.power@suse.com> Tested-by: Noel Power <noel.power@suse.com>
2013-05-29fdo#60724 change spelling error REMOVEABLE -> REMOVABLE IIThomas Arnhold4-6/+6
Change-Id: I0057b9174af6a83f3fde0e27c3a91a4aeca95873 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-05-23RHEL-5 baseline ver of flex doesn't have nounistdCaolán McNamara1-1/+1
lets try this hack, if it fails revert and I'll just manually update the baseline with a newer flex than flex-2.5.4a-41.fc6 Change-Id: If644489e80fc4e4cdba97808af0fe4d544d08646
2013-05-10soltools: remove Package_inc and empty unistd.h nonsenseMichael Stahl1-0/+1
Change-Id: Ic05de69951b28b9cc8d62f0a534b507c424e6b25
2013-04-07fix logical-op-parenthesis warningPeter Foley1-1/+1
Change-Id: Ib97f667c9acb644c3becb81f851ad8c7e4ff31ea
2013-04-07mass removal of rtl:: prefixes for O(U)String*Luboš Luňák17-57/+55
Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk have kept them, in order not to break external API (the automatic using declaration is LO-internal). Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09
2013-04-03Forbid old-style services/singletons inheriting new-style servicesStephan Bergmann1-1/+2
...does not make sense. Adapted some old-style services accordingly, where the inherited service had been changed to new-style after the fact. Change-Id: I5f3e4ddf99160778a319062a6c84f83529ff177b
2013-03-27-Wunused-macrosStephan Bergmann2-10/+0
Change-Id: Ifaa1637122d6f9cae1e29b77ac36ca5d1f220aed
2013-03-25remove external include guardsThomas Arnhold2-48/+1
Change-Id: If5758e02baa8e019923355b6c97043b306223fdd
2013-03-25Introduce HAVE_GCC_PRAGMA_DIAGNOSTIC_{MODIFY,SCOPE}Stephan Bergmann1-3/+3
...replacing hard-coded GCC version checks. Those checks that guard #pragma GCC diagnostic ignored "-Wnon-virtual-dtor" appear relevant only for GCC itself, not Clang (which used to fail the old guards because it typically announces itself with a rather low __GNUC__/__GNUC_MINOR__ version), see 6e67c03dc0225fc66343546b14e902b9d238b1a3 "Enable -Wnon-virtual-dtor for GCC 4.6" Change-Id: I6bfa4d5caa6192e7a203ce829682bf6bb8d61a1b
2013-03-25deleting NULL is safeTakeshi Abe1-4/+2
Change-Id: I3748d0034b2c8e9d9122ad2a9001763eb1265e1d
2013-03-23coverity#704237 Logically dead codeJulien Nabet1-2/+0
Change-Id: I7a5087b2ac89ee2839f0e0d8450efc7fca1dd68b
2013-03-22coverity#440066 Logically dead codeJulien Nabet1-6/+0
Change-Id: I54d1b22859bd4f0a1e68ce79ed2ce4d7fbeb5107
2013-03-22coverity#440065 Logically dead codeJulien Nabet1-3/+0
Change-Id: I6b735ca48bd996c71ad6c6103e68af2cd2890d98
2013-03-19further OUString cleanupThomas Arnhold1-2/+2
Change-Id: I19954ee5976cb881a5fe73a889c8e04a19b14eb6
2013-03-19removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStrings declarationsThomas Arnhold1-2/+2
s/(OUString\s+[a-zA-Z_][A-Za-z0-9_]*\s*)\(\s*RTL_CONSTASCII_USTRINGPARAM\s*\((\s*"[^")]*?"\s*)\)\s*\)/$1\($2\)/gms Change-Id: Iad20f242c80c4bdc69df17e2d7a69d58ea53654b Reviewed-on: https://gerrit.libreoffice.org/2835 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2013-03-19automated removal of RTL_CONSTASCII_USTRINGPARAM for quoted OUStringsThomas Arnhold3-6/+6
Done with a perl regex: s/OUString\s*\(\s*RTL_CONSTASCII_USTRINGPARAM\s*\((\s*"[^")]*?"\s*)\)\s*\)/OUString\($1\)/gms Change-Id: Idf28320817cdcbea6d0f7ec06a9bf51bd2c3b3ec Reviewed-on: https://gerrit.libreoffice.org/2832 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2013-03-07fix spellingNoel Grandin1-1/+1
Change-Id: I52eb4153c3396be88b72cab3b7899f0c6ab18103
2013-03-06fdo#60724 change spelling error REMOVEABLE -> REMOVABLEKenneth Beck1-2/+2
Only applies to PropertyAttribute::REMOVEABLE, and all instances in comments. All other instances of the misspelling have remained the same. Example: AF_REMOVEABLE Change-Id: I391f4101bbc3e06689318235a37d616065bc1686 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-02-21Fix type unkown -> unknownJulien Nabet2-2/+2
Change-Id: I8d55d5ae5ae38ae2b252859b5400e36301f19ef0
2013-02-13Some cppcheck cleaningJulien Nabet1-2/+1
Change-Id: I099171d85a51095f0ff6d592c1260d931f9769b2
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-09 17:28:01 +0000