Change-Id: Ibd370623f89341560d5d3a6c3bd073cc7365fab9

Complete commit 8310642c

It passed "make check"

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

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

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

by using for range loops

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

Project: help  f7e82f12585648e59c308607d8754616c5d930b0

Fix typo

Change-Id: I8fd351833236fe594a0480d9e09b0ef072ec7c46
Reviewed-on: https://gerrit.libreoffice.org/48997Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

Project: help  11adc4492b11253c5d24d217354581956da1f52b

Fixes for MS browsers and tweaks to fuzzy search

Some experimental JS tech had slipped into the mix.
Search settings should also be a bit more performant now.

Change-Id: Ifae9986efc02d20fefa9480ae463107fe4743944
Reviewed-on: https://gerrit.libreoffice.org/48988Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
Tested-by: Olivier Hallot <olivier.hallot@libreoffice.org>

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

PPTNumberFOrmatCreator's nTextOfs and nBulletOfs are both sal_uInt32, while
SvxNumberFormat's SetAbsLSpace and SetFirstLineOffset both take short parameters

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

Change-Id: I73c9ee7aef7062e3cb025c6ae0f3bb6a91485330
Reviewed-on: https://gerrit.libreoffice.org/48983Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

OBookmarkContainer actually re-uses the reference count of
ODatabaseSource, so converting ODatabaseSource::m_aBookmark to Reference
created a cycle, which somehow causes the dataloss, because evidently
something as important as storing the data must be done in the
destructor.

(regression from 2660d24a)

Change-Id: I4cad01dc9cdaf405c1eb31d6c0e161eb6712b78f

Change-Id: I903362c158190a9f1ee68c2e580a0c695a4c86d8
Reviewed-on: https://gerrit.libreoffice.org/48970Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>

Look for a whole column at once so we don't have to iterate all draw objects
for every cell, but only once per column.

Follow-up for 3a2a430a

Change-Id: Ic8740fca7d595528785b432c1cedf4fad4f13ba1
Reviewed-on: https://gerrit.libreoffice.org/48416Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>

...so there appears to be no good reason to cast from sal_Int32 to sal_uInt16 first

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

...implicitly casting it to smaller types where necessary, instead of the other
way around

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

...so there appears to be no good reason to cast from long etc. to sal_uInt16 first

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

...and none of the places that call SvxNumberFormat::GetAbsLSpace (also
returning short) look like they deliberately cast back to sal_uInt16 (i.e., to
tunnel a genuinely unsigned value through SvxNumberFormat), so it looks more
plausible to restrict values here to short and SHRT_MAX, instead of sal_uInt16
and USHRT_MAX.

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

...(and both calls of CorrectTabStopInSet pass in the result of a GetAbsLSpace
call), and SvxTabStop::GetTabPos() is sal_Int32& (and that's what nAbsLeft is
combined with inside CorrectTabStopInSet), so why use an intermediary parameter
of type sal_uInt16?

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

Width reduced to 75px

Change-Id: I269eed9054e75d3885cb53d6a773460d30c67a14
Reviewed-on: https://gerrit.libreoffice.org/48896Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
Tested-by: Heiko Tietze <tietze.heiko@gmail.com>

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

Change-Id: I363ad44545adae11fb4d74048d906eb6fdd8e819

...so there appears to be no good reason to cast from long to sal_uInt16 first

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

Change-Id: I3f3ebaea1fe3a3b3919fe204bd8769287e84deaf

Change-Id: Ie44698565865b3aacc1ebd0a3e41d5001bf8e09a

Change-Id: Iebcf12eec5cf5282e798ff5d4fe6649e3a8eea3f

Change-Id: I8bd18130c5fe87b2b352d285d58dd6a4f4499768

Change-Id: I85a096505b0de98fd1de6179b9239318622fc34a
Reviewed-on: https://gerrit.libreoffice.org/48933Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

by using for range loops
Remark: some iterators were used outside loops,
these have been replaced by the counterpart origin variable

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

Change-Id: If47716c6dbc353059c42a2bcd09f7c11ed53796f
Reviewed-on: https://gerrit.libreoffice.org/48937Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: I8ce26b7cddeb74fd86f60896c84bf3574ccbc9bc
Reviewed-on: https://gerrit.libreoffice.org/48421Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

Change-Id: I8bf7adb11e49d926a5910a4bd8279e16cf34833f
Reviewed-on: https://gerrit.libreoffice.org/48943Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

in scstyleloaderobj

Change-Id: I7702f6458a56a9a4f8514a354505cf8f93ec8390
Reviewed-on: https://gerrit.libreoffice.org/48932Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: I4c381b7f7520bf7d3103dfea22e3643b2eae01b1
Reviewed-on: https://gerrit.libreoffice.org/48940Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: I94900cd87e61b5ee6618ccb736bee82c349c8cc3
Reviewed-on: https://gerrit.libreoffice.org/48942Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

During inserting of the text from the external document
we should not change original default styles in
currently opened document.

Added unit test.

Change-Id: Ida754a0da5efaaa043464cd807c0b52cfb0d5670
Reviewed-on: https://gerrit.libreoffice.org/48826Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

Change-Id: I9a255067cce2ae013d1054fbad7efa10bce4378b
Reviewed-on: https://gerrit.libreoffice.org/48939Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: Id0d9e3f86f9e48333f415b4eb2a05b3211473b67
Reviewed-on: https://gerrit.libreoffice.org/48925Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: I054de2c9f421e13069314146f972e95ab9ae322b
Reviewed-on: https://gerrit.libreoffice.org/48924Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: Ia97f3e7b637c2b14a9c07c521dcd5b8ff1ff0686
Reviewed-on: https://gerrit.libreoffice.org/48935Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: I6c96bf694ec64fec2ddae66a42cf8cc2a8fc0e7a
Reviewed-on: https://gerrit.libreoffice.org/48931Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>