summaryrefslogtreecommitdiff
path: root/idlc/source
AgeCommit message (Collapse)Author
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl
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 Srivastava
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-06Revert "use the new OUString::fromUtf8 method"Stephan Bergmann
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).
2014-11-06use the new OUString::fromUtf8 methodNoel Grandin
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
2014-11-05markup with event type not checker typeCaolán McNamara
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
2014-11-05fdo#38835 strip out OUString globalsNoel Grandin
they are largely unnecessary these days, since our OUString infrastructure gained optimised handling for static char constants. Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
2014-10-30coverity#982268 silence Copy-paste errorCaolán McNamara
Change-Id: I10e5dd8ea4097892d2571304d982ab20ee230988
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin
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>
2014-10-28coverity#982167 Unchecked return valueCaolán McNamara
Change-Id: I14c306132a8f12c6f7ecebe5b012984d04bf0d91
2014-10-24coverity#441992 Uncaught exceptionCaolán McNamara
Change-Id: Ibd07b854bbcf9d9e83236669addfd61fe639882a
2014-10-03coverity#982268 Copy-paste errorCaolán McNamara
Change-Id: I68e8e6a564356bae620647b385e16279ff212d32
2014-10-02remove HAVE_GCC_PRAGMA_DIAGNOSTIC_MODIFY check and macroMichael Stahl
This has been supported by GCC and clang for a very long time. Change-Id: I410a2b39004c932003f8cbefe935aedb109b1163
2014-09-30idlc: std::auto_ptr -> std::unique_ptrStephan Bergmann
Change-Id: I0d64c9b2b7aeddfd70541a14c8a325fea9944c39
2014-09-07create clang plugin to warn about C-style castsNoel Grandin
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>
2014-06-17improve the inlinesimplememberfunctions clang pluginNoel Grandin
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
2014-06-10Resolves: #i122368# convert inc paths into short form on windows...Jürgen Schmidt
to avoid problem with ucpp (cherry picked from commit 6e370ddd6b532e7bf12df3aa52f93913f4ea8348) Conflicts: idlc/source/options.cxx Change-Id: Ia8c7dc3a9a7df68a483dd106cd47c8624952aeae
2014-06-09loplugin: inlinesimplememberfunctionsNoel Grandin
Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
2014-05-19fix-includes.pl: idlcThomas Arnhold
Change-Id: I62da508151614277d59e831c7506c3ec715348f4
2014-05-13Revert "CID#705985 ensure umask for mkstemp"Stephan Bergmann
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 Arnhold
Change-Id: I665e901eb6f65308b8f130da1c6da95bc3255490
2014-03-24callcatcher: update unused codeCaolán McNamara
Change-Id: I48990c044e4583e835f3e995527ba423e8c459fb
2014-03-12loplugin:literaltoboolconversionStephan Bergmann
Change-Id: I513aeeabbb4d6fc48b6a95e034c311a454dd5ca7
2014-03-08coverity#707935 Uninitialized scalar fieldCaolán McNamara
Change-Id: I7c957419b72cc97dbec819c51acb521bcd716624
2014-02-17idlc: sal_Bool -> boolStephan Bergmann
Change-Id: Ibeed903d73eb8b3fce7b8c6021c9107437a8c813
2014-02-06Fix spelling errrorTor Lillqvist
Change-Id: Ie5fc19fa223942e31899408324beb2e15c1d7bef
2014-01-30Remove UNOIDL "array" and "union" vaporware from idlcStephan Bergmann
Change-Id: Iccd5a78b54620796cfde672388b70ad97d17b7a5
2014-01-28bool improvementsStephan Bergmann
Change-Id: I0ec9d6ed41eddbc0177c26755b621f404cfd4703
2014-01-28coverity#440064 Logically dead codeCaolán McNamara
Change-Id: I52c9e37ca4d3238b9018264b6542a9dcebca13f6
2014-01-25coverity#440240 Dereference after null checkNorbert Thiebaud
Change-Id: I1ca5e4d31ab203a0ddd5e98928f5046f5fffb485
2013-12-13fdo#72598 Remove SunStudio cruft from code baseJelle van der Waa
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 Bergmann
Change-Id: I76be464200d486efef9c8a7e957c310c9adae3b8
2013-11-19remove most use of RTL_CONSTASCII_USTRINGPARAM macroNoel Grandin
This is largely unnecessary when working with OUString Change-Id: I3cf4d68357a43665d01162ef4a2d5346a45da9be
2013-11-11remove unnecessary use of OUString constructorNoel Grandin
Change-Id: Ifb220af71857ddacd64e8204fb6d3e4aad8eef71
2013-11-04Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin
This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
2013-10-31Convert indexOf->startsWith and lastIndexOf->endsWithNoel Grandin
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 Arnhold
So there is no need to do this locally, too. Change-Id: Ibc623235cae07a86dfd0dbc1d7672ca4273facec
2013-10-23convert code to use OUString::endsWithNoel Grandin
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 Lillqvist
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-09-12CID#705985 ensure umask for mkstempCaolán McNamara
Change-Id: I861b85ac38a2881ff4b993047c1a15335308ba62
2013-09-12CID#440614 use after freeCaolán McNamara
a bit of guesswork here I admit Change-Id: I774da6f585e4b81a25660b7823d6bb150630f44f
2013-09-04Minor clean upStephan Bergmann
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 Bergmann
Change-Id: Id28fb53a98ff164cd14f8f9dcb4ebee7a71bfd1e
2013-08-28parser.y: untabify, remove trailing whitespace, mode linesStephan Bergmann
Change-Id: Ief0109289c6dc70b51193fa5f4432efe2b48c3fb
2013-08-22downcast of address which does not point to an object of type 'AstInterface'Stephan Bergmann
(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 Bergmann
(note: object is of type 'AstDeclaration'); Clang -fsanitize=undefined Change-Id: I08eef7d3e8b298abd16b0c6d72cfb72b75190cd5
2013-08-21finish deprecation of O(U)String::valueOf()Luboš Luňák
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 Waa
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 Arnhold
Change-Id: I0057b9174af6a83f3fde0e27c3a91a4aeca95873 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-05-23RHEL-5 baseline ver of flex doesn't have nounistdCaolán McNamara
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 Stahl
Change-Id: Ic05de69951b28b9cc8d62f0a534b507c424e6b25