Change-Id: Ib703cb24619228c081c156c4ed52bf7b878fbedd

Change-Id: I82fad3f152fe5a563675686accedb583d07d2b13

Change-Id: I508eb91d7c32e8604525511c33c39471afd3ffbf

Change-Id: I9521ed7d75cae1a1e90fd73dd7b8bd822ab4521f

and

coverity#705666 Resource leak

still actually fails to dlclose module, but explicitly does that

Change-Id: Idbf5608b052e3a7c5a465a73d8ad067a0a9494c5

and

coverity#1244938 Division or modulo by float zero

Change-Id: I128b1ac10f514285152cf189af24885460c9c5ca

or

coverity#735615 Division or modulo by float zero

Change-Id: I6509e987eaf58349e81b9ebadcd7e2d0ff0bcbcb

and

coverity#735607 Division or modulo by float zero

Change-Id: Iceeab702e183d291d7b1b6ded9113d1a15163c94

Change-Id: I6497303e1b2d8d5ffd962489657582387ab974b1

Change-Id: I4e25b0a6929cc88aa2ec1d213b8f1115de6be607
Reviewed-on: https://gerrit.libreoffice.org/12146Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: Ied82b0a39ff9dc9c7cc9539dec2b963059f3d4a5
Reviewed-on: https://gerrit.libreoffice.org/12139Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: Ied831cb5d6645e2696ff2d3d57843fc7a1f82f7e
Reviewed-on: https://gerrit.libreoffice.org/12137Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: I3d02436446a79137ce42b1a190c9e3985fbb9246
Reviewed-on: https://gerrit.libreoffice.org/12135Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: I0598e9d0c9c33c538aa02eabc2c09d96ec3c9b4d

Change-Id: If50922e238bebd9a02e9950ca27d99c2df9ab7d8

Also kill some unused measuring.

Change-Id: I69521d36e4da4b50cd58f346f0adb0e5563c8bec

Change-Id: I6af10899a3e22c45b973439fd8c8ec6c8faac904

Change-Id: Ic06a3b4673446bbadd21f2380c04ec628a6f86a9

Change-Id: I09d0777ef51045054b8e3bc88aba9fb025e6f9ff

Project: translations  6464f1a0e1abc7b0db94aa161885aa08f3a06fe5

Change-Id: If222be9574af1078ffba15c07f048503dd8d7e8e

Change-Id: Ie5650c3a5593dcf68196802169c47270a0c3b0e6

Change-Id: Ia76df719b2f76a993e65e0bf6d55723b1a2f5b93

Change-Id: I0d89cc99a1d8f7b3c17986b45653dce5e4b436d7

Do not abort in tests which take longer then before.
Since 4f5f6d24 we have nice tool to
collect callgrind results, store them..

Also, make it not necessary to duplicate anything. Any test can be turned
into a performance test by using callgrindStart/callgrindDump. You only
need to duplicate the makefile name in gb_Module_add_perfcheck_targets.

So, if the test is run as part of 'perfcheck', it runs under callgrind,
otherwise it behaves like a regular unit test.

Change-Id: I7acbb855b1823c9a32ba126abb57dccc767ca239

all sorts of things crash, e.g. format->properties

Change-Id: Ida4fb97dcdddd7adde4b98fa67b107a514eef615

Change-Id: Ibdbfb4a88c1c75aaf5d33672d8639a9ea55afbac

by using a byte-swapping pattern that coverity doesn't detect as such

tested as a scratch coverity attempt with a smaller project which
has a far higher allocation of coverity attempts per week :-)

unsigned int readTaintedUINT32(const char* buffer)
{
    unsigned int v = (
            (buffer[0] << 24) |
            (buffer[1] << 16) |
            (buffer[2] << 8)  |
            (buffer[3] << 0)
        );

    return v;
}

unsigned int readUntaintedUINT32(const char* p)
{
    unsigned int v = *p++; v <<= 8;
    v |= *p++; v <<= 8;
    v |= *p++; v <<= 8;
    return v | *p;
}

void foo(char *buffer)
{
    char *pOne = new char[readTaintedUINT32(buffer)];
	// ^ coverity only reports this
    delete [] pOne;

    char *pTwo = new char[readUntaintedUINT32(buffer)];
	// ^ and not this
    delete [] pTwo;
}

should silence

coverity#1213371 Untrusted value as argument
coverity#1213372 Untrusted value as argument
coverity#1213373 Use of untrusted scalar value
coverity#1213374 Use of untrusted scalar value
coverity#1213376 Untrusted loop bound
coverity#1213388 Use of untrusted scalar value
coverity#1213389 Use of untrusted scalar value
coverity#1213390 Use of untrusted scalar value
coverity#1213423 Untrusted value as argument
coverity#1213424 Untrusted value as argument
coverity#1213425 Untrusted value as argument
coverity#1213432 Untrusted value as argument
coverity#1215304 Untrusted loop bound

Change-Id: Ib8c7fc9a8e8b36ca227c76577d991c10df7dcd5a

The same information can be seem from the size of the vector reference
returned by sc::opencl::fillOpenCLInfo().

Change-Id: I797aacdf85a852bc4fa65e0536fd5940aed396a1

Also works around some warnings from our Clang plug-ins.

Change-Id: Ic9d2bbafefac345b6319300514735143f92dd196

Change-Id: I3a207f86611a5078cd2de1437cd48e612232aaf2

Just link that code into the sc library. We use clew anyway to handle
optoinal presence of OpenCL at run-time.

This is a straightforward change for now. Could be cleaner; we could
get rid of the fairly pointless wrappers in
sc/source/core/opencl/formulagroupcl.cxx for instance.

Change-Id: I23215c6516ef04698f6891e2a7fc6611dc83b37a

Change-Id: Ib2c8da0d901a81a41be16ec0c08985f991f1b7ab

Change-Id: I25a96479eafa97eccace2879bf8e1af5e85f60c8

Change-Id: Ib41c48b328835da533cba4b67dc34ce3ddfc1f9a

- Clean up OSL_ASSERT

Change-Id: I1f3a5dcfe08876da9bb4a8486311eb0ca9bab215
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>

It turns out that almost none of them were necessary.

Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83
Reviewed-on: https://gerrit.libreoffice.org/12133Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

...when std::clock_t is sufficiently small.  Though the formula used in
SwLayAction::CheckWaitCrsr does look odd...

Change-Id: Iee32350850be4a1896c5cb41eff748f0cb5382a2

appends results on existing target file

Change-Id: Icd897b090e1d1ed896b88a2f5923e8f35e95e5d2

Change-Id: I045a65d4efbaf8fcb55318ffce4f8929e2fbbe15