Age | Commit message (Collapse) | Author |
|
Avoid heap allocating UNO object wrappers for the underlying Node
structures only to convert to Any and immediately free them agian
when we can.
Change-Id: Iae4612e9602f872f5d8cca2e516df594c9f1118c
|
|
Saves 13.5m cycles on a headless start, 1.3%.
Change-Id: I73f0dd8a523f84c765a217ee95aea9bcc3db1b5d
|
|
Change-Id: Ib170b3cec17402e38bf5555e21f83f44d70bb574
|
|
Change-Id: I57c81b5ec7a6541a825e42f9b68e7c5fa98f316f
|
|
Accelerates headless startup by 6.5m cycles or so, 0.6% or
so but something.
Change-Id: I9edca3d0c1c81d865e8369fee5cf193da7adb97c
|
|
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
|
|
Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
|
|
Change-Id: I9464179a736b91f5bc32eb5e5e89b3b4e3da3494
|
|
Change-Id: I5316693452427ed76a7738b090de023b110caa40
|
|
Change-Id: Ica1dc0c8003203ba0ea65a668ea3d86f341ad05d
|
|
Now that we have default values for Exception constructor params,
remove lots of boilerplate code.
Change-Id: I620bd641eecfed38e6123873b3b94aaf47922e74
|
|
Change-Id: I74f1a032ebce8e4fa13d4afd7f520bbb367b2df5
|
|
This partially reverts 78b9537b6aca0f998f939988e6e1a56528d28baf "various:
sal_Bool->bool."
Change-Id: I732b2fb2cca3e66b396a35645f12ae96a40163d5
|
|
Change-Id: I30fa5227f30373f95060148f269450d2945c1bb9
|
|
...which is explicitly supported by Boost.Container though, but we cannot use
boost::container::map here as Boost.Container is only available since Boost 1.48
and our base is 1.47 still. However, there appears no need for Children to be
sorted, and std::unordered_map (and thus boost::unordered_map) requires that
pointers to elements are not invalidated by insertions, which we do require.
Change-Id: I70ad6f40318d2bafae53ebb67d84c1c89a7d68f6
|
|
libicudata exports a single symbol, icudt52_data (for ICU version 52), and
only libicuuc refers to it.
Change-Id: Ia04356fda20cf8109156ccd1cbbd9091e70adf27
|
|
Change-Id: I166ce50c038cdd3474cbdcf2ac360c237a98307c
|
|
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
Change-Id: Ie87cc749556d0a9b135b285143d33dc80d49a2ad
|
|
It's not very efficient, because we generally end up copying it twice -
once into the parameter and again into the destination OUString.
So I create a clang plugin that finds such places and generates a
warning so that we can convert them to pass-by-reference.
Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
|
|
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
|
|
Change-Id: Id05440ffb8eb9990d2d08f721d6914f3dbc86cab
|
|
Change-Id: I5e405ec2689000c973cf8ff1f86c1c2d6202eda9
|
|
Change-Id: I8115fb899e4248e6eab1aa9349456103353918e6
|
|
Change-Id: I960aa77ce107c4f02f1e331cda07b75435beed5d
|
|
Change-Id: I3abf410e7235b8dd1d126a3c8705e3ef9722e93b
|
|
Change-Id: I3f9c98d235d1bd5155653c6a56b757393bca2979
|
|
Change-Id: I88796c85447571d446c398439066029c3a200b4b
|
|
Change-Id: I763dc95a393bf296a7f3f5ce2a739f175fd528fb
|
|
...it avoids false warnings about unchecked dynamic_cast results from static
code analyzers, and potentially makes the code a little faster, too. (Most of
these dynamic_casts were kind of a leftover from a very early design that
dispateched on Node type via dynamic_cast instead of a Node::Kind, but which was
much too slow.)
Change-Id: I8db08a1c35783f8ea6c51eed55b11faa2d958807
|
|
This reverts commit 753eeac8a641d70ad0c44bb15fa8949ac991f5d9, which added bogus
if (... != 0)
checks after dynamic_casts that cannot fail, making the code now look as if it
was expected that those casts can fail and the code is prepared to handle it
(which it is not). Silencing the false coverity reports will be addressed in a
follow-up commit.
(That commit unfortunately mixed those changes with mass cosmetic changes.)
|
|
Change-Id: I24b38b63d26bc2d4e1d479754120dfd5b5635737
|
|
Change-Id: Id9c8c3926f64826fd2295e0f75f007ce435f5915
|
|
Change-Id: I208f5892f6fb54d53ed4d17ddb189f0a24051a17
|
|
...for internal loader="com.sun.star.loader.SharedLibrary" components, instead
of exported component_getImplementationEnvironmen (or implicit
CPPU_CURRENT_LANGUAGE_BINDING_NAME). Adapted a few .component files as proof-
of-concept, more to follow.
Change-Id: I82332e0a48e6fc1da245990bb72265fe6e58447e
|
|
because Expert Config is no longer a leaf in option page tree,
it became a standalone window.
Change-Id: Ib82b5eb958073d1afbd9ad2b8493e2670dee591b
|
|
Convert code like
if( !aStr.isEmpty() && aStr[0] == 'x' )
to
if( aStr.startsWith("x") )
Change-Id: Iabc3a44ed3be2d29eed876e0eeef212ccd271edf
|
|
Change-Id: If7019fc334c0f11fae464b65c135d01acfb1a46b
|
|
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>: I wonder why I originally
introduced State::locked in addition to State:ignored in the first place, but
can't find a good reason for that now. So the patch looks ok, though it could be
further simplified to completely get rid of State::locked. Will do that in a
follow-up commit.
Change-Id: If07a07b21effbf42918408a0b60b2d18bdc8665c
|
|
Change-Id: If0b413a4fdd93465074548c7ea5451288c1d12aa
|
|
A final pass through the code, converting code to use the new
OUString and OString methods that can detect string literals.
Change-Id: Ifa6382335e5650a1c67e52006b26354e0692c710
|
|
Change-Id: I0fa482b3303f339afabf5ed8f3e432b56c6590aa
|
|
This is largely unnecessary when working with OUString
Change-Id: I3cf4d68357a43665d01162ef4a2d5346a45da9be
|
|
Change-Id: Ibf04062ca86ed866202d748c3b62a210d30ed6ec
|
|
They are practically always useless, often misleading or obsolete.
Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
|
|
Change-Id: I4c688a4aeedcae56ed6404574bd1bb392d4190cb
Reviewed-on: https://gerrit.libreoffice.org/6311
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I0c7ae8e0393f069259b943e04368e874abdf8386
|
|
to use cppu::supportsService and other pieces.
Change-Id: I16893b3d31a8055acd214ff23d01e63d38fe0826
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...as there are many cases where the code later wants to obtain this part, and
esp. for the string literal variants it is awkward to calculate the length of
the literal again if this is coded with a following copy() call. Adapt some
code to use this new feature.
(Strictly speaking, the @since tags for the---backwards-compatibly---modified
functions are no longer accurate of course. Also, clean up some sal_Bool and
SAL_THROWS(()) that are unnecesssary cargo-cult here, and where the clean-up
should have no practical compatibility consequences.)
Change-Id: I43e5c578c8c4b44cb47fd08f170b5c69322ad641
|
|
Change-Id: Ib0cac79b86ed60b4df1fc90db15842cc99abc1e9
|