Age | Commit message (Collapse) | Author |
|
Idea from bubli - look for loops where the index variable is of such
size that it cannot cover the range revealed by examining the length
part of the condition.
So far, I have only run the plugin up till the VCL module.
Also the plugin deliberately excludes anything more complicated than a
straightforward incrementing for loop.
Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
|
|
Change-Id: I7031c126179bbceebc40dcbdab5a0570db5bc29f
Reviewed-on: https://gerrit.libreoffice.org/15813
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ic439b33c974500f1b351b41a0e391ffec71007b3
Reviewed-on: https://gerrit.libreoffice.org/15808
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I19a6835eaab2efba530d30f871f1b4509b683801
Reviewed-on: https://gerrit.libreoffice.org/15812
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ie1b05375a8e8d634e6ef6b2bf6edde65b32574bd
Reviewed-on: https://gerrit.libreoffice.org/15811
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
The adjective was used as the German meaning, which translates to
maybe or possibly.
Change-Id: Iaf5b38f628b23796829a1c4bd3919d4e13b874f0
|
|
Change-Id: Iab575622819e245b1459cba72dc34a9c387d00b6
|
|
Change-Id: Ie048543ad19c12dc47f6638af7dc029a38da964e
Reviewed-on: https://gerrit.libreoffice.org/15805
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: I6046461b2e26a8506249c4a2fdd783bbd2ec223f
Reviewed-on: https://gerrit.libreoffice.org/15810
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: Iaf1387ea236bf3a5db9b2165d5b827dbdcf95207
|
|
Change-Id: Ife6ab3544f73b1de60f5241c68466045161a4d88
Reviewed-on: https://gerrit.libreoffice.org/15615
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
If used at all, lcl_ is supposed to be used for static functions truly local
to one source files. The functions here occur in several source files (well,
two, the one where they are defined and one other) so they are not "local" in
that sense. (But they could be local in the one file that uses them...) Also,
they are already in a basegfx::internal namespace.
While at it, drop the :: prefix eyesore from basegfx::internal, and align
parameter lists consistently.
Change-Id: I68b91075e0b1779ac0fa884d8f9e956f1ab7b308
|
|
Change-Id: Ic0e8d409e33dcc494a6111eba9ba405424ecd53f
|
|
Change-Id: I1dd087ea6803f29756e1b78385a4a98f7378e616
|
|
Change-Id: I982ba552579019e4902ae59fddf14a6b34ba5954
|
|
Change-Id: I98dc5488954a4d46f7e0158790a4c3dabebfc1b3
|
|
Change-Id: I8a7528366156b288dc422b09cff0d5a32cde3c91
|
|
Fixed some cppcheck defects
Change-Id: I25fd6aba9d76df98d20b9a1bb4c9d3c1bf6f84bb
Reviewed-on: https://gerrit.libreoffice.org/14487
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I955c8ac4dbe002d23531df7eb10fb4444d6b5157
Reviewed-on: https://gerrit.libreoffice.org/14292
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I8cc33007e99da38869daa48ff898593abc52c4ac
|
|
It just calls OUStringBuffer.append() and doesn't do anything special.
Change-Id: Ie59a797a8d0d24cfab826516e2b5c4e58f90dd01
Reviewed-on: https://gerrit.libreoffice.org/13850
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: If69a57aa9ff011cc670f868cb87b3a1c4d904435
|
|
Beside scope changes, it fixes lack of initialization in a few places.
Change-Id: Ia09fdb9845d8ac17256330a5ec5168401c84f0f2
Reviewed-on: https://gerrit.libreoffice.org/13755
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
This eliminates some of the warnings from Cppcheck report of
2014-11-29.
Change-Id: Id5bf885de12a6de262d2cca9fa71627ac67f50d9
Reviewed-on: https://gerrit.libreoffice.org/13330
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
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
|
|
Change-Id: If4fcb8a2c6ac40a4d694522ce4ed020bcb4466b8
|
|
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
|
|
Change-Id: Ie0250745c9bd39e6e81f5f9eb96e661de2e4e2ef
|
|
Change-Id: Iec4c69e7fea727cc507506b9570f1f6d0c20a70f
|
|
tools to correct svg:d imports
(cherry picked from commit f077f99da3cb2903fa903dcf30e6cfd714fd009c)
Conflicts:
basegfx/source/inc/stringconversiontools.hxx
basegfx/source/tools/stringconversiontools.cxx
Change-Id: I7de1558682990f83e66fdded3d9d30e21e4f97fe
|
|
Change-Id: Ifa0c1b1983d1cc7f22d55b15dee88b7f709dbcf8
|
|
and automatically seed from time on first use
coverity#1242393 Don't call rand
coverity#1242404 Don't call rand
coverity#1242410 Don't call rand and additionally allow 0xFF as a value
coverity#1242409 Don't call rand
coverity#1242399 Don't call rand
coverity#1242372 Don't call rand
coverity#1242377 Don't call rand
coverity#1242378 Don't call rand
coverity#1242379 Don't call rand
coverity#1242382 Don't call rand
coverity#1242383 Don't call rand
coverity#1242402 Don't call rand
coverity#1242397 Don't call rand
coverity#1242390 Don't call rand
coverity#1242389 Don't call rand
coverity#1242388 Don't call rand
coverity#1242386 Don't call rand
coverity#1242384 Don't call rand
coverity#1242394 Don't call rand
Change-Id: I241feab9cb370e091fd6ccaba2af941eb95bc7cf
|
|
Change-Id: I66000e2e3ae8d28f5f04bb6404c73feb2a1e8905
|
|
Put the TOOLS PolyPolygon class in the tools namespace. Avoids clash with the Windows
PolyPolygon typedef.
Change-Id: I811ecbb3d55ba4ce66e4555a8586f60fcd57fb66
|
|
Change-Id: I8b1f0f6c100b4cf6d45c9e0c0f1e0a38ec081218
|
|
Change-Id: I56eb340317d8b7991caad58dd0e4045d6e69fca9
|
|
Change-Id: I57b039c7114c03944e8eaba9ca1bb81da563640a
|
|
(cherry picked from commit a02eb39b84d130e5923f72edb2abb3b21adf6fff)
Change-Id: Id068928f77a6efed44da9b83ecbf547302826591
|
|
(cherry picked from commit 7c5e9b9b3c5c899d63bf171ee1c9050db860337e)
Change-Id: I570e92c78196895bef329eb308eeb68ffc5e23d3
|
|
and
coverity#983562 Arguments in wrong order
Change-Id: Ia9e95c8c5dbbf0a12ac06162e4fc3ff5e568f035
|
|
attempt to silence this warning
Change-Id: Icd8fa41a63bd3fe70b956f7af1818325b471e0f6
|
|
Change-Id: I4f910494114f7dd965a75f3f9482153e0268b89c
|
|
Change-Id: I6c90b95bdf2de441594d5d5c36f76e6aeeb74011
|
|
Change-Id: Ia97fb3895e855e9fb970b0874fc100e2b900a8b7
|
|
Change-Id: I5558177fe61adbbff191042f1be22ecb0ce8f11a
|
|
Change-Id: I758b6bc8da33a20a0807da171120a8f84214dcfd
|
|
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
|
|
Added header guards to files in directories avmedia/, basegfx/, chart2/, and
connectivity/
Change-Id: I94ec438ec918ccef1c518edfd9ab4944e9e2961b
Reviewed-on: https://gerrit.libreoffice.org/9555
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I166e781cb54181a8e5d08021cf5f6dfcd1fde5cb
|
|
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5
Reviewed-on: https://gerrit.libreoffice.org/9360
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|