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

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

Change-Id: I1f9ebbd77a8d7cf18facfe1783507fd9ba3b566c

lcov points out this was uncovered previously.

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

> In file included from gcc/include/c++/7.1.1/vector:69:0,
>                  from jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx:39:
> gcc/include/c++/7.1.1/bits/vector.tcc: In function ‘javaPluginError jfw_plugin_startJavaVirtualMachine(const JavaInfo*, const JavaVMOption*, sal_Int32, JavaVM**, JNIEnv**)’:
> gcc/include/c++/7.1.1/bits/vector.tcc:407:15: error: variable ‘__new_finish’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
>        pointer __new_finish(__new_start);
>                ^~~~~~~~~~~~

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

Change-Id: Idcfd542c6caede3d987e9b8219cf48114800daa5
Reviewed-on: https://gerrit.libreoffice.org/66634
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Since it is now possible to use C++14, it's time to replace
the temporary solution with the standard one

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

Move XDrawPages Java tests to C++ for ScDrawPagesObj.

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

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

Move XElementAccess Java tests to C++ for ScDrawPagesObj.

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

Move XIndexAccess Java tests to C++ for ScDrawPagesObj.

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

for FILESAVE_AUTOEXTENSION_TEMPLATE only so far (e.g. menu Insert >
Image)

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

Change-Id: I8cca2db6157996dd25afa4c95e769725188fdfb9
Signed-off-by: Gülşah Köse <gulsah.kose@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/66686Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Jenkins

* Update helpcontent2 from branch 'master'
  - Removed executable permission on image files
    
    Change-Id: I74aba60b5c77fe9acb6252c207846035a9f67473
    Reviewed-on: https://gerrit.libreoffice.org/66667
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

* Update helpcontent2 from branch 'master'
  - Add SMATH Elements in Contents Tree
    
    Change-Id: Ie2bcfa12cc85dab248063c1977638e044396e374
    Reviewed-on: https://gerrit.libreoffice.org/66697
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

* Update helpcontent2 from branch 'master'
  - Update New Features video for 6.2 help online
    
    Change-Id: Ibe32ecee310b5d844c18e3cebec534328219325f
    Reviewed-on: https://gerrit.libreoffice.org/66695
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

* Update helpcontent2 from branch 'master'
  - tdf#121065 Replace Attributes, Brackets and Format Icons with SVGs
    
    Replace Attributes, Brackets and Format Symbols PNG images with SVG
    and updated 03090800.xhp, 03090200.xhp and
    removed at21706 in Package_html_media.mk
    also removed lines with at21717, at21718, at21719, at21720
    and removed line with im21106.png in Package_html_media.mk
    
    Change-Id: If504f6a39845695aef30a61826fecf8433a326ed
    Reviewed-on: https://gerrit.libreoffice.org/66659
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

...which happens to avoid an unhelpful -Werror=strict-overflow with GCC 7 and
--enable-optimized:

> vcl/source/edit/texteng.cxx: In member function ‘bool TextEngine::CreateLines(sal_uInt32)’:
> vcl/source/edit/texteng.cxx:2197:9: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
>          if ( nTmpWidth > nXWidth )
>          ^~

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

Somewhat magically, just adding XATTR to the FlySet makes
background colors/gradients/images appear. This has been
broken since XATTR was introduced in LO 4.3.

Change-Id: I84554045f49618a5eb927edfebdf641c5a93b673
Reviewed-on: https://gerrit.libreoffice.org/66310
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>

Change-Id: Ifd472f4ca79ab97a1d6d5c5007537375121f6f58

Change-Id: Iafb5be16ebb23aa09e0300a77c2f2dbe29b85cad

Change-Id: I1b3fd47a23002673cb81811ebfd0034ca546d837

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

* Update helpcontent2 from branch 'master'
  - Link 2 Help pages on GetPathSeparator()
    
    Change-Id: I81c23ff64baff8303d2f1936c2e426bd7ba59163
    Reviewed-on: https://gerrit.libreoffice.org/66685
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

converting ooo67907-1.doc to ODT asserts in fillSoftPageBreakList()
while mapping an invalid (too large) GetOfst() of a follow frame.

There are some problems with the lcl_ModifyOfst() implementation and
usage:

1. the implementation iterates over all follows; but the follows will
   get the same SwClientNotify() call as the frame itself, so in effect
   the follows' Ofsts will be modified multiple times

2. the calls to lcl_ModifyOfst pass in a positive nLen whether the
   text is deleted or inserted; surely for deletions the Ofst should be
   subtracted from instead of added to

3. the special-casing of RES_DEL_CHR to COMPLETE_STRING while
   RES_DEL_TXT uses the length does not appear to be justified

The assert happens since b15f1ed2
but the lcl_ModifyOfst logic never made sense.

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

Change-Id: I8688c65695080804bae22209ed5d4890ced1a8dc
Reviewed-on: https://gerrit.libreoffice.org/66678
Tested-by: Jenkins
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>

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

...in 113536e9 "Avoid
-Werror=format-{overflow,truncation}="

Change-Id: Ie26abed8c5694a3b26777771737ded2b3b484935

Selection clipboard now should properly get
and set data from/to selection buffer
and not from/to copy buffer.

Related: tdf#122239

Change-Id: Ia5298932ee277385c6f8a587d4fcb8204647c491
Reviewed-on: https://gerrit.libreoffice.org/66577
Tested-by: Jenkins
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>

During export of the TOC into DOCX all TOC field parts
should be placed inside one STD frame: field start-begin,
field-separate and field-end mark.

Updated unit test.

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

Finally the bugdoc rendering result is reasonable and even looks like a
tree as it should.

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

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

rather than searching pointer each time

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

Change-Id: I823a809048905a93f1413e3332dcb5fe18748a4a

Change-Id: Ia46b136f64c2750ed466d86967de9dfe90b4ce65
Reviewed-on: https://gerrit.libreoffice.org/65003
Tested-by: Jenkins
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>

error: base class ‘class salhelper::SimpleReferenceObject’ should be
explicitly initialized in the copy constructor

after
    commit 3b16e997
    Date:   Fri Jan 18 15:38:07 2019 +0200
    cleanup SwAuthEntry manual ref-counting

Change-Id: I291a8a4f6c2a6ec38506265f6a0382f89774ecd4

Change-Id: I021d63c9d57f1e0435bcc5f97abc57bc39fece01

Change-Id: I1492c604ac39ce72fe2cfe8804ac15e8c41b9a46

Does nothing. Needed for customer application to proceed. Once we are
further along in getting it to work, we can investigate what the
parameters passed to this ToolsOptionsView method actually are. (This
WordBasic thing is something that has been deprecated since last
century, I suspect, so no wonder it is hard to find authoritative
documentation on it.)

Change-Id: I62a6d6d9abb9364afca110570fa341a2375a77a6

Change-Id: Ia577174fac926295db439349f6caca178b363ba0

End-user VB6 applications might not like the exception. Better to just
ignore.

Change-Id: I42718de7855079e38c7983db39180c184ab7224f