Age | Commit message (Collapse) | Author |
|
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: Ib30aa4e8f430131f5f073169d694e53a3bcd6960
|
|
Change-Id: Ibc1580d04d896a953edc15fb6cc9381914230e32
|
|
Change-Id: I8cfdf792541b351a19e16f95d4e6f7127e3d33b2
|
|
This partially reverts commit d87c2c59c9c1d5f5825f355c9eb941fdf95b42f6
"sdbc file driver (Prepared)Statement: created ResultSet owned by *caller*"
From that commit, we keep the part about not reusing the same
ResultSet on reexecution (client code may have disposed it or closed
it), but we revert the part about not closing / disposing the previous
ResultSet on reexecution.
Change-Id: I4946207f9740484b2f5fbc3575a5708fe39f357c
|
|
Change-Id: Ib48999c7fe532b2a8bf5222ef27d8d4929937e3b
|
|
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
|
|
There's no property USEBOOKMARKS for mork/moz
(regression from 0fadbdbbdbd55cf60f9a202f1cf7b1d60517078c)
Change-Id: If81f063fad2d702c9d8b0bb4147757655cb016cf
|
|
Change-Id: I7f22aa5576ca3c62fa363971f4fa5992b7711563
|
|
also
coverity#706301
Change-Id: Ie6eba530c9a2f822226fbd1b1a6aa49deaf99659
|
|
See https://bugs.freedesktop.org/attachment.cgi?id=100839 and other comments
Change-Id: I1187b171caf54997f5e8850ade4a3fb35ad9ec29
|
|
Change-Id: I451b13be6b72925e4cfa0e66c5690eca22c54dad
|
|
Change-Id: Ie6e8d4272b0b1d0d2ce93bcbc2e818a9eac1a56b
Reviewed-on: https://gerrit.libreoffice.org/9629
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
Change-Id: Ia0f1951f1e3edcd858a37ea091550feb2b9fd451
|
|
Change-Id: Id3bbb7a0d39a41b7106303f42d55ba4968235b63
|
|
connectivity: 3m52s -> 2m47s
cppcanvas: 28s -> 13s
cppuhelper: 20s -> 14s
dbaccess: 2m38s -> 2m01s
hwpfilter: 16s -> 13s
sot: 21s -> 16s
Change-Id: I49286bfe6be73dd1b861be632b95e17a99e82f8a
|
|
Change-Id: I5d5dc6871789865189c78fe9135e7f8bff829489
|
|
Change-Id: Ia36e192d32c46a5c1129eca050b2863ec1213bb4
|
|
Change-Id: If3d2dbe70fc3448ca6ab8ef8af62f5b571888c86
|
|
This required some changes to the framework:
* Init-/DeInitVCL is no longer done per individual test in BootstrapFixture, but
once per CppunitTest invocation in a new vclbootstrapprotector (similarly to
the exisiting unobootstrapprotector). CppunitTests that need VCL now need to
declare gb_CppunitTest_use_vcl.
* For things to work properly, the UNO component context needs to be disposed
from within DeInitVCL (cf. Desktop's Application::DeInit called from
DeInitVCL). The easiest solution was to introduce an
Application::setDeInitHook (where the hook is called from DeInitVCL)
specifically for vclbootstrapprotector to call.
* PythonTests don't (yet) call DeInitVCL; they still hook into
BootstrapFixture's original test_init functionality (to call InitVCL), and do
not make use of the vclbootstrapprotector.
Change-Id: I4f3a3c75db30b58c1cd49d81c51db14902ed68b2
|
|
Comparing with hsqldb part, sdbcx::OCollection link wasn't present in Tables constructor
Also include Catalog.hxx to be able to call "refreshTables" method
Change-Id: I1d8f2b0b361f8fd90f3065c07b3224894a3700d6
|
|
and then throw an exception when the interface is used
Change-Id: Id5e2c852323d49f51ab55cf46ab54223cf8ccf03
Reviewed-on: https://gerrit.libreoffice.org/9590
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
See http://nabble.documentfoundation.org/Strange-part-in-connectivity-source-drivers-jdbc-tools-cxx-td4110680.html
Change-Id: I1ed37671e6c248ada7a972dba00e0fe98b565e75
|
|
Change-Id: I64a74ea23d9a2410ebff0c42c54601fd0b0190f0
|
|
Change-Id: I30ab0dcf47d236c8ebda3a2d1da0aaa4f0cd7477
|
|
Change-Id: Ia9176826b89b947408230bf65db26a83f269c845
|
|
Change-Id: I70522657f968ed4f84e9447dd999d8be746fc973
|
|
Change-Id: I0e5c3af0504f3090a5db7692c6bfb1d33b3aa81f
|
|
Change-Id: I435fcfd6d10875565b543ea76fa8bfbf52e5f5d0
|
|
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: Icc81d3067969d1d61182865d36ecabec4e7f85b7
Reviewed-on: https://gerrit.libreoffice.org/9551
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
Thank you Lionel and Tor!
(see http://nabble.documentfoundation.org/Strange-part-in-connectivity-source-drivers-jdbc-tools-cxx-td4110680.html)
Change-Id: Ibbe68b8b3d12e4dfacf7f66bb3adbc2fd831d1a5
|
|
Change-Id: I0fc77b50792e48be175ab7cc151509e097676849
|
|
when throwing exceptions
Change-Id: I6edfb6b6745499f802b0e3c0e096a36fb7c32aac
|
|
.. now that we have a default value for that parameter
Change-Id: I54d6b0a0a01ce2f2e5168ada0c427424de0477bc
|
|
Change-Id: I1c21dfb48f4ecdcc3c599d69fe76bde006a85313
|
|
Change-Id: I3b63a42444a116afe7d88e2f806b17a709651e34
|
|
Change-Id: I53aab00dfad0310a7846fdf7185c50036329343c
|
|
Now that we have default values for Exception constructor params,
remove lots of boilerplate code.
Change-Id: I620bd641eecfed38e6123873b3b94aaf47922e74
|
|
Change-Id: I629b5f6e57c5d8368bafc544c1c23785ee209a1e
|
|
Change-Id: Iab50c52a132c90389992ef68c2d31df95a193ab9
|
|
Change-Id: Ib92122311186da16fbef9441560bb7a013072e76
|
|
Change-Id: I7769625289d8bf47fe5905dfb91d3bce9e0f5c85
|
|
Change-Id: I1a4d66a78549ea49649a6bed92acb44dbd8b13e2
|
|
for i in `find . -name *.src -or -name *.hrc`; do FILE=$(cat -s "$i"); echo "$FILE" > "$i"; done
Change-Id: I2bac5ad3e1eb3c566e5c867ccf45893a19e1561e
|
|
Change-Id: I7bde1200dbe9da76e062f2783cc223a07084c6fd
|
|
Change-Id: I42787db31769e8bde984c5f4f0aa90335e889b1c
Reviewed-on: https://gerrit.libreoffice.org/9356
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
|
|
Change-Id: I7f4785088151490ae1d55d2bb78bce28357d5c45
Reviewed-on: https://gerrit.libreoffice.org/8863
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
|
|
Change-Id: I8a2b54c97f20e581b9703bf33dbba537c2f603c2
|
|
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5
Reviewed-on: https://gerrit.libreoffice.org/9360
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|