Age | Commit message (Collapse) | Author |
|
Change-Id: Ib15413e73409cc33de01fa92a47b9d1237cfc4b2
|
|
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: I670fbcd83e368f2821d37a392cbc007b1f4cd55e
|
|
...and also corresponding param of CreateTabPage function type and corresponding
Craete functions. There were some call sites that passed undefined "null
pointer references" and SfxTabPage internally uses a pointer member pSet that is
checked for null anyway.
Change-Id: I4eb3636155eac46c9c9d26e6e6e842e85d7e95af
|
|
...there was a call site that passed undefined "null pointer reference"
(apparently in a case where the passed argument was actually unused)
Change-Id: I663d4264b7a84f44ca69c732f3bc502f614b2b2a
|
|
...there were a number of call sites that passed undefined "null pointer
references" (apparently in cases where the passed argument was actually unused)
Change-Id: I19799e90f0cd8e98367782441a5ea9df27b59830
|
|
Finish the work of ec8617568ff091026b2f17cce3b547b1633fa6c4
Only one is left: SC_PROGRESS_CXX
Change-Id: Ifdc07ec377d7a84e92bc4b0a66ab82b1fc93f76a
Reviewed-on: https://gerrit.libreoffice.org/9636
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>
|
|
Change-Id: Ie7a05e694585bcc624bd69b4400d84a05fbc48fb
|
|
Because the same dialog is used when creating a new pivot table, where
the Apply and Close semantic feels a bit weird. Also, some features of
the dialog were designed with Ok / Cancel in mind (like the destination
location).
Change-Id: I9ac889bca9f967cb97b34ab45ea563547b58d422
|
|
Change-Id: I963419baec24d7599ddb3e481738cbe6ec4e0a69
|
|
for i in `find . -name *.src -or -name *.hrc`; do FILE=$(cat -s "$i"); echo "$FILE" > "$i"; done
Change-Id: I2bac5ad3e1eb3c566e5c867ccf45893a19e1561e
|
|
Change-Id: I7dc862794f7f0973bd4de7b54f7a1fb15235bc3e
|
|
Change-Id: Iade3fedac5d2f8e978b7dd9c30f001d7d1564946
|
|
Look for code like:
xxx ? yyy : false;
Which can be simplified to:
xxx && yyy
Change-Id: Ia33c0e452aa28af3f0658a5382895aaad0246b4d
|
|
Change-Id: I7b4161217ccd70d00bd43740719825ba19058bf7
|
|
Change-Id: I6572f70c1c0eaa149759189c6b196281e74e218d
|
|
Change-Id: If72fff8b6cc7e5e55e1f395ddc34b4c5810faff3
|
|
Change-Id: I0ce6965f191d1455bcc7a793025282e20b7433a5
|
|
Change-Id: I70aad0b38979f45a313b8ac36890fb6c64d11bb0
|
|
and remove some unused stuff
Change-Id: Iea6177c3d2f9d0668187e66155b62688ae2ecb04
|
|
also remove some commented out code
Change-Id: Ia80c5c57d8d2a74418032de50eee95642cc0969d
|
|
sorry, huge one...
|
|
Change-Id: Ic226202bb1ce429f9dd8e13183ec64ffc9059d05
|
|
Change-Id: I1cb588ce7df30886354ac0cf2bd81fb95a7e9d37
|
|
Change-Id: Ib14d24970faa35ad0caf8c8a8bf7f9cef5764b89
|
|
Change-Id: If2c233a8ced8815b065a72caff907f10ef6166ff
|
|
OAuth2Handler is from libcmis
Change-Id: Ia1986d6df7ab45580c66b4e536c5882af41f357f
|
|
Change-Id: I1a4f1f4f06ec1ab0b28637380a8bfcb3bc64ee30
|
|
Change-Id: If24d4cec9ef4369f20419fe70de7392614a35316
|
|
Change-Id: Id1dcadcac179c52977e48a6912ce4d5fd542f60c
|
|
Change-Id: If6aa0a955d8b80b2c1eb4d944829f46167b6fae9
|
|
Change-Id: I9e80a0d6661c29343103a27baa752cebbc876314
|
|
Change-Id: I3d367b8b638217f18143118df00324f21a0470f2
|
|
Change-Id: If8560d47c5b4e64367c441237c1e38f964cf6825
|
|
Change-Id: I78aa55539c2a3e522736bb178ba7d445300bf1f0
|
|
This commit adds a new pivot table layout dialog which was implemented
from scratch. Instead of custom controls this one uses list boxes
for field entries which greatly reduces the code. It also fixes
some visual and behaviour bugs and adds the possibility to edit the
"Data" field.
Change-Id: I6c01252acee5a2e8910e40e65904504d00e03057
|
|
Change-Id: Ic9cec3fc6749672b30bb1e0c96159897e50a84e5
|
|
Change-Id: Ib0f01bf529a739e5f92cd41979c18d76d7b545d1
|
|
Change-Id: I2fd5d432aff737016948a4f7bd45888cf6f4dba2
|
|
Change-Id: I1fd02cc148fd9a54d2092dad1e548f51a0813a14
|
|
Change-Id: I7b72c5680d5665b3f1f720f50a2d3ea6fc0c3e39
|
|
Change-Id: I85e8823e872bf53182ce90576225e462edb1930a
|
|
f70d03436b7b501e0ed1d745935a204b9b97b855 "coverity#1158232 have a stab at
silencing warning with function markup" claimed that NamedDBs::insert always
takes ownerhip of its argument, but boost::ptr_set::insert(std::auto_ptr<U> x)
simply calls insert(x.release()), so only takes ownership when it returns true.
ScDBDocFunc::AddDBRange (sc/source/ui/docshell/dbdocfun.cxx) relies on this
behavior, deleting the argument when insert failed.
ScDBDocFunc::RenameDBRange (sc/source/ui/docshell/dbdocfun.cxx) relied on this
behavior, deleting the argument when insert failed, until
f55cc330dec0dec60c755e2ce28a840c7fca1956 "Fixed the fallout of the changes in
ScDBCollection" removed the delete (presumably in error?). I put it back in
now.
All other uses of NamedDBs::insert ignored the return value (witnessed with
SAL_WARN_UNUSED_RESULT). Some are insert-if-not-found cases, where I added
asserts now (Sc10Import::LoadDataBaseCollection,
sc/source/filter/starcalc/scflt.cxx, is not entirely clear to me, so I added a
TODO), while others would have potentially leaked the argument, in which cases I
fixed the code.
Change-Id: Iad40fbeb625c8ce6b0a61cbf16298d71cdc7de80
|
|
Change-Id: Ia7db5746fa5d8f60111d10d09865120981956947
|
|
Change-Id: I795d296bae700d1e3a273fbe71fc842d145c5138
|
|
Change-Id: I7d8967f90da3ad19c82a313636910279d6ef0f73
|
|
Change-Id: I97cc37c05782b6ab2cd7b8b63a4315ba2386f1ef
|
|
Change-Id: I8d5a959fdca8f7deefffb0c3a0529d2e32595df4
|
|
Change-Id: I3c6256f1b2d176790a8eb7c094c1583ce053971b
Reviewed-on: https://gerrit.libreoffice.org/8382
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|