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

in this case, we can just drop the child member and re-use the parent
member. The parent class barely contains any logic anyway.

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

This patch is sponsored by ULAKBIM/Pardus project.

Change-Id: I9f1f5bd0f05b7f92204615b48a1e7be06272a8a3
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/47597Tested-by: Jenkins <ci@libreoffice.org>

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

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

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

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

Previously it could easily be misread as "Include comments - only in boundary columns"
(and the German translator obviously also read it that way).

Change-Id: I37f58ebd6b1700e157d2355bbf05c4b42d5aae29
Reviewed-on: https://gerrit.libreoffice.org/48352Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

Instead update them unconditionally as it is done with comments.

The data range has been set beforehand (considering the option
to include boundary columns with images only) so at this point
the swapping should happen unconditionally.

Follow-up for 3a2a430a

Change-Id: I1d75dd6e22db5cc28aa45f7b86dab013f3595199
Reviewed-on: https://gerrit.libreoffice.org/48383Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

Follow-up for 3a2a430a

Change-Id: I46a4f8a4528b73e4dcb01770af127ddea4000fdb
Reviewed-on: https://gerrit.libreoffice.org/48382Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

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

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

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

Change-Id: Idaf9e1858afb8f22bba1767470138ec820a8d7cb

...presumably forgotten when the following cast of nRepeat in the
SetRowsToRepeat call was changed from USHORT to sal_uInt16 in
7f33ed41 "removetooltypes01: #i112600# Remove
tools types from sw"

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

...as the default Point ctor already zero-initializes its members, removing the
need for the memset call (that causes -Werror=class-memaccess, "clearing an
object of non-trivial type 'class Point'" with upcoming GCC 8).  Also use
unique_ptr.

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

...to avoid GCC 8 -Werror=class-memaccess ("clearing an object of non-trivial
type ‘struct SwSrchChrAttr’").  Similar to
<https://gerrit.libreoffice.org/#/c/48488/> "Allocate ImpXPolygon::pPointAry as
a true Point[]", std::fill appears to produce adequate code here with recent
compilers.

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

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

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

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

and flatten some data structures, no need to allocate some of the the
vector members on the heap

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

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

The patch was sent for the ULAKBIM/Pardus project

Change-Id: I2c8c5d92bc6755043e086a0cad7739dd06a7655e
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/47390Tested-by: Jenkins <ci@libreoffice.org>

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

The checks against USHRT_MAX were like that already in
84a3db80 "initial import", while the casts to
sal_uInt16 were later introduced with a7762676
"#83294# SwXTextTable/Cell::get/setData with a sequence of rows instead of a
sequence of columns".

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

since...

commit 56b2cf0c
Author: Florian Reuter <freuter@novell.com>
Date:   Tue Sep 14 14:40:15 2010 +0200

    sw-collapse-empty-table-par-like-html.diff:

    n#376690

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

* fixes 2 CVEs
* disable some new optional dependencies

Change-Id: If7725d126e68de04b67969a83c0ea08573a43679
Reviewed-on: https://gerrit.libreoffice.org/48493Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

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

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

...as the default Point ctor already zero-initializes its members, remvoing the
need for some memset calls (that cause -Werror=class-memaccess, "clearing an
object of non-trivial type ‘class Point’" with upcoming GCC 8).  Other such
problematic memset calls are replaced with std::fill, which appears to produce
adequate code with recent compilers (looked at GCC 7.2 and 8, Clang 7, at -O2),
a tight loop of filling the memory with zeroes.

A follow-up commit might want to use unique_ptr or vector for pPointAry.

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

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

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

this is a safe backportable fix, the unconditional

nFlags |= SelectionEngineFlags::CMDEVT

looks the real dubious thing to me, but only the gtk3 backend has swipe support
so just returning immediately on a swipe definitely ok

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

The size returned by insertAt falls back to the master size if needed.

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

Change-Id: I74724ba61aadfad19501a87ab548219ea0a1f236

The bugdoc is affected by the change of default vertical alignment;
apparently it's not even possible to set the vertical alignment of a
Word 6 drawing object (\do) so we have to set the Word default.

(regression from c79467ba)

Change-Id: I4084a7a7e2a55f864cb569e04632e034d59eefdb

Change-Id: I5f6d47afc2f27b1033201b3fa280359a215f38b5

Change-Id: I6c20957fe3f95a8e63b30e5c733b99f71c193972
Reviewed-on: https://gerrit.libreoffice.org/48290Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Muhammet Kara <muhammet.kara@pardus.org.tr>

The check against USHRT_MAX was introduced with
9c034462 "INTEGRATION: CWS cmcfixes30: #i72614#
crash in loading corrupt .doc", apparently to ensure the static_cast<sal_uInt16>
in the call to ShortToSVBT16 below (which already was that same
static_cast<sal_uInt16> back tehn) is OK.

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

Change-Id: I50973ddb45ec9269eac4954026ff25e28289dc5d