summaryrefslogtreecommitdiffstats
path: root/soltools
Commit message (Collapse)AuthorAgeFilesLines
* -Werror=implicit-fallthrough= (GCC 7)Stephan Bergmann2017-01-262-1/+3
| | | | Change-Id: I2e9ab956fc1e59d09b7409333f18230e34b6a9b7
* loplugin:externvarStephan Bergmann2017-01-0910-23/+31
| | | | Change-Id: I304144c0ae6b51656ad326a77e9402cc34d5aa18
* tdf#99589 - tolower / toupper - dangerous to Turks ...Gökhan Gurbetoğlu2016-06-231-2/+3
| | | | | | | Change-Id: I8b0fe9354232a7b60f3605fa6f90f6741f7bf683 Reviewed-on: https://gerrit.libreoffice.org/26537 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* Drop redundant parenthesis.Pedro Giffuni2016-04-101-4/+4
| | | | | | | | | | Found with coccinelle: http://coccinelle.lip6.fr/ Currently only applied to C code. (cherry picked from commit aaf15142bcbc44f0e06a3357d2b21ecf9c673d21) Change-Id: I7d43b024752ebd47e84e2e51028f8fd132410800
* Avoid reserved identifierStephan Bergmann2016-03-101-4/+4
| | | | Change-Id: I3df7e42b69e3900eae8d10d28a97d776fabd9f4b
* Avoid reserved identifiersStephan Bergmann2016-03-101-14/+14
| | | | Change-Id: Ifa0efa673cd60eb7bd668b72b4550da6025ffd1e
* -Werror,-WundefStephan Bergmann2016-03-041-1/+1
| | | | | | | | | | | | For MSVC/clang-cl: "__STDC__: Indicates conformance with the ANSI/ISO C99 tandard. Defined as the integer literal constant 1 only if the /Za compiler option is given and you are not compiling C++ code; otherwise is undefined." (<https://msdn.microsoft.com/en-us/library/b0084kay.aspx>) Presumably fine to just replace #if with #ifdef here (as is used in all the other cases surrounding it), and not care whether it actually expands to true. Change-Id: Ie58c76d7e72321b06a11a370fd28221e85fcf871
* -Werror,-Wunused-macrosStephan Bergmann2016-03-042-6/+0
| | | | | | | | | Whatever those _POSIX_ for _MSV_VER were supposed to be good for (have been in there at least ever since 4f4b74153d27a9579c49b225b2690e7f9416fe78 "Moved cpp from cpp to soltools"), but cause -Wunused-macros at least when building with clang-cl. Change-Id: I356bf663cd1e86298ba72a179f46bb9d0c10016b
* new loplugin: commaoperatorNoel Grandin2016-02-231-3/+6
| | | | Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff
* use consistent #define checks for the Windows platformNoel Grandin2016-02-174-6/+6
| | | | | | | | | | | | 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>
* use consistent #define checks for the Windows platformNoel Grandin2016-02-164-8/+8
| | | | | | | | | | stage 1 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro Change-Id: Iece73abdee530937e0737190b1aa97a46cd3075f Reviewed-on: https://gerrit.libreoffice.org/22390 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* -Werror,-Wunused-parameter (clang-cl)Stephan Bergmann2016-01-261-0/+2
| | | | Change-Id: I1ac58714e04b995ef5229888f0c427104c001318
* Fix typosAndrea Gelmini2016-01-102-2/+2
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* loplugin:unusedfields in soltools/Noel Grandin2015-11-231-1/+0
| | | | Change-Id: I895a487b4c7be27fe81a2c4c122adb2aba37c638
* loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann2015-11-101-2/+2
| | | | Change-Id: Ib52f7fb846886dd970ddd51475f890cd81c8f2b5
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-104-8/+8
| | | | Change-Id: I3ae78d781e686bfe9b0513183da7fc8edd81024b
* loplugin:unusedmethodsNoel Grandin2015-10-021-1/+0
| | | | | | | Change-Id: Ie1603adf3908fd0668bcbe8f75c6bafa0d0bfd6c Reviewed-on: https://gerrit.libreoffice.org/19072 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2015-09-302-9/+7
| | | | | | | Change-Id: I89ff6d31662824d83961d6457b82db8d8031168c Reviewed-on: https://gerrit.libreoffice.org/18977 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* Fix typosAndrea Gelmini2015-07-043-3/+3
| | | | | | | Change-Id: Ic6415423f46aaee7ba90239a617c318cf92ae222 Reviewed-on: https://gerrit.libreoffice.org/16711 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
* -fsanitize=nonnull-attributeStephan Bergmann2015-06-161-1/+2
| | | | Change-Id: Id214fb1f0decd4d39684a846c06ff97d8baa209d
* loplugin:staticmethodsNoel Grandin2015-04-021-3/+3
| | | | Change-Id: I1f6c6d13697aa397067478d6b07429120106e6bd
* V804: Decreased performanceCaolán McNamara2015-03-021-1/+2
| | | | Change-Id: I6430442cc407e02810d849cf60833abe7c7d2830
* remove unnecessary parenthesis in return statementsNoel Grandin2015-02-234-32/+32
| | | | | | | found with $ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;' Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
* brute-force find-and-remove of unused #define constants.Noel Grandin2015-01-081-1/+0
| | | | Change-Id: I7223530ae37297a76654cd00cc1fedb56dbe3adb
* warning C4267: conversion from 'size_t' to 'int' (MSVC 64-bit)Stephan Bergmann2014-12-091-1/+1
| | | | Change-Id: Ife7d223f5e751e246c537561d3ce24395b4c9f3b
* warning C4267: conversion from 'size_t' to 'int' (MSVC 64-bit)Stephan Bergmann2014-12-093-8/+8
| | | | Change-Id: I2fa40ebe467a65b0d6023883a14c79ff4303bb2f
* coverity#1000834 Use of untrusted string valueCaolán McNamara2014-11-061-0/+1
| | | | | | | | | and coverity#706196 Use of untrusted string value coverity#1244947 Use of untrusted string value Change-Id: I5083aba2eff9e852a24513db431585922302b279
* coverity#982185 silence Unchecked return value from libraryCaolán McNamara2014-10-281-2/+2
| | | | | | | | and coverity#982184 Unchecked return value from library Change-Id: I4b4ba39abc4c008b68cfc1498ee56ac9c4934c98
* coverity#706158 Copy into fixed size bufferNorbert Thiebaud2014-10-041-5/+24
| | | | Change-Id: I5d540e6e3a21b0563febb70696882439a10b9b86
* coverity#1233618 Overlapping buffer in memory copyCaolán McNamara2014-10-021-1/+1
| | | | Change-Id: I062dd96bfc4abf2bdaec2c17d1cf57d853d655d8
* coverity#1019334 Explicit null dereferencedCaolán McNamara2014-09-041-74/+5
| | | | Change-Id: I22b85cbfda1c1bd705b35095e03cfae4071d2fb7
* mkdepend: cleanup indentation and function declarationsNoel Grandin2014-06-248-238/+186
| | | | | | | | | to make it easier to read Change-Id: Iff0fe055c12358edc1be335ec83d0855cc32f03c Reviewed-on: https://gerrit.libreoffice.org/9877 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* remove use of register keyword in C filesNoel Grandin2014-06-248-86/+86
| | | | | | | | | C compilers have been ignoring it for the last decade Change-Id: I42918263121dd189bab9d27077798b779b9e8da1 Reviewed-on: https://gerrit.libreoffice.org/9873 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* -Wstrict-prototypesStephan Bergmann2014-06-178-76/+25
| | | | | | | | ...no reason to not have it enabled for URE C include files and what little real C code is still left. (But note that Clang ignores that warning.) Change-Id: Ia6940f9f940a0c226e9b724331d65c9862ce32e6
* coverity#705235 Missing break in switchCaolán McNamara2014-05-301-0/+1
| | | | Change-Id: I19e8e6aa08dc879f273f2a0177b40435c0ac553d
* coverity#705236 Missing break in switchCaolán McNamara2014-05-301-0/+1
| | | | Change-Id: I8065789618f806d01cdb69a18502dfafd962b9f3
* coverity#705237 Missing break in switchCaolán McNamara2014-05-301-0/+1
| | | | Change-Id: I1f8c9aa6c7b4403346ee43501a3555abbccf1cfb
* Remove unnecessary semicolonsPeter Senna Tschudin2014-05-271-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | A simplified version of the semantic match that finds this problem is follows: (http://coccinelle.lip6.fr/) // <smpl> @r1@ statement S; position p,p1; @@ S@p1;@p @script:python r2@ p << r1.p; p1 << r1.p1; @@ if p[0].line != p1[0].line_end: cocci.include_match(False) @@ position r1.p; @@ -;@p // </smpl> Change-Id: Ib9708d37fbb4c6060f88d5dae3814a2d37b2091e Reviewed-on: https://gerrit.libreoffice.org/9493 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* coverity#982647 Missing break in switchCaolán McNamara2014-05-241-0/+1
| | | | Change-Id: Ifa318214079eeb2f903b3e0c4a27c1a807787fd4
* Fix some memory leak in soltools/cpp toolStephan Bergmann2014-05-222-0/+2
| | | | | | | ...though many more remain, as the design appears to be let heap grow until exit. Change-Id: I3e5cc95a0896c8add33a767209ab0ff60d6a4b52
* Resolves fdo#70681: fixincludeguards.pl: all that's leftThomas Arnhold2014-05-152-4/+4
| | | | | | | Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5 Reviewed-on: https://gerrit.libreoffice.org/9360 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
* Clean up function declarationStephan Bergmann2014-04-103-3/+32
| | | | Change-Id: I92c54f61fe8608d788cc236956f4a5a58e20a7df
* Move prototype to includeStephan Bergmann2014-04-042-2/+2
| | | | Change-Id: I90c093c704609429f9997f3ffed6a9a79c8956a6
* Typo #elsif -> #elifStephan Bergmann2014-03-311-6/+6
| | | | Change-Id: I95d4dc2a3364172e8917d0a029564e9114943148
* WaE: 'PATH_MAX' : macro redefinitionTor Lillqvist2013-11-172-2/+6
| | | | Change-Id: I153e34d24493bbbebf37422fb6382969153506a8
* Fix for VS 2013 compilationJesús Corrius2013-11-162-2/+2
| | | | | | | I don´t know why it doesn´t work anymore, but the fix is trivial may be related of using the Windows 8.0 SDK. Change-Id: Ia09109b53d4f022a732ea781c98ddd4eb0c721d9
* WaE: size_t/unsigned int: possible loss of dataTor Lillqvist2013-08-301-3/+3
| | | | | | With MSVC, the third parameter to read() is unsigned int. Change-Id: I607089fb2a9e6bf794293187be48e910ac40158f
* left shift of 1 by 31 places cannot be represented in type 'int'Stephan Bergmann2013-08-221-2/+2
| | | | | | Clang -fsanitize=undefined Change-Id: I2b3f1bd28e098d5eb57892d2c7438c18c5c3b357
* We don't seem to actually need glibc getoptTor Lillqvist2013-08-212-9/+3
| | | | | | | Change-Id: I1f221533f5ea684d113bdc45c04bf84405140f9f Reviewed-on: https://gerrit.libreoffice.org/5539 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Mark as constTakeshi Abe2013-07-141-1/+1
| | | | Change-Id: I0bf93f1293fb08b8b558fcac0cb496ed99ad13d9