Commit graph

478023 commits

Author SHA1 Message Date
Christian Lohmaier
fcfb30e81a windows packaging: remove unused msp creation
Change-Id: I71903967f89d968d963f66a3e160d6dead7757f2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138011
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
2022-08-18 16:07:43 +02:00
Christian Lohmaier
de3fcf4bea crashreport: __cpuid/__cpuidex are not available on arm64
…and from a quick glance similar features are not exposed to userland
applications on arm64:

https://docs.kernel.org/arm64/cpu-feature-registers.html
"The ARM architecture defines a set of feature registers, which describe
the capabilities of the CPU/system. Access to these system registers is
restricted from EL0 and there is no reliable way for an application to
extract this information to make better decisions at runtime.[…]"

Thus the CPU information would have to be gathered from the OS, probably
from the Windows registry, at least it should be possible to read the
name/model that way…

Change-Id: I39e3679a7c4c581d8e3e4f71842d7ea7cdc2ba67
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138380
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
2022-08-18 16:06:46 +02:00
Caolán McNamara
64c60a6793 cid#1509305 Dereference null return value
Change-Id: I17175b9ce05b784844e49aed53d85a55e33a0f45
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138484
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 16:06:33 +02:00
Caolán McNamara
b95038cd67 cid#1509313 Unchecked dynamic_cast
we can get the ScDocument another way

Change-Id: Ie1953b911d34a647fbac1923bc9166f67f346a69
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138483
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 16:06:19 +02:00
Caolán McNamara
f6e24b10c9 cid#1500536 silence Resource leak
Change-Id: Ibd626a90136f89af731dc2bfa684d99609d44b90
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138481
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 16:05:53 +02:00
Caolán McNamara
1b3d662f29 cid#1500387 alternative silence Resource leak
This reverts commit d40bfecddc for
a different approach

Change-Id: I8e058efbff214456b91346ce16e7eba7a89a8a53
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138480
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 16:05:39 +02:00
Caolán McNamara
6105706deb cid#1500586 Resource leak
Change-Id: I32d837af7574f6eb4ac7db4759139d0b78f63e75
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138479
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 16:05:26 +02:00
Justin Luth
1ad2d1336a related tdf#150197 doc import: ANLV - always SetListFormat
Any time there is numbering, we expect to see an
optional sListFormat set.

This solves a few different cases found by:
assert(false && "depricated format still exists
and is unhandled. Inform Vasily or Justin");

Change-Id: I127e100a003c46c7a9dfa681014ffc11ceda24ac
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138452
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
2022-08-18 15:34:07 +02:00
Caolán McNamara
6e1be9f852 cid#1500645 silence Resource leak
Change-Id: I1548017a39f5080dce0731169811fcb0a059c4d9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138478
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 15:10:02 +02:00
Caolán McNamara
05dba1a123 cid#1509312 Unintentional integer overflow
Change-Id: I93ca395f6a74dd6c67d4388512b5de3441b362fd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138476
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 15:08:51 +02:00
Caolán McNamara
9446efdf58 cid#1500682 silence Untrusted loop bound
Change-Id: I44ae78fe7427d2647e3efe2688be8c124c16743c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138475
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 14:29:19 +02:00
Caolán McNamara
292fd4d83b cid#1509254 Uninitialized scalar variable
Change-Id: I2d1f0944a069230a435890734eb18e8ee3e275e9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138470
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 14:28:50 +02:00
Olivier Hallot
ef40d3be2f Update git submodules
* Update helpcontent2 from branch 'master'
  to 21c64d4e09d23bf8933f8d513a857d4df3029cfa
  - Update LO7.4 new features video
    
    Change-Id: Ic1290ef2d90aca04f1bf6df1aae5d19f64f5919f
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138486
    Tested-by: Jenkins
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-08-18 14:03:07 +02:00
Olivier Hallot
7d5c4a1777 Update git submodules
* Update helpcontent2 from branch 'master'
  to abecf1a85a2dded5a59c543f275df51d79f649bc
  - tdf#150445 Change Gear icon to Hamburger icon
    
    Change-Id: I7abe1c0d84550baad982d694922119a1af827bd0
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138377
    Tested-by: Jenkins
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-08-18 14:01:03 +02:00
Rafael Lima
410bff99a7 tdf#149768 Do not fix Templates dialog width and height in pixels
This patch removes the fixed size of the Templates dialog in pixels so it works better in HiDPI displays.

This patch also makes the dialog a bit wider and non-resizable.

Change-Id: I256d8d5c8b76c8e32a4f008f3a235f80bf59634d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136922
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
Tested-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
2022-08-18 13:47:14 +02:00
Noel Grandin
010e2a574c loplugin:moveit
Change-Id: I34de7408553e4ca702cab9aa611c03dc60b9b6a1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138472
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18 13:44:43 +02:00
Tünde Tóth
e59db22b3b tdf#150434: chart2, XLSX import: strip long legend labels
Full text of legend labels could overflow the chart area,
if the legend text was too long. If it's longer than
520 characters, strip it at the first space from the
500th character (or if there is no space, at the 500th
character). This results better XLSX interoperability, too.

Change-Id: I23a94f6baaf620b40e9b2819738eba5c5a921722
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137492
Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>
2022-08-18 13:39:31 +02:00
Noel Grandin
326534372e no need to allocate these on the heap
Change-Id: Ia5eec87a4b36ac2dd15c3369bd7c630277a177c0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138473
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18 13:19:35 +02:00
Noel Grandin
a6535669f9 use more pass-by-value in SfxItemSet
which avoids some copying

Change-Id: Id774c8947d0c2676425a1a192e0ac28c8f8402db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138468
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18 12:01:00 +02:00
László Németh
f0454fe3bb tdf#150394 sw: fix crash of "Don't hyphenate last word"
Multiline last words, e.g. long URLs resulted crashing
using the new hyphenation option "Don't hyphenate last word".

Follow-up to commit 8c018910ae
"tdf#149248 sw offapi xmloff: add option to not hyphenate last word".

Note: .fodt format is not applicable for unit testing.

Change-Id: I8633af1517f09003b40a06825ad14f3ed7f882a5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138296
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
2022-08-18 11:47:15 +02:00
Caolán McNamara
30d454e149 cid#1509397 Uninitialized scalar field
Change-Id: I250a900298f455ce594bf7dd99acbe984efc5ada
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138466
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 11:38:53 +02:00
Caolán McNamara
67224d9ea0 cid#1500394 Explicit null dereferenced
Change-Id: Ie97b047b793c0d9546a17f9e9f1d20cbeed62626
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138469
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 11:38:36 +02:00
Stephan Bergmann
b2760b7d2a external/epoxy: Fix -Wint-conversion
...with recent Clang 16 trunk since
<7068aa9841>
"Strengthen -Wint-conversion to default to an error", causing

> workdir/UnpackedTarball/epoxy/src/gl_generated_dispatch.c:114547:77: error: incompatible integer to pointer conversion passing 'uintptr_t' (aka 'unsigned long') to parameter of type 'GLhandleARB' (aka 'void *') [-Wint-conversion]
> GEN_THUNKS(glAttachObjectARB, (GLhandleARB containerObj, GLhandleARB obj), ((uintptr_t)containerObj, (uintptr_t)obj))
>                                                                             ^~~~~~~~~~~~~~~~~~~~~~~
> workdir/UnpackedTarball/epoxy/src/dispatch_common.h:150:40: note: expanded from macro 'GEN_THUNKS'
>     GEN_GLOBAL_REWRITE_PTR(name, args, passthrough)                  \
>                                        ^~~~~~~~~~~
> workdir/UnpackedTarball/epoxy/src/dispatch_common.h:95:14: note: expanded from macro 'GEN_GLOBAL_REWRITE_PTR'
>         name passthrough;                                        \
>              ^~~~~~~~~~~

etc.

That only hit on macOS because of the different

> #ifdef __APPLE__
> typedef void *GLhandleARB;
> #else
> typedef unsigned int GLhandleARB;
> #endif

in OpenGL's glext.h, which
<0cfb0a044b>
"Fix most GLhandleARB warnings on OS X with a big comment in our code" had tried
to address by adding "a cast to uintptr_t to shut up the compiler" in its
src/gen_dispatch.py (but without stating what compiler diagnostics exactly were
supposed to be silenced by that cast that now started to cause the above issues
with Clang 16 trunk).

It turns out that at least my macOS build with Clang 16 trunk builds fine
without that cast, so just drop it for good.  (And patching epoxy's
src/gen_dispatch.py means that UnpackedTarball_epoxy needs to switch from
gb_UnpackedTarball_set_pre_action to gb_UnpackedTarball_set_post_action.)

Change-Id: I2c87bca2cc5510d17098028d4532989f48e349a9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138407
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-08-18 11:37:00 +02:00
Xisco Fauli
5db76fd0ec tdf#148671: sw_ooxmlexport16: Add unittest
Change-Id: I634a1f31c93c9186be6aea35f4a020cceed6dd9c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138459
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-08-18 11:12:20 +02:00
Caolán McNamara
1286014bbf cid#1485150 Uncaught exception
Change-Id: I793683bfb59ff6b8751baa14a999b699cc28fdc9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138467
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18 11:00:41 +02:00
Liu Hao
bef0cfcf2a tdf#39593 Ruduce copy/paste code in Converter::converterDuration
Change-Id: Ic1b82b244ff614c8d5ab44f32bdc728deee24165
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138465
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2022-08-18 10:55:59 +02:00
Andrea Gelmini
712901b388 Fix typo
Change-Id: Ie0757b332f4ddb0e9d9d6e2c97b7497766221dec
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138310
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18 09:59:50 +02:00
Andrea Gelmini
1bcb84a391 Fix typo
Change-Id: I6daee630eb433998711091780452dc0534398f79
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138460
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-08-18 08:48:57 +02:00
Tomaž Vajngerl
9923f2416b chart2: improve keys (symbols) rendering in a data table
- better symbol positioning taking symbol height and cell height
  into account.
- symbol height depends on font size (as with the legend)
- take symbol aspect ratio into account
- remove hardcoded values

Change-Id: I3537d82538035cd31b928d4bb6fc5572914fcd13
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138463
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18 08:48:34 +02:00
Tomaž Vajngerl
e425ee936e chart2: convert the fill color only when style is "solid"
The CharBackColor should only be set when the fill style has the
value "FillStyle_SOLID", otherwise don't set it as we can't handle
any other fill style.

Change-Id: Icaec8996d47966bb32bd8cf88b8c5aad70ab07a8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138462
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18 08:48:17 +02:00
Tomaž Vajngerl
3115a9f3a9 oox: set fill properties correctly for the data table at import
Change-Id: Ia108c5b67e85ba770bd84d90ad1d3dd27046b245
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138461
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18 08:48:05 +02:00
Tomaž Vajngerl
66faff4d3c chart2: use only one "Data Table" string to simplify translation
Change-Id: Idff337523a66195277b364be474baffcebca26aa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138369
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18 08:47:53 +02:00
Noel Grandin
1085810635 convert more nNode to SwPosition::GetNode
in places where we are using the result with operator<

Change-Id: I3440d1204e69e30f21eaf833c2d0c42cde59e55c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138454
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18 08:47:39 +02:00
Miklos Vajna
ca05ad4c8d CppunitTest_sw_ooxmlexport15: use more SwParaPortion::dumpAsXml()
See commit feeed3e762
(CppunitTest_sw_layoutwriter: avoid some a11y-based layout testing,
2022-04-07) for motivation.

Change-Id: I5b4e566139e37be0f8f106521600127ef72e20da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138453
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-08-18 08:07:28 +02:00
Mike Kaganski
ca8760a836 Make ElementsDeck the default deck for Math first start
Change-Id: I471e268b6bb55d0cac9faed1b6631724b08b34a7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138457
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2022-08-18 08:03:45 +02:00
Mike Kaganski
29f230a633 Show Math sidebar by default
Change-Id: Id807d0c39f78a3a4be2e5eb31c75b31b1790f71e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138456
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2022-08-18 08:03:34 +02:00
Xisco Fauli
8475b36729 tdf#150452: sc_subsequent_filters: Add unittest
Change-Id: Idf7727f71fb27d6316c360bf6d3671c03a3e91cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138449
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-08-17 22:05:55 +02:00
Noel Grandin
dcd45e2540 convert more nNode to SwPosition::GetNode
in places where we are using the result with operator<

Change-Id: I2aae14e9130efc31f5afd3450defdf6df4099950
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138403
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-17 20:52:58 +02:00
Caolán McNamara
7e8be84f54 fix SfxBindings::Invalidate(const sal_uInt16 *): Assertion
assert `*pIds > *(pIds-1)' failed.

seen in impress, view, outline, press tab

Change-Id: Ie5b61aac614789f13e59c49ab390e07a7a0a55a4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138447
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-17 20:36:00 +02:00
Noel Grandin
b376181053 use more SwPosition::Assign
Change-Id: I2a073fc8350aa85e44e57b587450e012f411b47f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138402
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-17 18:19:16 +02:00
Noel Grandin
3a02b5f8aa convert more nNode to SwPosition::GetNode
in places where we are using the result with operator==

Change-Id: I28f8f6fa2c3754ec6612ab2334c3a58b61ecd065
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138401
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-17 18:18:52 +02:00
László Németh
a18a74d676 tdf#150200 tdf#150438 sw, DOCX: fix drop cap dash, quotation etc.
In drop cap layout, set smaller size for all glyphs positioned
over the baseline, e.g. dashes (dash, en-dash, em-dash, figure dash),
bullet, asterisks and quotation marks by extending the bounding box
of the glyph to the baseline, like MSO does.

Add "DropCapPunctuation", a new default compatibility option for this.
Only old ODT files loads the old layout (which was partially broken:
e.g. dashes were too long, often missing from the drop cap area or
the drop cap was disabled). New ODT and imported DOCX documents
use the new default layout for better typesetting and interoperability.

Change-Id: I3aba0727fd15f6edb9245e31f523e12f407d189e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138356
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
2022-08-17 18:07:24 +02:00
Stephan Bergmann
9135cdbc1a Missing test dependency
> warn:vcl.builder:232474:232474:vcl/source/window/builder.cxx:480: DBG_UNHANDLED_EXCEPTION in VclBuilder
>     when: Unable to read .ui file exception: com.sun.star.container.NoSuchElementException message: "file:///.../instdir/share/config/soffice.cfg/dbaccess/ui/appborderwindow.ui at xmlreader/source/xmlreader.cxx:67"
> warn:dbaccess:232474:232474:dbaccess/source/filter/xml/dbloader2.cxx:460: DBG_UNHANDLED_EXCEPTION in load exception: com.sun.star.container.NoSuchElementException message: "file:///.../instdir/share/config/soffice.cfg/dbaccess/ui/appborderwindow.ui at xmlreader/source/xmlreader.cxx:67"
> unotest/source/cpp/macros_test.cxx:73:Tdf126268Test::testNumbers
> assertion failed
> - Expression: xComponent.is()
> - loading failed: file:///tmp/Tdf126268Test__testNumbers_2f43ug.tmp

Change-Id: Ic8b530418319b56b3da824ebb66fc3bef6b7809e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138444
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-08-17 17:35:05 +02:00
Xisco Fauli
938c899831 tdf#150452: Revert "tdf#130795 use concurrent hashmap in SharedStringPool"
This commit reverts 3749d9af37
which removes the dependency on the external library cuckoo

Without using cuckoo the same file in tdf#130795 takes

real	0m4,892s
user	0m5,298s
sys	0m0,449s

With it, it takes

real	0m4,914s
user	0m5,276s
sys	0m0,444s

pretty much the same time

Change-Id: I4cc9000ac5bf26de22bb9835283ae8d5b3230196
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138435
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-08-17 16:59:14 +02:00
Rafael Lima
4d78ed5c00 Update SF translation to Portuguese
Change-Id: I96ae7fabb65e44e6d70f0c4048609f93aed8be1a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137722
Tested-by: Jenkins
Tested-by: Jean-Pierre Ledure <jp@ledure.be>
Reviewed-by: Rafael Lima <rafael.palma.lima@gmail.com>
2022-08-17 15:21:17 +02:00
Noel Grandin
94e8f2e0cd Revert "workaround clang-16 complaint in skia"
This reverts commit 7548748e34.

No longer necessary after doing a fresh pull of llvm, so llvm
is at commit 4c244b2833bdc51bbb7970ea6433e844dfddf622

Change-Id: I6dc721b47a905f553b227db3a036899173b35bae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138408
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-17 14:15:30 +02:00
Michael Stahl
7d730cd580 tdf#147466 sw_redlinehide: avoid crashing by delete un-deleting
After the first delete, there is a single redline from 170,0 to 177,9,
which contains a section.

[ 169]  0x6133358            TextNode "b) 9 (Robin Cook, Jack Straw, Margaret Beckett, David Milliband, William Hague, Philip Hammond, Boris Johnson, Jeremy Hunt, Dominic Raab)",
[ 170]  0x61349d8            TextNode "Hangul script is used to write which language?",
[ 171]  0x61396f0         SectionNode ,
[ 172]   0x6138cc8           TextNode "Japanese",
[ 173]   0x613ab58           TextNode "Vietnamese",
[ 174]   0x613ad78           TextNode "Korean",
[ 175]   0x613bc28           TextNode "Chinese",
[ 176]  0x24a61c0             EndNode ,
[ 177]  0x6139568            TextNode "c) Korean",

Then the next delete is from 169,137 to 170,0 and it is combined in
AppendRedline() with the existing one, removing the existing one.

Now the code for tdf#119571 at the end of AppendRedline() that splits up
the one redline into 4 redlines, with the section start node and section
end node not covered by any of them.

AppendRedline() of a delete redline causing previously deleted nodes to
become un-deleted is a situation that UpdateFramesForAddDeleteRedline()
doesn't expect.

Additionally there seems to be an issue with CheckParaRedlineMerge()
setting the Section's m_bHiddenFlag, which was unintentional, it should
only set m_eMerge flag, but fixing that isn't sufficient.

Reportedly this crashes since commit
6433dc223f but the problem appears older.

Change-Id: Ic83a93d5aaec4ee562fb960693b52bd7b25cb1a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138411
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2022-08-17 14:05:15 +02:00
Michael Stahl
477e489e71 forcepoint#108 sw: tweak a bit
* the fly-at-fly case didn't work, iteration also skips it
* delete the bad redline section in the nodes array

Change-Id: I21d3635fb53068e79984b0ea74a0a0913513bd51
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138434
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2022-08-17 14:04:42 +02:00
Jean-Pierre Ledure
9f4405c94b ScriptForge - (Calc) redesign CompactUp() and CompactLeft()
This commit is mainly about the introduction of
a new internal method:
   _ComputeFilter(range, filterformula, filterscope)

A FilterFormula is a Calc formula that returns TRUE or FALSE
   The formula is expressed in terms of
   - the top-left cell of the range when FilterScope = "CELL"
   - the topmost row of the range when FilterScope = "ROW"
   - the leftmost column of the range when FilterScope = "COLUMN"
   After pasting, the relative and absolute references
   will be interpreted correctly.
The FilterScope indicates the way the formula is applied,
once by row, column or individual cell.

The concept of FilterFormula was already used by CompactUp()
and CompactLeft(). Their implicit (Filter)scopes were resp.
"ROW" and "COLUMN".

The _ComputeFilter() method returns an array of subranges
contained in the initial range that match the filter.

The isolation the code for the management of filters
applied on ranges makes the later use of the concepts of
FilterFormula and FilterScope reusable for other methods.

CompactUp() and CompactLeft() are functionally unchanged.
No impact on documentation.

Change-Id: I7c4e890b54f315486f29b5434a3c236167e2f9ea
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138368
Tested-by: Jean-Pierre Ledure <jp@ledure.be>
Reviewed-by: Jean-Pierre Ledure <jp@ledure.be>
Tested-by: Jenkins
2022-08-17 14:01:07 +02:00
Mike Kaganski
1f088bb8dd Move application check to the point where it matters
This simplifies and deduplicates call sites, and also makes sure that
we don't lookup strings until we need that. Also helps adding a check
for Math in a follow-up.

Change-Id: I6b073253f19afbb9a95f29806ab306c6a9f63f18
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138413
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2022-08-17 13:02:33 +02:00