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

At least during JunitTest_sw_unoapi_4
sw.SwXTextTable::com::sun::star::text::TextTable::TableColumnSeparators, some
SwFormatFrameSize is created with a width of USHRT_MAX ("Set USHRT_MAX as the
Table's default SSize") in SwDoc::InsertTable
(sw/source/core/docnode/ndtbl.cxx).  In SwTabFrame::Format
(sw/source/core/layout/tabfrm.cxx) that leads to nWishedTableWidth with a value
near USHRT_MAX and substantially larger than nMax, causing nLeftSpacing to be a
large negative value in the text::HoriOrientation::CENTER case.  That in turn
causes SwTextAdjuster::CalcRightMargin (sw/source/core/text/itradj.cxx) to
set

    pRight->PrtWidth( sal_uInt16( nRealWidth - nPrtWidth ) );

with a value of 32768 (where pRight is a SwGluePortion).  And a later call to
that SwGluePortion's GetPrtGlue would try to convert that 32768 to 'short',
getting flagged by Clang's -fsanitize=implicit-signed-integer-truncation.

In addition to changing GetPrtGlue (and MoveGlue) to use 'long' instead of
'short', drop some unncessary narrowing casts from client code.

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

after
    commit b792e4ee
    sal_uLong->sal_uInt32 in drawEPS

Change-Id: Ic35c165cd9dd756e229c33f643355150d6249688

The reason was that A2 had an explicit paragraph style reference, but A1
did not, so table buffering caused A2 style to affect A1 style as well.

Combine this with style deduplication, and then A2 style considered the
direct formatting (font size) in A1 to be redundant, so it was lost on
import.

Fix the problem by moving the copy&pasted properties buffering to a
single function, and there buffering not only the properties, but also
the active style index.

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

sal_uInt32 seems reasonable given that this is the number of rectanges
in an image.

And then convert all of the other BeginSetClipRegion methods to use
sal_uInt32 too.

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

This way bottom docked toolbars appear above each other
to avoid Table pushing Track Changes out from under the mouse cursor

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

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

because the call site specifies millis in a sal_uInt64 variable

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

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

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

This patch allows the whole interactive sequence to
restart automatically when it's done. User don't
need to insert multiple pause toggle command to
a media object just for toggling the media
multiple times.

Change-Id: I000a55f580917327ae438ea8e79e62f63275cce7
Reviewed-on: https://gerrit.libreoffice.org/68283
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

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

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

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

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

Change-Id: I0350bfbe32afc68f94e0fd10c379ac39f2b7c2ca
Reviewed-on: https://gerrit.libreoffice.org/68353
Tested-by: Jenkins
Reviewed-by: Zdenek Crhonek <zcrhonek@gmail.com>

The aspect ratio request of the Shape is not yet used in
AlgAtom::layoutShape(), though.

The heavy-lifting is needed, because the number of cols/rows in the
snake algorithm depends on the aspect ratio request from the child
algorithm, so need to transfer the aspect ratio from child algorithm ->
layout node -> shape -> parent algorithm.

Still no functional changes intended.

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

(And probably the same holds for Android, too.)

This change makes at least the Format>Character... and
Format>Paragraph... dialogs in the iOS app show up. For some reason
they are claimed to be modeless, and that meant that the mechanism for
dialog display in the iOS app did not work. The exact mecahnism is
unclear to me, but the alleged modelessness was the root cause.

Change-Id: I237a5e16ec34221912a2fb062068101562ba7dbb

* Update helpcontent2 from branch 'master'
  - Mutual references (links) in Session & Locations help pages for Python
    
    Change-Id: I96c55ca44c1b04598fcdd663a394c68211f4a29d
    Reviewed-on: https://gerrit.libreoffice.org/68310
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

* Update helpcontent2 from branch 'master'
  - tdf#123192 - Improve help for function ATAN2
    
    updated text in ATAN2 section of 04060106.xhp
    added coordinates-to-polar-01.svg
    changed example and image
    removed 'Important!', added 'radians'
    changed 'LibreOffice' to '%PRODUCTNAME'
    created new folder 'scalc' under 'media/helpimg/'
    moved coordinates-to-polar-01.svg to new folder
    and added the path/image name to Package_html_media.mk
    droped xml-lang="en-US" tag, removed 'Note',
    removed '<pre>' tag, did not add <embed> as
    Jenkins rejects it.
    
    Change-Id: I5cff6289e8ab25f511e7d31ba2d4ad733dea184a
    Reviewed-on: https://gerrit.libreoffice.org/67619
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

In favor of std::make_unique

Change-Id: I0428076a10fb7b61c5add994c9970661b375b82c
Reviewed-on: https://gerrit.libreoffice.org/68254
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

Reorder the order the filelists are initialized per package.
Optimize speed of find_files_for_package:
  We can simply check if the modules entry is the same as last time.
  The code uses iterators instead of that C-ish for loops.
  Create a hash from the packagemodules to speed up the check if
  a module is included in the hash.

Change-Id: I1e8ae394634a527880e08ef8ff333d94d04f49fd
Before: 12s per language/package
After: <1s per language/package
Reviewed-on: https://gerrit.libreoffice.org/68339
Tested-by: Jenkins
Reviewed-by: Tomáš Chvátal <tchvatal@suse.cz>

No functional changes intended.

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

Change-Id: I17fa4d503d197b9b7c7405281001ea92714494a9

In order to get the Impress-specific dialogs in the (new,
Online-based) iOS and Android apps, too.

Change-Id: I7400326bafc1b8a0470b5ed0fb631ae984b7b795

Change-Id: Ife76336dbdeaadf08f631e3542be73a8568f5368

We will call it just a few lines later.

Change-Id: I967ff4eec2cfb1dcc7e8b02f32232be65b6366db

Change-Id: I083583b86fa404eb16eea354ef9e6d22cf4c62fe

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

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

Change-Id: I58d25348f4b43145a19ca0edbc5cde2e9a57f468
Reviewed-on: https://gerrit.libreoffice.org/68209
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

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

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

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

* Update helpcontent2 from branch 'master'
  - APSO incorporates Python debugging facilities.
    
    Change-Id: Ia01579acd089d99a99630737aa1d77bcc842af52
    Reviewed-on: https://gerrit.libreoffice.org/68308
    Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    

* Update helpcontent2 from branch 'master'
  - Propagate xsltproc failure, part 2
    
    ...following up on c3509fb7e6675a38560c7e2764b21861d1e23362 "Propagate xsltproc
    failure".  The while loops would only have a non-zero exit code if the xsltproc
    invocation in the last iteration of the loop fails.  (Oddly, adding `set -e`
    instead seems to have no effect.)
    
    Change-Id: I9296a59147697595ebe7b0d2b742044f8a8a3f02
    Reviewed-on: https://gerrit.libreoffice.org/68309
    Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
    

Change-Id: Ief089d6d2a43b4fbd18519cb2c8ab1d867c47118

Handle timenode condition properly. Begin and End of
TimenodeContainer allow Sequence, Event, Timing, double,
and VOID. Only events with Source set are valid.

Change-Id: I25519110715072ca36b7f5c55c26415f9fbe9deb
Reviewed-on: https://gerrit.libreoffice.org/68210
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: Iabf2dd2daf9d079e7588a1175a2d1e763ccc1c64

It removes the HACK where the doc model was changed.
We introduce a pointer to a surrounding frame: an indicator
telling us that the table was placed in a frame to make it
floating.

During export, remove the frame and use the relative width of the
surrounding frame.

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