Age | Commit message (Collapse) | Author |
|
Change-Id: Ib21a3cfe4b02e421f8848a6e2cca4e2a1c04adb9
|
|
Signed-off-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>, undid one remove that was
detrimental to loplugin:unreffun
Change-Id: I18d8252084d828f94ef7a954e1dbfb45743d7970
|
|
Added clear() method to OString and OUString class, Updated appropriate call-sites.
Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
This reverts commit 05050cdb23de586870bf479a9df5ced06828d498,
not all places that use e.g. OStringToOUString to convert potential UTF-8
are guaranteed to fulfil the prerequisites necessary to use fromUtf8 (and
some places like e.g. in codemaker are happy with the best-effort effect
of OStringToOUString's OSTRING_TO_OUSTRING_CVTFLAGS).
|
|
Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
|
|
Change-Id: I5362d997bfa086c9fb1726efcb15132a966684f6
Reviewed-on: https://gerrit.libreoffice.org/12160
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
It turns out that almost none of them were necessary.
Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83
Reviewed-on: https://gerrit.libreoffice.org/12133
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I579bfc6f4a57aea49e2b023917d0dbd7c3db6c88
|
|
Change-Id: Icd11b81566fec7ce4855cd4e54ff3a3d9862d354
|
|
Change-Id: I58082770ca79da9c4d235b5ea0dc79c688c7498b
|
|
Change-Id: I205c0908254f13970315f8e159997a5ec2f96ed3
|
|
so that we get a nice complete stacktrace when it hits the final
handler
Change-Id: Iec4fcc15a2a25c55f591b5e069dce3d010197a90
|
|
i.e. stuff like "x == true"
Change-Id: Ib82a4a30e736df392405332fa197b588482cffcf
|
|
Change-Id: Ia8befb8d69914ce971174fc5f2ffc0e2f506a940
|
|
Change-Id: I3d8b598e86339fdd51af48e62393550821b6278d
|
|
Change-Id: I1bedfb601320bc27adc0d0d72fcb8b3c5dfe73ff
|
|
Change-Id: Id7c517fb37bc28797c45fc0dde83e866f2aa4aac
|
|
Find places where we are returning a pointer to something, where we can
be returning a reference.
e.g.
class A {
struct X x;
public X* getX() { return &x; }
}
which can be:
public X& getX() { return x; }
Change-Id: I796fd23fd36a18aedf6e36bc28f8fab4f518c6c7
|
|
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
|
|
Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
|
|
Change-Id: I9464179a736b91f5bc32eb5e5e89b3b4e3da3494
|
|
Change-Id: I682ca0b62e6eb0d02113218de97d1440089b782c
|
|
Change-Id: Ib3f0776048ca3dbb8a7efa4d9240f4ee9d6ac9dd
|
|
Change-Id: Ic340559c9ab9b74f4f932c90e08f2530c08e8d08
|
|
Change-Id: I9c7893f7c9af2b42dcc50f2d1463ec497bb85e30
|
|
after commit
c5d47c327a57df55
"add default value for Context param in uno::Exception constructors"
Change-Id: Ib429b58c559ecd48764a8df2f49d5470057c2c63
|
|
and all it's subtypes, which is almost never used, so this allows us to
simplify lots of call sites.
Change-Id: I0b05793ea2bdd1027679f63252d42ce4af89433b
|
|
Change-Id: I1240656cc2a4d713c838eb80fa90ce3485aad614
|
|
Detect arguments larger than 64 chars passed by value.
Change-Id: I9b0ea9ccb99d115984a26eab67c9cf6afd5f6cae
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I6f55030ea7bebce99537c7cdd5156c83c38a765a
|
|
Change-Id: I9cc9bfbddd3a90e00eee3e674994e5d6207f9034
|
|
I guess even in the debug builds these strings do not add much info, but...
Change-Id: I5cf765539a821513b5ce77c5484071d181459881
|
|
Change-Id: I0a0691df58455b712eb27695f4cce15fd3494b65
|
|
It appears that the C++ standard allows overriding destructors to be marked
"override," but at least some MSVC versions complain about it, so at least make
sure such destructors are explicitly marked "virtual."
Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
|
|
...mostly done with a rewriting Clang plugin, with just some manual tweaking
necessary to fix poor macro usage.
Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
|
|
Change-Id: If3e2dd3905cc33f1e7fc9fbfbb9f2bb49a756a34
|
|
Change-Id: I098c636ed8794c858317fdbefa9ffa1c6d81c320
Reviewed-on: https://gerrit.libreoffice.org/8241
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I2cacac2aa7e48b3b9d8d060137d5c6d6f1d06b3f
|
|
Change-Id: I546c3f497dc98c88a0d678622c20cced387a4ab2
|
|
...and deprecate what cannot be removed for compatibility.
Change-Id: I1ea335af775b867b468b8285113631167729a92a
|
|
Change-Id: I528a6a747e88fe2a4b4abfd9372daddf4e8aa502
|
|
Change-Id: I6db797c950b8222b3ad95faff86adf5e7b41ff4b
|
|
Change-Id: I77428c6cf1f79dcc0e2c5e1d52f9b4b8c88130e3
|
|
Change-Id: Id071c8658b473b9d393a32e1da2c2474bb7361d3
|
|
Change-Id: Icaa2062b00e0a68486bcbbc4a26b4089016bf2c8
|
|
Change-Id: I9f5d48c5c5c0f76aa3386cfedcaf43ced162e4b0
|
|
There is really no need to see tons of
component context fails to supply service <something long> of type <something else long>:
and
component context fails to supply service <something long> of type <something else long>
in the binary ;-) - let's split the strings to parts that the linker can
share easily.
Change-Id: I07f149a587c05985045028c3b6530c6efd081ff5
|
|
Missing in c2c530da69152ff9192b9726aa95961803ce9b29 "Introduce static
inline cppu::acquire(), and make use of that."
Change-Id: I4a856eeaff1fce3f429c62c1897fe3898371d1ec
|
|
Most of the constructors are supposed to be only a call of
new TheInstance(arguments)
or an equivalent; so let's just change the constructor caller accordingly, to
accept unacquired new instance.
If there are exceptions that need to do more heavy lifting, they do not have
to use the constructor feature, or there can be a wrapper for the real
implementation, doing the additional work in their (C++) constructor.
Change-Id: I035c378778aeda60d15af4e56ca3761c586d5ded
|
|
Change-Id: I329ff80da68c629d7a82921253c4412563743b34
|