Commit Graph

5884 Commits

Author SHA1 Message Date
8ab4d2f29f drop crashrep unused since start of LibreOffice
Change-Id: I3df1216054c133314b2317849744a0a37e9fbc8f
Reviewed-on: https://gerrit.libreoffice.org/13733
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-01-07 11:41:32 +00:00
7f8f277b94 fdo#84938: convert STREAM_ #defines to 'enum class'
Change-Id: Ibbf14c7e9a5c1883c1311d4c86f948f74f8e473e
2015-01-07 11:20:44 +02:00
9fd89a5952 fdo#39440 reduce scope of local variables
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>
2015-01-07 06:47:04 +00:00
8e94c40f13 how about scope
Change-Id: I4c9e5f754974b6726fc86bc87775ba81832b3984
2015-01-06 11:59:53 +01:00
549b7fad48 workaround a weird gcc optimization werror bug
gcc (GCC) 4.9.2 20141101 (Red Hat 4.9.2-1)

framework/source/fwe/classes/addonsoptions.cxx: In member function ‘void framework::AddonsOptions_Impl::ReadAndAssociateImages(const rtl::OUString&, const rtl::OUString&)’:
framework/source/fwe/classes/addonsoptions.cxx:267:16: error: array subscript is above array bounds [-Werror=array-bounds]
         struct ImageEntry
                ^

The combination of aScaled[2]; aImage[2]; aURL[2] in sequence apparently lead
to some overoptimization and/or alignment problem, already declaring aImage[3]
helped (but not aScaled[3]), but that's not what we want.

Change-Id: I82e28d4887ab8072a17d0a9341d322c1cf61aedc
2015-01-06 11:40:23 +01:00
bae855e667 fdo#84938: replace BUTTONTYPE_ constants with 'enum class'
Change-Id: I54f9019297913683605b5aea9f79b3defc1dcc13
2015-01-06 10:59:40 +02:00
566f16a2fd java: unnecessary imports
found by CodePro

Change-Id: If1b75e43f81d70984422e437147048a491395b66
2015-01-05 10:15:13 +02:00
10039462df java: remove more dead code
found by looking for unused parameters (in Eclipse)

Change-Id: I03cf9bc8312e59747b2d0ac153ee2fc8d76be893
2015-01-05 09:59:03 +02:00
46540d08b1 java: these fields can become local variables
found by PMD

Change-Id: Id852cfec984a181b91aca1c00a41e342b720a4bf
2015-01-05 08:23:30 +02:00
43cc8ad33e java: remove dead code
found by UCDetector

Change-Id: I6b0f49529379072da566e927b86815f173e7a90b
2015-01-05 08:23:29 +02:00
fd1372ba8b boost::unordered_map->std::unordered_map
you can get debug stl this way

Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2015-01-02 16:14:36 +00:00
8a009858aa Adjust the statusbar item widths according to HiDPI scaling factor
Change-Id: Id697b3b799c7437d937de6534edf88083e5c4dfd
2014-12-26 20:42:12 +09:00
2d82619a52 java: static fields that should not be static
Found by FindBugs.

Change-Id: I223841f7bb8c515c9612322abc0b13e134385abd
2014-12-19 10:41:58 +02:00
cba3956750 java: this does not need to be a Thread
It is just being used as a Runnable

Change-Id: I1e652ab0f3b0450d3a85384b9dd513419e2f8046
2014-12-19 10:41:54 +02:00
b98352996c framework: Use appropriate OUString functions on string constants
Change-Id: I5da5cd3254102c742021b3e7b4e1f907ca897ca5
2014-12-18 14:12:22 +01:00
38b8c1e880 fdo#39440 reduce scope of local variables
This addresses some cppcheck warnings.

Change-Id: I57f336c6ab3c418eac82397edeadecb4c1b168a7
Reviewed-on: https://gerrit.libreoffice.org/13495
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-12-16 20:42:30 +00:00
b733f5c4f4 framework: Use appropriate OUString functions on string constants
Change-Id: I72bb5ff2ab8a55061072fde567fcc1c4a8e20a2f
2014-12-12 17:52:18 +01:00
7557f23b31 java: reduce visibility of fields and methods
found by PMD

Change-Id: Id6737916b68ccbdbdeec5d314747a38410923ac6
Reviewed-on: https://gerrit.libreoffice.org/13409
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-11 06:09:03 +00:00
ffffc526e3 java: remove comment noise
Change-Id: If6347197ff4436b4edd06e6e62d8e8e6fe1d4db6
Reviewed-on: https://gerrit.libreoffice.org/13408
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-10 08:06:38 +00:00
9adb820158 java: simplify the getMSF() methods
Change-Id: Ib459799f4a3224f8c9683ac4b6cf37982d2077a3
Reviewed-on: https://gerrit.libreoffice.org/13406
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-10 08:04:58 +00:00
1367e4b2af changed timers to idle
Change-Id: I05ccb8bd73070462edcc911956859aa7967b5901
2014-12-09 12:35:09 +00:00
439c8eba11 changed some timer to idle
Change-Id: If137dbd80f7260d9507e48709d3f0f14b60a25f6
2014-12-09 12:35:07 +00:00
e4200feb14 changed timers to idle
Change-Id: I5846940cc45781f4a0264047107dbb568604d543
2014-12-09 12:35:06 +00:00
f0a6ec25d0 changed timers to idle
Change-Id: I05ccb8bd73070462edcc911956859aa7967b5901
2014-12-09 12:35:03 +00:00
9f3e3923d7 changed timers to idle
Change-Id: I5846940cc45781f4a0264047107dbb568604d543
2014-12-09 12:35:03 +00:00
d1684e16b5 changed some timer to idle
Change-Id: If137dbd80f7260d9507e48709d3f0f14b60a25f6
2014-12-09 12:35:03 +00:00
9cc2a28323 framework: avoid double-free in ToolBarManager()
The timer could run after the ToolBarManager is deleted because the
dispose() method can accidentally restart the timer somehow.

Change-Id: I86868f08f436976761e814d4cea1cd5a0e348935
2014-12-09 12:35:01 +00:00
e5f67b2e94 changed timers to idle
Change-Id: I05ccb8bd73070462edcc911956859aa7967b5901
2014-12-09 12:34:57 +00:00
85efe9c2f0 fdo#39468 Translate German Comments - sc/source/core/data/documen2.cxx
Change-Id: I9a426f40f8cdce1f1ebe207dd9ada8154d8fde46
Reviewed-on: https://gerrit.libreoffice.org/12195
Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2014-12-09 12:34:56 +00:00
18102201f4 changed some timer to idle
Change-Id: If137dbd80f7260d9507e48709d3f0f14b60a25f6
2014-12-09 12:34:51 +00:00
eae0dda3e2 Resolves: fdo#86820 executing toolbar menu entry can destroy menu
Change-Id: Ide0ea8f8ff145d487e409b0091918b59f1f83405
2014-12-08 12:11:18 +00:00
16e1fbb9aa Avoid automatic toolbars in the non-desktop case
Change-Id: I9e5bff735b0035c147e10ae066da3a4873d66749
2014-12-04 23:04:24 +01:00
75f4ad0f6f Ignore exceptions when getting the background color configuration
We are experimenting with a pruned configuration database in the desktop case,
and letting the exception propagate here killed the document loading.

Change-Id: I59e5d016617c17c2bc36de2fd69c6691bfa6b135
2014-12-04 23:04:24 +01:00
179810cdf2 Further clean-up
Change-Id: I20049b482c831e4ac2221fddfe80deb9847e72c3
2014-12-03 20:30:18 +01:00
9de4fe9853 Code clean-up/consolidation task.
Removing the duplicate code,and consolidating changes
by using  method "getExpandedFilePath" of comphelper.

Signed-off-by: Stephan Bergmann <sbergman@redhat.com>: adapted to recent change
from comphelper::getExpandedFilePath to comphelper::getExpandedUri; fixed the
two changes in framework to actually modify the by--non-const--ref argument;
fixed a loplugin:unreffun.  More clean-up to follow.

Change-Id: Ie8875bcb61b616385bd64151f0a915bf7cce04e5
2014-12-03 20:30:18 +01:00
14596452e7 fdo#84938: replace EVENT_ constants with enum
Change-Id: I8275832d8dae43b374bddd48520d11592e9a6a1f
Reviewed-on: https://gerrit.libreoffice.org/13134
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-01 07:43:31 +00:00
58eab57648 java: final fields that can be static
found by PMD

Change-Id: I2b48f35d252d6e914c2a18a2c175b075a9d3ac0f
Reviewed-on: https://gerrit.libreoffice.org/13099
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-25 06:06:48 +00:00
1658c017a2 java,framework: remove unused vars and parameters
Change-Id: I7df5ff730436b95cb93af9f813d7bc572da2b951
Reviewed-on: https://gerrit.libreoffice.org/13098
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-25 05:05:24 +00:00
fe095dada4 java,qadev: remove unused parameters and local vars
Change-Id: Ifb9f9374051fe88dc4cd5a7a28b2c8c992ced873
Reviewed-on: https://gerrit.libreoffice.org/13097
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-11-24 20:45:17 +00:00
4730af6a24 typo: unknown
Change-Id: If89443c1668ca01c826c8fc353b20097c7da7bf4
2014-11-24 12:48:25 +01:00
9dc7bae64d Revert "coverity#1169848 Uncaught exception"
This reverts commit b0a7722ebfa720255b6ce1e383a3aacae0ce5e9f, now that
fced31697bcc2a2a4da34d804582002de0320302 "comphelper::SequenceAsHashMap: throw
IllegalArgumentException" made the underlying problem go away.

Conflicts:
	framework/source/services/modulemanager.cxx

Change-Id: I6599cf25d4f28d2bab19da67d5de5caacd84a622
2014-11-20 08:34:50 +01:00
30bf5787e1 Revert "cid#706553 Uncaught exception"
This reverts commit 0a87f4ad1a45aaaaad885a1418d0d6bed21db456, now that
fced31697bcc2a2a4da34d804582002de0320302 "comphelper::SequenceAsHashMap: throw
IllegalArgumentException" made the underlying problem go away.

Conflicts:
	framework/source/inc/loadenv/loadenv.hxx
	framework/source/loadenv/loadenv.cxx

Change-Id: Id2f468bf0f24f5a8a7a0e6eb113ccfedd35b03cc
2014-11-19 18:30:46 +01:00
6f58f34fc0 Revert "cid#706552 Uncaught exception"
This reverts commit ecd00991fd5884cd769ed00c9c3f42ba44d64511, now that
fced31697bcc2a2a4da34d804582002de0320302 "comphelper::SequenceAsHashMap: throw
IllegalArgumentException" made the underlying problem go away.

Conflicts:
	framework/source/inc/loadenv/loadenv.hxx
	framework/source/loadenv/loadenv.cxx

Change-Id: I7821ae67b5cc99b8591fbf06396ac20098e55c8d
2014-11-19 18:28:46 +01:00
e753f8b9dc Revert "cid#706551 Uncaught exception"
This reverts commit ec651eaead5aaa2a74f4c87d3bf17f7feddb958e, now that
fced31697bcc2a2a4da34d804582002de0320302 "comphelper::SequenceAsHashMap: throw
IllegalArgumentException" made the underlying problem go away.

Conflicts:
	framework/source/inc/loadenv/loadenv.hxx
	framework/source/loadenv/loadenv.cxx

Change-Id: I473815256b95f6e20a2e1a4a5617f491636305ab
2014-11-19 18:25:42 +01:00
fced31697b comphelper::SequenceAsHashMap: throw IllegalArgumentException
...instead of IllegalTypeException (where the latter is not a RuntimeException).
Fixes cid#707218, cid#1224993, etc., and reverts previous fix
358f5280e0f6afb968edc295d8f1dce55f05e18f "Fix coverity#1224993 w/o breaking
JunitTest_framework_unoapi."

Change-Id: I13ce153e1513749fb0be2ee8e560cec86a30c5ba
2014-11-19 18:18:06 +01:00
15b4ab8d19 cppuhelper: clean up public headers with include-what-you-use
Change-Id: I41ba46831f24b2960a1fe982b74a2b623e682e0b
2014-11-18 18:51:49 +01:00
721366ebe1 cppu: clean up public headers with include-what-you-use
Unfortunately iwyu gets quite confused by the weird cyclic dependencies
between various foo.h/foo.hxx and cppumaker generated headers, so it's
not obvious if any improvement here is realistic...

Change-Id: I0bc66f98b146712e28cabc18d56c11c08418c721
2014-11-18 18:51:48 +01:00
0063cf2856 java: make fields final where possible
found by PMD

Change-Id: I87780366119c141cd2dafe6ca1bf2d9798b10aec
2014-11-18 12:44:28 +02:00
62155d2611 fdo#82708 editing statusbar tooltip texts
Change-Id: I3ac2e46f89d5c1e4decb6d6aaf8487663e904a43
Reviewed-on: https://gerrit.libreoffice.org/12433
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-11-18 09:28:14 +00:00
639f6dcb1f coverity#705177 Missing break in switch
this is the "close toolbar" button on right clicking a toolbar.
close toolbar continues to work as expected

Change-Id: I436fbe9f133442136b6e60a1f6080926f64eaef6
2014-11-17 14:53:22 +00:00