This changes the condition to only select row if it's not selected yet;
previously, because of the always-true condition or'ed, SelectRow was
always executed.

Change-Id: Id12b640518f3d72810e997735216db336d79f7f6
Reviewed-on: https://gerrit.libreoffice.org/63267
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

...that had been added to the stable URE interface, apparently by mistake, with
7474699a "contains some slightly changed macros
from vos/macros.hxx", then removed with 2f0803e5
"remove unused macros, and add SAL_N_ELEMENTS", then re-introduced with
d441fa1f "remvove vos/macros.h usage".

The last internal uses have now been removed with
<https://gerrit.libreoffice.org/63264> "Replace uses of SAL_BOUND with
o3tl::clamp".

Change-Id: I0dd842533d97fa878606cbf62efb9fa46c029088
Reviewed-on: https://gerrit.libreoffice.org/63265
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Change-Id: I66a7aad64623d778b4bf2fea591f227fdac2fdc7
Reviewed-on: https://gerrit.libreoffice.org/63264
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Change-Id: Iefc820a4c4bb87fe113f97b085d8e89f30ff2db5
Reviewed-on: https://gerrit.libreoffice.org/63261Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins

Change-Id: Ic900d5116fd1cf5a90526078c6e4802f09589aa7
Reviewed-on: https://gerrit.libreoffice.org/63253
Tested-by: Jenkins
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Change-Id: I04b0b0dc85afdceead9766d8a45e21055074d5fc
Reviewed-on: https://gerrit.libreoffice.org/63248
Tested-by: Jenkins
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Change-Id: Iaf27254896783deaeaee65bd51617d8e4e457873
Reviewed-on: https://gerrit.libreoffice.org/63244
Tested-by: Jenkins
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Change-Id: I8a27c11c4d8f345b316d7b2bf3bc0fdfd2e03b4b
Reviewed-on: https://gerrit.libreoffice.org/63258
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

Change-Id: Ib9203fa65d7b9825b0a7181dcfc0f522bc575541
Reviewed-on: https://gerrit.libreoffice.org/63242
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I829c660c6c73868ac6a0f74e554d99c54d9f0ea8
Reviewed-on: https://gerrit.libreoffice.org/63245
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: Ic7c97853eeeaa87e7ed7ecfad638f516f2be2969
Reviewed-on: https://gerrit.libreoffice.org/63246
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: Ifacc27284201f74da7b4693a74d40e83ffc66408
Reviewed-on: https://gerrit.libreoffice.org/63251
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: If2e1643a507f13e7da733896b1115ef39b512f00
Reviewed-on: https://gerrit.libreoffice.org/63252
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I0e5e2d607bddd1f73fc8c256169a2fbe070f6448
Reviewed-on: https://gerrit.libreoffice.org/63256
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: Id2478ac637140b604cb0f7e3aa4267f02aa859c4
Reviewed-on: https://gerrit.libreoffice.org/63255Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins

V547 Expression is always false. Probably the '||' operator should be used here.

Was like that since commit df64bd53.

Trying to fix it by restoring the intended logic (|| instead of && to disallow
setting stylesheets for Para or Page) breaks several tests in
CppunitTest_sd_import_tests; apparently the code relies on these families
getting stylesheets. So just drop the non-functional (always false) condition.

Change-Id: Ia547c089ba0ed4ae879ede89043aa737e4409a75
Reviewed-on: https://gerrit.libreoffice.org/63234
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

V555 The expression 'xBlob->length() - nDataWritten > 0' will work as
     'xBlob->length() != nDataWritten'.

Calling XBlob::length() only once allows to avoid overhead of acquiring
mutex per each iteration.

V555 The expression 'xBytes.getLength() - nDataWritten > 0' will work
     as 'xBytes.getLength() != nDataWritten'.

Change-Id: I2e1772fe3af16ac6a6d6f5d2854e84f2fc69ff5f
Reviewed-on: https://gerrit.libreoffice.org/63243
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

Change-Id: I7a6c1e452897db1a2bd9820cf681e883ae71bb5b
Reviewed-on: https://gerrit.libreoffice.org/63257
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

Change-Id: I1ddf3fe0e5fad265ae14712a23469b684253079d
Reviewed-on: https://gerrit.libreoffice.org/63241
Tested-by: Jenkins
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>

Change-Id: I4a3614461740bc921ef99d71e2d057f5a17a75a9

Change-Id: I3d160c71ac4874a66f036e3d91910e00cf47836d

Change-Id: I67e8e9e2ecd7ec89d51bd9597b54be53c3203d46

Change-Id: If4dcd0a3f316bd9812ffb12b8e64489d25ce89ca

Change-Id: I336924c43b6680c3742ca8c83801cbff3f346968

Change-Id: I74669145a1341ff2a715cdec32676dfdd290d1f3

Change-Id: I56aa03bd727b8cdea38de7948a4dad4532ee214c

Change-Id: I04279faaf11bd1249bcea0d07d60facefb5186d8

Change-Id: I6c367b00024e96aa4eef5490b3b296617851016f

Change-Id: I5e03fceb83a22b590a62b605c4ada279b937730a

Change-Id: Ifc509b3c31f6dbfd4d620aa451fd85b82b0ddb90

Change-Id: I0ffcbbfe2380755ff21532b314961c317ce0c69d

Change-Id: Ia71e4822ce863a296c7c0eb137c164538f9a0d27

Change-Id: I810172d045860f49819b9e25de8a2d5fe5f80dd8

Change-Id: I0a0068649fd688f311e1ba4865a51406c4ffd672

Change-Id: Ic5397601267f1953379a13a3e1ac69813d363045

Change-Id: I769660ab1832e76ed7ac272d0d2c973d14171c06

Change-Id: Ic72837cbb345a26faf3fd7cbf8739ddb6dc76cf9

Change-Id: Ibaa2835e73dc42a7296e4749fdfbd05e282e04eb

Change-Id: I8009565c5ae2580d0cdbf3b323b95d61c25c0dcd

Change-Id: I1f6468e1050924b0c6f6971ad6276932a3c91f75