Age | Commit message (Collapse) | Author | Files | Lines |
|
Thank you Lionel about your advice for this one.
Change-Id: I4ec51b694d3b674158cb7fb4efa580e084cffda9
Reviewed-on: https://gerrit.libreoffice.org/15163
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
(cherry picked from commit 21a8671e24fe19a3ba5ef14c991bae302234b229)
Reviewed-on: https://gerrit.libreoffice.org/15191
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
I'll quote Lionel (I hate pretending I did the job when it's not the case :-)):
"
It was misguided; the order of entries in these arrays is not order
of priority, but corresponds to the constants (which are used, with
an offset of 1, as indexes into these arrays) in
offapi/com/sun/star/mozilla/MozillaProductType.idl
"
This reverts commit 9b92d8afbabe564de703f8bc083723d97e765011.
Change-Id: I166237285f13291083a506852d5217fc3ad59d45
Reviewed-on: https://gerrit.libreoffice.org/15164
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
stopgap measure until tdf#57873 is fully implemented,
since these days by far more people use Thunderbird
than Seamonkey (or Firefox address book, if the latter
even has any...).
Change-Id: I638ec90277063b1e7536c045bffb1052d3b76911
Reviewed-on: https://gerrit.libreoffice.org/14800
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
at *m_aValue.m_pValue.
But there could not even be a pointer there, e.g. if m_aValue.m_nIntXX is in use.
Then the pointer dereference usually leads to a crash.
Can e.g. be reproduced by calling getBytes() on an integer column of a RowSet.
Change-Id: Ib5361d838d2869142fd797d4e3454e2562ea7acf
Reviewed-on: https://gerrit.libreoffice.org/14718
Tested-by: David Tardon <dtardon@redhat.com>
Reviewed-by: David Tardon <dtardon@redhat.com>
|
|
broken since 4bf95c4c31dd871065d4042028a4b3ae476dab70 "make use of startsWith()"
where the tricky
!url.compareTo(::rtl::OUString("sdbc:address:kab:"), 16)
was replaced with
url.startsWith("sdbc:address:kab:")
even though the original exempted the trailing ":" from the comparison
(cherry picked from commit 27e85dd314e02dc97612bf1682fccacaf0244127)
Conflicts:
connectivity/source/drivers/kab/KDriver.cxx
Change-Id: I45175b2669d61eb4cee36d4456ccb69d0f11ef63
Reviewed-on: https://gerrit.libreoffice.org/14670
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
broken since LO 3.6, commit 07d512eff12fc89adbfb73e546b3946494f21472
"connectivity: convert kab to gbuild"
(cherry picked from commit d8a2a0b84a9aa570116febf23a6f1d6430fec287)
Conflicts:
connectivity/source/drivers/kab/KDriver.cxx
Change-Id: I65cc32fa55f1d5c0358585489b8b3dc3aaf8004b
Reviewed-on: https://gerrit.libreoffice.org/14669
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I323f6532d00f30cde50616e9399cad6bbe00cb8c
Reviewed-on: https://gerrit.libreoffice.org/14309
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
Change-Id: I25b4ccdc2f50ba89687e1f4c871f286251a28e21
Reviewed-on: https://gerrit.libreoffice.org/14232
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I1a39378d521f128e6b8f35a0bce781435f94e5a8
|
|
Change-Id: I0bb6df332a6666a0ae96bba38626b2626f880490
Reviewed-on: https://gerrit.libreoffice.org/13923
Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: Iddb551b62b1ff3fc9ce977483e41faa9ad8aef13
Reviewed-on: https://gerrit.libreoffice.org/13797
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I651c7b062b454fad85eff8852f7e62804a0d0058
Reviewed-on: https://gerrit.libreoffice.org/13796
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I73abbc7cfe0804e729a711a247b6e085b4e41d95
Reviewed-on: https://gerrit.libreoffice.org/13741
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
Cherry-picked from 84e3c5acb07937a073fedbe3d5389fc72123923a
Change-Id: I4f8d7e2bec006e6d0a0041e2145aa0920f64aa57
Reviewed-on: https://gerrit.libreoffice.org/13698
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I3c0cf976e0a9fbafe6eee768799a2f48c2ee0ea9
Reviewed-on: https://gerrit.libreoffice.org/13251
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
regression from
commit d60392fb5b387175dbd2e575848993c02c56922f
Author: Caolán McNamara <caolanm@redhat.com>
Date: Thu Mar 6 12:05:24 2014 +0000
coverity#706316 help out coverity re tortured logic
Change-Id: I2ee7bd3b7421ae46878d1b565ef14dd7b0bbd10e
(cherry picked from commit 6269b62b525ec22471db56015ac2daed813ec5ff)
|
|
Change-Id: I7e3c79d854a08c94aa699d86994704b9360b69bf
|
|
Change-Id: I299c856814e09a93a71f8acf8745a0590e2ecc72
|
|
Change-Id: I5e4c29ce5a1f427c5642a253f985b42938912075
|
|
Change-Id: I41ba46831f24b2960a1fe982b74a2b623e682e0b
|
|
found by PMD
Change-Id: I87780366119c141cd2dafe6ca1bf2d9798b10aec
|
|
Change-Id: I5c24856bfe48646bc0e9cd00f0849a5db12dfef2
|
|
Change-Id: I1dbf7a73ec60115f11d3a5455de2871041cd1605
|
|
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: I6752a534bca8f81627cdd616ac0f2268824299e0
|
|
Change-Id: Idf2709382dc431b6e34c2993d04c41aa8a3786ab
|
|
Added clear() method to OString and OUString class, Updated appropriate call-sites.
Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
it's a workaround, but given the conflicting requirements of
MSVC not liking duplicate symbols, but the unit test needing to
link the objects directly to access internal symbols, there is
not much else we can do.
Change-Id: I9bdcc5f2b6ab0712cc3317033c4fbae4791cbb6b
Reviewed-on: https://gerrit.libreoffice.org/12387
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959
Reviewed-on: https://gerrit.libreoffice.org/12164
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
|
|
found by PMD
Change-Id: I05b45382b8fb1b734657ce9421a20e6ef6fbe542
Reviewed-on: https://gerrit.libreoffice.org/12376
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.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
|
|
(Original idea from Kendy)
Look for code that is calling std::find on a sorted container
(set/map/vector) and warn about it - the code should be using
the find method on the container itself, since that is considerably faster.
Change-Id: Ib74e5d3faa836eeb0df16a736d202696626bdfd2
|
|
Change-Id: Iae4c439de43fba471916463da8939709bbd068e5
|
|
they are largely unnecessary these days, since our OUString infrastructure
gained optimised handling for static char constants.
Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
|
|
Change-Id: I3d198d60524224e22d7f609ddde1ed8c3c319da9
|
|
Change-Id: If89d6613394220fa4ca1da9b5b8820d9dbc205c7
|
|
Change-Id: I2dde714f847f6df4d9b4ede744bef089a3715d7a
|
|
Change-Id: I7200fe17c66cd904f0fc243bf8e33d6c24079729
|
|
Change-Id: Iee569c32569339ec2eeb6ef038e5bdc83e541a8f
|
|
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>
|
|
and
coverity#1158394 Uncaught exception
Change-Id: I73430678c658202475e0073c3c9bde0a072c1ee9
|
|
Change-Id: Ia43976d84eede6f699381bc4f3daf89b95e4cb4f
Reviewed-on: https://gerrit.libreoffice.org/12150
Reviewed-by: Bryan Quigley <gquigs@gmail.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I66499283f4218662146c380f605bb20b8a2ff49a
|
|
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: I3b1b78127d7d4080a23c0ac9f7d37706197de62c
|
|
Change-Id: If3963e0ae2c6477eb074e89f411c22a82d339634
|
|
Change-Id: Ie3b66520ab0c859272a4b0291f6c907e7fce61e5
|
|
Change-Id: I1e957afe1dc35571128fac4432b54e8ae35aa9ab
|
|
Change-Id: I3923a6a83bfc0a35f5a5af86cfd0e5308cfda24c
|