summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2020-10-17sd: use the fastparser API when possibleNoel Grandin
part of the process of making SvXMLImport fastparser-only Change-Id: I34b9781ff905094b2231940bb69eb68b18faeb22 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104452 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-17Angle of the watermark: step by 45°Andras Timar
Change-Id: Ia4ab9a56f346625cf191c1f5a648e1744dad29d6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102728 Tested-by: Jenkins Reviewed-by: Andras Timar <andras.timar@collabora.com>
2020-10-16tdf#137210 fix field selection regressionJim Raykowski
restores field selection functionality that went missing after commit f51005bc969c9296fa3e64d82c13f84fdfb90fe4 Change-Id: Ifbab4c1ce272be75ced981e3d36717510684b93b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104100 Tested-by: Jenkins Reviewed-by: Jim Raykowski <raykowj@gmail.com>
2020-10-16Database Wizard: show info label instead of empty list.Gülşah Köse
In Apple Store version or in case --without-java and disable-firebird-sdbc we cannot have Java based HSQLDB or Firebird. Instead of the empty list, there should be a static text label. Change-Id: I6034a749f28a3c29c9c9e50929cb42aa1312a46b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104359 Tested-by: Jenkins Reviewed-by: Andras Timar <andras.timar@collabora.com>
2020-10-16tdf#127635: sw_uiwriter: Add unittestXisco Fauli
Change-Id: Ib25d5723057b79f49876df816bff5971ee3fa7c3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104444 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2020-10-16tdf#135733 sw: fix mistake in SwUndoInserts::SetInsertRange()Michael Stahl
First init m_nDeleteTextNodes, then check it; it is inited to 1 in the ctor so the ++m_nSttNode was skipped. This then caused bJoinNext to be true in UndoImpl() when it should be false. (regression from dc7e7b94a7211c576454267c09eb108e761e4487) Change-Id: I74038ef7f8036581dd77341dc8372e87139bdb6c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104433 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@cib.de>
2020-10-16support set_font for weld::ButtonCaolán McNamara
Change-Id: Ie56ba126bf7677e1605c0080064aa463f9277033 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104441 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16vcl pdf tokenizer: fix handling of dict -> array -> dict tokensMiklos Vajna
Needed to be able to parse the /Reference key of signatures. Change-Id: I6b81089a3f58a2de461ad92ca5a891c284f8686a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104443 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Tested-by: Jenkins
2020-10-16msfilter: Fix 0x%d debug printfsDr. David Alan Gilbert
Generally you want either %d or 0x%x, this had a few 0x%d debugs. Change-Id: Ibe55addd5efb18b17b958f96e53ca1441ae55720 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104440 Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com> Tested-by: Jenkins
2020-10-16sw reqif-xhtml import, embedded objects: handle non-package Ole10Native streamMiklos Vajna
Commit 800085d4fb0831f2065e86bfd99164cd89998fcd (sw reqif-xhtml import, embedded objects: handle Ole10Native stream, 2020-05-04) added support for handling an OLE1 stream which contained something else than OLE2 data. However, that assumed a fixed class name ("Package") and a matching class id. Improve this, so that the class id is created dynamically, based on the OLE1 class name. The class id can be figured out by putting the OLE1 data in an RTF file and converting that RTF file to DOC using Word. Change-Id: I1623a42a8f9b1278fd69641f1ae1ee467a6f0143 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104439 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Tested-by: Jenkins
2020-10-16remove thread-unsafe static bufferNoel
Change-Id: I13aa210ed6120ff9d8e541d53eceba0304ba28d9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104424 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16tdf#136924 draw lines using field text colorJan-Marek Glogowski
Interestingly, these are not icon images, as I expected, but it's a custom DrawingArea widget, which is completely drawn by some algorithm. Currently it uses the themes field color as background fill color and an independent font color for the lines. So just use the themed, matching field text color instead. Regressed-by: 39e027c669fc0a91e223d7635c10adb5778586c7 Change-Id: I9f0073f41127207d2391ee23c9553512ffe4ba01 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104422 Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de> Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-10-16unused forward declarationCaolán McNamara
Change-Id: I565f4d9627dc906db816864b5a6fc5e47ffd5ac9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104431 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16svx: use the fastparser API when possibleNoel
part of the process of making SvXMLImport fastparser-only Change-Id: I30aa8a429fabf54e39cf401419f627cc13ed719c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104427 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16sw: use the fastparser API when possibleNoel
part of the process of making SvXMLImport fastparser-only Change-Id: I240c92f8c1b06e1e17a836982d918e39719d2569 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104428 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16writerfilter: use the fastparser API when possibleNoel
part of the process of making SvXMLImport fastparser-only Change-Id: I9bb2d337954354be63984892e1fc4a7950ca3628 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104411 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16cid#1468188 dead codeCaolán McNamara
Change-Id: I5c80f0a0883e899e2bd2e1b76849772fad2ea8d5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104421 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16tdf#135260: sw_uiwriter: Add unittestXisco Fauli
Change-Id: Iec50c3129097e99ae57543601d40c5a380db678f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104391 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2020-10-16sc: use the fastparser API when possibleNoel
part of the process of making SvXMLImport fastparser-only Change-Id: Idf785f38b233e26c00d699302b7eab7e41fe817a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104413 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16add a DrawingAreaUIObject to avoid need to include layout.hxx for uitestCaolán McNamara
Change-Id: I2d7f9d38f9eac5af7b8b4d738335507beb6627df Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104357 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16WindowType::CALCINPUTLINE newly unusedCaolán McNamara
originally introduced by... commit cba0dad4566652f63ea3114660da4e5c7dd88c31 Date: Wed Jun 13 14:08:28 2012 +0200 calc input line: Fix crash with accessibility enabled. Change-Id: Iee0914b79d3b48f4479e62622fbdb5dbdf26d527 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104356 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16weld InputBarCaolán McNamara
this also restores that DnD of a selection from the inputbar is pasted as plain text not rich text formatted with the happenstance formatting of the inputbar's EditEngine Change-Id: If4934f83c14357afec2e0a7e1d51c8a1aea1d292 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104037 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16tdf#123025 - ReDim Preserve fails if array is filled by SplitAndreas Heinisch
The Split function sets the datatype of the array to SbxVARIANT instead of SbxSTRING preventing any subsequent assignments to the array and to the elements itself. Change-Id: Ib099eee7bfd222c97520ac8970352bcf2b44bc3e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104039 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2020-10-16chart2: use the fastparser API when possibleNoel
part of the process of making SvXMLImport fastparser-only Change-Id: I9fd7dbcc8b5c345f7e86bc06b1b898b7d39f36da Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104412 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16tdf#128212 Change the text z camera rotation order.Gülşah Köse
Old order: 1) Apply all rotation values (normal and camera z) 2) Do the adjustment (top, middle, bottom) New Order: 1) Apply rotation values except camera z 2) Do the adjustment (top, middle, bottom) 3) Apply camera z rotation. We rotate the text at the center of the already positioned rectangle of the text, which depends on e.g. the length of the laid out text. This matches the PowerPoint behavior. Change-Id: I8741cf443834bf01cc07c04318fd6205ed6a0dd6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103965 Tested-by: Jenkins Reviewed-by: Gülşah Köse <gulsah.kose@collabora.com>
2020-10-16pdfium: add an FPDFBitmap_Create() wrapperMiklos Vajna
Fixes a leak in PDFiumPage::getChecksum(). Change-Id: I57471a26702f1d3dd69e9e18a4067ce0e724e358 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104386 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2020-10-16Rename CLANG_CC, CLANG_CXX configuration vars (avoid clash with scan-build)Stephan Bergmann
Clang's scan-build tool uses the CLANG_CXX environment variable (setting it up in the scan-build script to pass it to the ccc-analyzer script), but happens to erroneously set it to a non-existing path (see <https://reviews.llvm.org/D89481> "[scan-build] Fix clang++ pathname again"). So wrapping LO's autogen.sh and make in scan-build picked up that broken CLANG_CXX and caused build failures like > [CXX] external/skia/source/SkMemory_malloc.cxx > /bin/sh: ~/llvm/inst/bin/clang-12++: No such file or directory (see <https://lists.freedesktop.org/archives/libreoffice/2020-October/086113.html> "Re: llvm/clang static analyzer reports"). So rename CLANG_CXX, and for consistency also CLANG_CC and the various CLANG_CXXFLAGS_INTRINSICS_*, by prefixing each with LO_. Change-Id: Ib41cabe940f8bfb1997f74e865cca5725f859e07 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104383 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-10-16fix some *printf which are using "%l" ie. long specifiersNoel
which are not portable between Linux and Windows because long is not portable. In preparation for converting long -> tools::Long Change-Id: I8bf1aa1570946ca887a6c83dd5f99c024d437336 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104374 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-16tdf#137245 sw: fix AutoFormat SetBorder deleting flysMichael Stahl
Since commit e1629c210ad78310e3d48c0756723134a27b89df ReplaceRange() will preserve flys, so split the delete into a DeleteAndJoin() just to join the paragraphs - which should not delete any flys because it doesn't include the "---" so isn't at the end of the section - and a ReplaceRange for the "---". (regression from 28b77c89dfcafae82cf2a6d85731b643ff9290e5 and e75dd1fc992f168f24d66595265a978071cdd277) Change-Id: Ib995e41649f69963c823a463538958d533082ee7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104380 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@cib.de>
2020-10-16try to prefetch also graphics for background fill bitmapLuboš Luňák
Change-Id: Ib6be487500e45ab984b7ca63d85352696d9d4051 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104132 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2020-10-16try to prefetch in threads also all graphics for the active pageLuboš Luňák
Change-Id: Ib4df3914a4f510a76f08aa2f6633bfdadf54a1e4 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104131 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2020-10-16try to make available all slide images using threadsLuboš Luňák
Graphic::makeAvailable() is not thread-safe, but the jpeg loader is capable of that, and the graphic can be loaded using the stream data (which is what ultimately makeAvailable() will do anyway). This loads all images faster using threads instead of them being loaded one by one on-demand. Change-Id: Ifc39a2757834a9fb0dbafa61f13f5454e69af330 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104082 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2020-10-16Resolves: tdf#137508 use AtkObject::accessible-roleCaolán McNamara
Change-Id: I385d16668a7145cc2f6a6574533b07281e36d37a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104382 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-16Fix another misuse of __FreeBSD_kernel__ define in LibreOfficeKitInit.h.Gleb Popov
The #if in question clearly checks for an operating system, not the kernel only. Fixes build on FreeBSD. Change-Id: Ib9452056a0d28f685fbae0b4bca65ed814cfd1cc Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104371 Tested-by: Jenkins Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-10-16Fix a confusion between UNOIDL long (i.e., sal_Int32) and C++ longStephan Bergmann
Change-Id: Id4c2645652a9588d1a9a2658217662e93c59fdbe Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104387 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-10-15tdf#137105: crash in table with Style Inspector activeAron Budea
xParaEnum->nextElement() is not an XTextRange then, it'd be an XTextTable. Since the Style Inspector doesn't support inspecting table properties, yet, simply avoid that case. Change-Id: I4ae211713433ccc30d397c04f6cbf7f1f4d1f9e7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104327 Tested-by: Jenkins Reviewed-by: Aron Budea <aron.budea@collabora.com>
2020-10-15uitest: this test should crash without the fix in placeXisco Fauli
using undo just once doesn't make it to crash Besides, minimize and anonymize the document and add some more asserts Change-Id: If83de05ee751fc54d422937c365e408b5c25df78 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104353 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2020-10-15Replace list by vector in vcl/svpframeJulien Nabet
Change-Id: I0980a46869f11998f1529c8b95d93e302cb2692c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104181 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2020-10-15tdf#137432 only do EnableInvalidate(false) optimization when not shownCaolán McNamara
Change-Id: If7f35d19061a7a2db2e5df31227834526bbf0905 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104381 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-15don't crash if not hosted in a vcl::WindowCaolán McNamara
Change-Id: I6c1e774461db5d144d09de7d5c532407f1c572b1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104373 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-15Use correct icon on additions dialogHeiko Tietze
Change-Id: If0dd70721dd5edd95a46f8735fe1d90b6843a2af Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104355 Tested-by: Jenkins Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2020-10-15pdfium: add a dict. key constant for interior colorTomaž Vajngerl
This adds the interior color key constant, so it is possible to determine if the interior color is set for an annotation (with hasKey method). Change-Id: Ic56f7d7e647f6858ede117af69269cb92dd7d1f6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104363 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2020-10-15pdfium: add reading of attachment points to the wrapperTomaž Vajngerl
Attachment points are used to define where to highlight the text. Quad points - points of the rectangle that can be rotated (typical text). Change-Id: I5438a6520195b89149c9cd1a8235a77dbc622243 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104362 Tested-by: Tomaž Vajngerl <quikee@gmail.com> Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2020-10-15pdfium: add support for border property of PDF annotationsTomaž Vajngerl
This extends PDFium with readon of "Border" property from the PDF annotations and adds the API to PDFium wrapper. Border is mostly used for border (line) width and the radius of rounding of the border (for drawing a rounded rectangle for example). Change-Id: I03f189eee03155ec699b2f56ceed23e6839a3f03 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104361 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2020-10-15pdfium: add getColor and getInteriourColor to PDFiumAnnotationTomaž Vajngerl
Interiour color is used as a fill color. Change-Id: If0c55fdaef083001abeb55e5c7ce3bae8d232713 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104360 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2020-10-15tdf#34355 sw,offapi,officecfg: show track changes in marginLászló Németh
In ShowTrackedChanges mode, hide tracked deletions in text optionally and show their shortened text in margin next to the vertical "Changed line" mark (showing only the first deletion of the line). Add "Tracked deletions in margin" checkbox to Writer->View page. Add "Display tracked changes" and "Outline View" sections to View, too. Add property ShowChangesInMargin to com::sun::star::text::ViewSettings. Add config setting: org/openoffice/Office/Writer/Content/Display/ShowChangesInMargin Show more information in Manage Changes dialog window about the tracked deletions with restricted visibility in ShowChangesInMargin mode: show shortened text content of a deletion in its (otherwise empty) Comment field. Change-Id: I1d19a4bf8225f536a635c6dd5062344c51e03b79 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104350 Tested-by: László Németh <nemeth@numbertext.org> Reviewed-by: László Németh <nemeth@numbertext.org>
2020-10-15add a GetUITestFactory for WeldEditViewCaolán McNamara
Change-Id: I9d174279163de1cd2297234441c8e0263e0a6631 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104354 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2020-10-15This field only holds a compile-time constantMike Kaganski
Change-Id: I9796f97af5d45211b67070fc020547e2b63ef411 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104348 Tested-by: Mike Kaganski <mike.kaganski@collabora.com> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2020-10-15Fix check for inclusion of <sys/sysmacros.h>Stephan Bergmann
30a5d201144cba04dd708c11d87cc5517c04c0c3 "Do not include <sys/sysmacros.h> header unconditionally, but perform a check for it in the configure script" had added an AC_DEFINE without a corresponding mention in any config_host/config_*.h.in (we use AC_CONFIG_HEADERS), so the #ifdef was always false. The #include <sys/sysmacros.h> had been added with 01bf741a79241829b0d5c048e8f45e3cf6914d3e "WaE: include needed header" for the declaration of a major function, but which is only used in #ifdef LINUX code anyway. Change-Id: I81b574c4a3e5fa2ef4e64a507b4841044217409b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104351 Tested-by: Tor Lillqvist <tml@collabora.com> Tested-by: Jenkins Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-10-15Fix checks for "older than Clang 11"Stephan Bergmann
...following up on b6d0ca04581eae01817aac18d32a312bc9a2d514 "The Clang RecursiveASTVisitor change is already in Clang 11"; no longer sure why I originally wrote the checks using <= rather than < in 5d546de67b44dec23ecfa5a6378e2968912f8253 "Adapt to Clang 12 trunk RecursiveASTVisitor change" Change-Id: I79877e21823334c939ecdf9c64e4efe5e0b1571b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104349 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>