summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2019-07-23RTF import: add tests for RTF_TLMDOT/HYPH/TH/EQMiklos Vajna
lcov points out these were not covered previously. Also, Word maps TH to underline and EQ to nothing, so do the same. Change-Id: I994f78cbe1c6c2edec73edc8944f739e2a7cb8d8 Reviewed-on: https://gerrit.libreoffice.org/76144 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2019-07-23cid#1448493 Dereference after null checkCaolán McNamara
Change-Id: Iafd8c2a80d9abab419ada995f18221212434df54 Reviewed-on: https://gerrit.libreoffice.org/76136 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23async modal dialogs need to inc/dec parent frame modalityCaolán McNamara
like sync ones do, so vcl knows those frames are modal-locked, so e.g. a blank calc document with a modal async dialog open, e.g. page dialog, is not considered a candidate for reuse if soffice is instructed, e.g. via command line pipe, to open another calc document Change-Id: Id24c2ca5277e5a0e379a3e4f718514ca5e9feb03 Reviewed-on: https://gerrit.libreoffice.org/76145 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23Improve some tip_of_the_day stringsRoman Kuznetsov
Change-Id: I5177b41aa3be91f1b5003e49a729757405eae4d5 Reviewed-on: https://gerrit.libreoffice.org/75184 Tested-by: Jenkins Reviewed-by: Roman Kuznetsov <antilibreoffice@gmail.com>
2019-07-23Resolves: tdf#126460 implement reading grayscale+alpha tiff formatCaolán McNamara
Change-Id: I3300ae21c74f5a25c767ce643e93d2232f3b9381 Reviewed-on: https://gerrit.libreoffice.org/76123 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23tdf#126340 sw_redlinehide: only move merged frames in JoinPrevMichael Stahl
The problem was that the first and second node were not actually merged by a redline, so moving the frames doesn't make sense. (regression from 41d8ca9686c7c184f586e99674b443c34bfd4f33) Change-Id: Ib401e4b0b2b207666f65c038ab5c346807bfea92 Reviewed-on: https://gerrit.libreoffice.org/76125 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
2019-07-23cid#1448446 Unchecked dynamic_castCaolán McNamara
Change-Id: I888ca249e0e9551c74611dcfb8ba7c7c1dc36880 Reviewed-on: https://gerrit.libreoffice.org/76133 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23Fix typoAndrea Gelmini
Uhm... maybe missed traslantion from German? Anyway, this just to spot it to smarter ones Change-Id: Icba3458d9acf9d123a73973c749c94aebd518881 Reviewed-on: https://gerrit.libreoffice.org/75258 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Jenkins
2019-07-23cid#1448376 SIC: Inner class could be made staticCaolán McNamara
Change-Id: Id9923031c1fe91ba71dfe8d68cbe23b72e9637b5 Reviewed-on: https://gerrit.libreoffice.org/76143 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448466 RV: Bad use of return valueCaolán McNamara
Change-Id: Ied76faa2a2745f16d67484d9a7f587081379f3c7 Reviewed-on: https://gerrit.libreoffice.org/76137 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448370 Explicit null dereferencedCaolán McNamara
Change-Id: I5c0744c4e348f50a22f9c687659310f6c75178be Reviewed-on: https://gerrit.libreoffice.org/76129 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448373 UC: Useless codeCaolán McNamara
Change-Id: Ie7555620655b890b9790300e2f19c40b3b55d381 Reviewed-on: https://gerrit.libreoffice.org/76138 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448398 Dereference before null checkCaolán McNamara
Change-Id: I55bddd8f152f34919e9818048aaf2a77a94ccaf0 Reviewed-on: https://gerrit.libreoffice.org/76130 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448404 Dereference null return valueCaolán McNamara
Change-Id: I8701430be9282e2120678dc7b55f82c4efeef1ae Reviewed-on: https://gerrit.libreoffice.org/76131 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448364 UC: Useless codeCaolán McNamara
Change-Id: I583b4d947b18147bcfac7c518002c2bd676e0bb3 Reviewed-on: https://gerrit.libreoffice.org/76139 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448428 Dereference before null checkCaolán McNamara
Change-Id: Ied813af9926f405aef36dc3b8b450f32f9106d9c Reviewed-on: https://gerrit.libreoffice.org/76132 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448463 Dereference null return valueCaolán McNamara
Change-Id: I61285b8e34f58bbaaedb5fd67d181aee7e1e591d Reviewed-on: https://gerrit.libreoffice.org/76135 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23cid#1448354 UC: Useless codeCaolán McNamara
Change-Id: Ic4bac0378ee71550d5581a2c8e4c1cb5108b0a58 Reviewed-on: https://gerrit.libreoffice.org/76140 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23tdf#122774 Make all ScCheckListMenuWindow items keyboard accessibleJim Raykowski
This patch provides keyboard tab navigation to the menu window menu items. In addition to the Ok button being already disabled when search is empty the patch disables the check list box, toggle all check box select, single button and unselect single button. This prevents keyboard focus from being trapped in the check list box and from tab navigation visting useless controls when search is empty. This patch also provides a hack fix for tdf#122772 Change-Id: I7677d544ae63acc8eca08877ecd001d394fcfaca Reviewed-on: https://gerrit.libreoffice.org/75684 Tested-by: Jenkins Reviewed-by: Jim Raykowski <raykowj@gmail.com>
2019-07-23loplugin:unusedfieldsNoel Grandin
Change-Id: If1e0b32f2cce7678f454009c0180d7612b4fb7c6 Reviewed-on: https://gerrit.libreoffice.org/76119 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-07-23using an algorithm of the STLAdrien Ollier
makes code clearer Change-Id: I0bf26b6a75354ccbb79d431c947277d621985385 Signed-off-by: Adrien Ollier <adr.ollier@hotmail.fr> Reviewed-on: https://gerrit.libreoffice.org/76127 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-07-23close some more holes in structuresNoel Grandin
and improve the pahole script so I can just run it once over the whole codebase Change-Id: I7e1775974a3a61f8c0e40646158f01163ace60cc Reviewed-on: https://gerrit.libreoffice.org/76122 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-07-23loplugin:unusedmethodsNoel Grandin
Change-Id: I3bde62d674f5beaa6d19dcd22e6be15dfefe4205 Reviewed-on: https://gerrit.libreoffice.org/76118 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-07-23dispose() methods should clear their smart pointersNoel Grandin
especiall the ref-counted ones Change-Id: Ib3bb029043b1b923010ef4a47bfc377e1f569da7 Reviewed-on: https://gerrit.libreoffice.org/76102 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-07-22cid#1448459 Dereference after null checkCaolán McNamara
Change-Id: I0a36171d00c6b72807744db16335a624219c2081 Reviewed-on: https://gerrit.libreoffice.org/76134 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448328 UC: Useless codeCaolán McNamara
Change-Id: Ie6a64927bff7536ce34cc802341e13046d655ab4 Reviewed-on: https://gerrit.libreoffice.org/76141 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448291 UC: Useless codeCaolán McNamara
Change-Id: Idba951ec37529438fb23089feebfb192716c9c76 Reviewed-on: https://gerrit.libreoffice.org/76142 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1451627 Division or modulo by float zeroCaolán McNamara
Change-Id: Ifb4b446edb3da3096230e99a898e2fa8c4e9d66c Reviewed-on: https://gerrit.libreoffice.org/76098 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448262 Dereference before null checkCaolán McNamara
Change-Id: I998e1c463d86aeb53b93ece58b46b2b6b7cd6360 Reviewed-on: https://gerrit.libreoffice.org/76107 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22tdf#106000 sc ui: fix missing invalidation when cancelling note creationMiklos Vajna
Regression from commit dca01def7885ad69cf66edd75cf8207a5adb64f9 (refactor ListBox/ComboBox to use RenderContext, 2015-05-07), invalidating the area of note anchors worked like this in the past: - user ended text edit of a new, empty comment - the Calc shell was updated from shape text edit to the tab view, which called ImplListBoxWindow::SelectEntry() - this called ImplPaint(), which invoked maUserDrawHdl (if it was set) - and at the end this invoked ScViewFunc::UpdateSelectionArea(), all triggered by the font name and font size listboxes So the update of the cell of the former note depended on the presence of font name/size listboxes, by accident. This broke, because ImplListBoxWindow::SelectEntry() now calls Invalidate(), and Paint() calls ImplDoPaint(), which doesn't invoke maUserDrawHdl anymore. Fix the problem by invalidating the cell explicitly, so in case the user customizes the toolbar to not have these listboxes, the behavior is still correct. Do this in FuText::StopEditMode(), because there we know the position of the deleted note and that's where we delete the matching SdrCaptionObj (via the RemoveLastUndoAction()). Change-Id: Iaa4b609477ac6c7f23efc575fddac6e39aa6a7b5 Reviewed-on: https://gerrit.libreoffice.org/76120 Reviewed-by: Luboš Luňák <l.lunak@collabora.com> Tested-by: Jenkins Reviewed-by: Dennis Francis <dennis.francis@collabora.com> Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2019-07-22MSForms: DOCX filter: fix crash when the date field is inside a shapeTamás Zolnai
Change-Id: Ida6ff48e6e743e41dd793e31c11065f870e8959b Reviewed-on: https://gerrit.libreoffice.org/76117 Tested-by: Jenkins Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2019-07-22Simplify Sequence iterations in shell..svgioArkadiy Illarionov
Use range-based loops, STL and comphelper functions Change-Id: I612d36abcc09a91c60f7212de6747a1a1bdcfc69 Reviewed-on: https://gerrit.libreoffice.org/76056 Tested-by: Jenkins Reviewed-by: Arkadiy Illarionov <qarkai@gmail.com>
2019-07-22cid#1222237 try silence Untrusted value as argumentCaolán McNamara
Change-Id: I9a8303cd996c91855fc624b875fbc8f1286a58b5 Reviewed-on: https://gerrit.libreoffice.org/76100 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22avmedia: Drop 'wrap_*' functionsMichael Weghorn
Just call the corresponding GStreamer functions directly. The wrapper functions are pointless since commit 6b911ae9eb9484bebbdc1323210020486f5ef33f ("Drop support for gstreamer-0.10"). Change-Id: I2963bd25a83632f4e693a1691180790452edffda Reviewed-on: https://gerrit.libreoffice.org/76116 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2019-07-22cid#1448433 Unintended comparison to logical negationCaolán McNamara
Change-Id: I78488ff5b8087eabb4dd306f226defa2607b1660 Reviewed-on: https://gerrit.libreoffice.org/76096 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448485 silence Copy-paste errorCaolán McNamara
Change-Id: I6c82f9d87203d90099e39880bfd3feb2c2ff8354 Reviewed-on: https://gerrit.libreoffice.org/76097 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448321 Dereference after null checkCaolán McNamara
Change-Id: Ic8f70f01ea78d40ae1676c2d6561fa39dcb8ae9e Reviewed-on: https://gerrit.libreoffice.org/76113 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1242892 Untrusted value as argumentCaolán McNamara
Change-Id: Ie08f75468fc7acbdf8b89496c750e5fec6d19f0d Reviewed-on: https://gerrit.libreoffice.org/76103 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448269 Dereference null return valueCaolán McNamara
Change-Id: Ief2dfb8a87030d992a867a7a9014d3787e201831 Reviewed-on: https://gerrit.libreoffice.org/76111 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448325 Dereference after null checkCaolán McNamara
Change-Id: I7ca6b442f9aadaeb6debe7806e30693aa06b90ff Reviewed-on: https://gerrit.libreoffice.org/76114 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448361 silence Explicit null dereferencedCaolán McNamara
Change-Id: I70d5f91d9f4f66e7fdfee67c7d9398d5435e2374 Reviewed-on: https://gerrit.libreoffice.org/76115 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448318 Dereference null return valueCaolán McNamara
Change-Id: Ibdb452f5c8a6da255e3641d38078870d4206f913 Reviewed-on: https://gerrit.libreoffice.org/76112 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22kf5/gtk3_kde5 fpickers: Drop unused 'allowRemoteUrls'Michael Weghorn
Change-Id: I591bf9fbe6e22bedda5a94334130a4f674cd66e6 Reviewed-on: https://gerrit.libreoffice.org/76110 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2019-07-22cid#1448263 Dereference after null checkCaolán McNamara
Change-Id: I48f5bbd0f51bed68a3a1772e17c6570248545370 Reviewed-on: https://gerrit.libreoffice.org/76109 Reviewed-by: Michael Stahl <Michael.Stahl@cib.de> Tested-by: Jenkins
2019-07-22cid#1448220 Explicit null dereferencedCaolán McNamara
Change-Id: I0e612cf738d54e2108458da5c4afe78f1699f475 Reviewed-on: https://gerrit.libreoffice.org/76105 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448223 Dereference after null checkCaolán McNamara
Change-Id: I2c8909d7005442532054596c681d96dfc6f5d462 Reviewed-on: https://gerrit.libreoffice.org/76106 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1448542 Unintended sign extensionCaolán McNamara
Change-Id: I8673f177a0ae6fe9bfd6e2ee7a87f80b058bb24f Reviewed-on: https://gerrit.libreoffice.org/76104 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#705118 Using invalid iteratorCaolán McNamara
Change-Id: If90375ee49e71357482f86e31185c95774aef17c Reviewed-on: https://gerrit.libreoffice.org/76092 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1401334 Uncaught exceptionCaolán McNamara
Change-Id: I155680d8f78c1bca854b9e491fc26f82b48e0f07 Reviewed-on: https://gerrit.libreoffice.org/76091 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-22cid#1209863 Untrusted loop boundCaolán McNamara
Change-Id: Ie9c3672a065b9df4580559cd927c6b1524edde0e Reviewed-on: https://gerrit.libreoffice.org/76099 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>