summaryrefslogtreecommitdiffstats
path: root/soltools
Commit message (Collapse)AuthorAgeFilesLines
* 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
* soltools: remove Package_inc and empty unistd.h nonsenseMichael Stahl2013-05-106-31/+2
| | | | Change-Id: Ic05de69951b28b9cc8d62f0a534b507c424e6b25
* error: too many arguments in call to 'pr_dummy' [-Werror]Stephan Bergmann2013-04-241-1/+1
| | | | Change-Id: Ib03b6d3af9909c971245314bbde2085d8c3c73b2
* gbuild: get rid of processdeps.awkMichael Stahl2013-04-243-4/+43
| | | | | | | | | | | | | | | | | | The only thing that processdeps.awk does that is actually useful is to be able to set the target name freely, but that is only important for the default resource file which was just renamed. Instead hack makedepend to provide the required functionality directly: - write dummy rules for included files so builds don't break on rename/removal - format deps one per line like concat-deps expects - concat-deps expects standard POSIX line ends, so open in binary mode (also disable that awful backup copying nonsense) Though in retrospect it would perhaps be a better investment of time to try to replace makedepend... Change-Id: I54fafdcdcf1a52692e62b8f1f4b96fb9a93d4421
generated by cgit (git 2.34.1) at 2024-06-03 09:42:18 +0000