Change-Id: Ib268a6b32257aee812e5bae27a6db94431d2abbb
Reviewed-on: https://gerrit.libreoffice.org/71739
Tested-by: Jenkins
Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Regression from commit a5a836d8 (DOCX
filter: effect extent should be part of the margin, 2014-12-04), the
problem was that extending margins as-is based on the effect extent
values only work correctly in case of non-rotated shapes.

For example, with 90 degree clockwise rotation the top effect extent
should extend the right margin, etc. Fix the bug by limiting this
extension to the non-rotated scenario.

Test the behavior at a layout level, so in case later the effect extent
feature is implemented, it won't be necessary to adjust the test.

Change-Id: I97271bbb7c079951980b436cb8d8e5e54eeead55
Reviewed-on: https://gerrit.libreoffice.org/71878
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

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

need to release the pointer we get back from demangle

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

the Add can fail, and this is a reference-counted type

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

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

XParaPortionList::RefDevIsVirtual() does not depend on eRefDevType

Change-Id: I22182bbe26502552125d24aa1a8c33ffb5a38971
Signed-off-by: Adrien Ollier <adr.ollier@hotmail.fr>
Reviewed-on: https://gerrit.libreoffice.org/71649
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Change-Id: I0db9ec275cc95184ceb2cdbce8ae5343a10582c1
Reviewed-on: https://gerrit.libreoffice.org/71861
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

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

The output of "<strong>qlt</strong>" was literally
<strong>qlt</strong> instead of "qlt" strong formatted.

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

Change-Id: Iead091fee3b6c72d610f42598edb7cbb681d0cd4
Reviewed-on: https://gerrit.libreoffice.org/71870
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>

* Update dictionaries from branch 'master'
  - Updated the English dictionaries: GB
    
    Change-Id: I7d1f80c516b34d6ff7548869bec95e13f40735f9
    Reviewed-on: https://gerrit.libreoffice.org/71621Reviewed-by: Aron Budea <aron.budea@collabora.com>
Tested-by: Aron Budea <aron.budea@collabora.com>
    

Pointed out in the post-commit review of
<https://gerrit.libreoffice.org/71592>, thanks Mike.

Change-Id: I49006e041a5110e995af44b7437ba0b48c4c860d
Reviewed-on: https://gerrit.libreoffice.org/71868Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins

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

... differently; commit b7d4418c
was partially wrong because if SetPrepWidows() isn't called and the
master and follow have fewer lines than the sum of orphans and
widows, then the frames must be merged, i.e., master must move
forward to the next page and follow must be deleted, but that only
happens with SetPrepWidows().

However if the SwTextFrame::PrepWidows() determines that no lines can be
moved from the master, SwTextFrame::CalcPreps() will grow the master
to force it to move to the next page, and then it sets SetWidow() too,
so check this flag to break the loop in SwTextFrame::CalcFollow().

Another question is what is the point of the "very cheesy" follow
formatting fall-back, which might be removable.

Change-Id: I0769a5a4f13ca4c95c2229a496207809d05576f9
Reviewed-on: https://gerrit.libreoffice.org/71866
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>

Correct the position & size by roundtrip conversion
from/to wmf as a temporary solution.

Simplify a bit.

Change-Id: I59f32bd29750f9ac759800893583308f29b8aad5
Reviewed-on: https://gerrit.libreoffice.org/71860
Tested-by: Jenkins
Reviewed-by: Muhammet Kara <muhammet.kara@collabora.com>

As seen in tools/source/inet/inetmime.cxx

Change-Id: I3122e07a01020e84b5b75bc46dcaca33560b84bb
Reviewed-on: https://gerrit.libreoffice.org/71558
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

This had manual consistent formatting. Recently it was broken, so bring
back consisency by using clang-format.

Change-Id: If4303e379ddc77e5190f8e8e42737b4f7d680a71
Reviewed-on: https://gerrit.libreoffice.org/71857Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins

Kind of a regression from commit 4d382636 ("qt5: Add basic
a11y support"), which made it public for a single call.

Change-Id: I631a861a98388223770cfca2704c3ddee6a0a8a0
Reviewed-on: https://gerrit.libreoffice.org/71836
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>

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

No idea why this has been disabled, Clang can handle both plugins
and PCH at the same time.

Change-Id: I957b57f5f86b041c0437a121990b40bfa0095a0d
Reviewed-on: https://gerrit.libreoffice.org/71564
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>

Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.

Change-Id: I564e3dd53e7718326fc1e8b98c9ae3367381703c
Reviewed-on: https://gerrit.libreoffice.org/70550
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

Change-Id: I26deeefc8cf335f53aa55c8523cd08687f27b63b

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

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

Titel -> Title

Change-Id: I1b5b670ee372f948f624bd4b21bb41b4e0004d4a
Reviewed-on: https://gerrit.libreoffice.org/71789
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>

Translated the defines, but kept the their NC_(...) translation
references to prevent a need to update translations.

Standardized terms from http://www.upu.int/en/activities/addressing/s42-standard.html
were used, although textual strings use US English terms.

Change-Id: Ic5b13d21f9552d3e8b2adce92cca36f651eaa7eb
Reviewed-on: https://gerrit.libreoffice.org/71792
Tested-by: Jenkins
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>

Change-Id: Iaf7ed92d3ef132a79de30f0d596acdfba9c7185a
Reviewed-on: https://gerrit.libreoffice.org/71848Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins

These are larger C++ libs and without gdb-index gdb takes a moment
to load such libs.

Change-Id: I555a629199f761060199a528415f7d5fbe9d9533
Reviewed-on: https://gerrit.libreoffice.org/71822
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>

Non-ASCII letters were stored using percent-encoding, resulting
broken bookmark names after export/import. For example, the word "Első"
became the wrong "Els%C5%91". Now only the reversed ASCII characters
are stored in percent-encoding.

For example, the name "Első!" stored in the following form:

<w:bookmarkStart w:name="Első%21" w:id="0"/>
<w:instrText> REF Első%21 \h </w:instrText>

Change-Id: I65168e071b6baa12385c0aaa12d9f2ae4ccf9f98
Reviewed-on: https://gerrit.libreoffice.org/71299Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>

To give extension developers a clue what is missing

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

need to check for ownership passing in both variants of the loop, so
just move it into the common block below.

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

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

- fix unittest assuming the double-insertion of annotation
  throwing an IllegalArgumentException was intentional
- remove SwModify/SwClient in SwXTextField
- also: add basic C++ api test

Change-Id: Ia4657dc65dfadc3e975bdf409bd5e43413ea1f5a
Reviewed-on: https://gerrit.libreoffice.org/71452
Tested-by: Jenkins
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@libreoffice.org>

For Offimisc Writer template
zip mimetype file with -0 option
--filesync option must be removed as it remove mimetype
remove unused test of exiting files
restore script extras_Tploffimisc_make_file_deps

Change-Id: I7851635f483d10bcf179b568250be31d763d60b9
Reviewed-on: https://gerrit.libreoffice.org/71473
Tested-by: Jenkins
Reviewed-by: Laurent BP <laurent.balland-poirier@laposte.net>

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

Change-Id: I3ab8371b14068601995616dc9979a3297313d761
Reviewed-on: https://gerrit.libreoffice.org/71563
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>

...after eeeec33a "merge
--enable-selective-debuginfo into --enable-symbols" had removed it

Change-Id: I83aed6e21c4b983d8645707daa65bd85ec16ff6b
Reviewed-on: https://gerrit.libreoffice.org/71798
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>

* Update dictionaries from branch 'master'
  - Fix logo management for "ca"
    
    Thread 1 "soffice.bin" hit Breakpoint 1, (anonymous namespace)::GraphicProvider::queryGraphic
    (this=0x55555b446140, rMediaProperties=uno::Sequence of length 1 = {...})
        at /home/julien/lo/libreoffice/vcl/source/graphic/UnoGraphicProvider.cxx:430
    430	                SAL_WARN("svtools", "Could not create graphic: " << error);
    (gdb) p rMediaProperties
    $2 = uno::Sequence of length 1 = {{Name = "URL", Handle = 0, Value =
    uno::Any("string": "file:///home/julien/lo/libreoffice/instdir/program/../share/extensions/dict-ca/images/SC-logo.png"),
        State = com::sun::star::beans::PropertyState::PropertyState_DIRECT_VALUE}}
    (gdb) c
    Continuing.
    warn:svtools:21636:21636:vcl/source/graphic/UnoGraphicProvider.cxx:430:
    Could not create graphic: 0x8201(Error Area:Vcl Class:General Code:1)
    
    Change-Id: I4b160ccd6777988ddb0107b663006efdaa48e366
    

Change-Id: I098e894d80c2b319307bf9e9eece7a59d2ff0bd0
Reviewed-on: https://gerrit.libreoffice.org/71810
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>