summaryrefslogtreecommitdiff
path: root/sc
AgeCommit message (Collapse)Author
2017-11-26Add debug output operator<< for ScConditionModeTor Lillqvist
Change-Id: I031d080dc8cbfcc5619bf7d5ee6112dde0a2ef04 Reviewed-on: https://gerrit.libreoffice.org/45277 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-26Convert ScFormatEntryType to scoped enumTor Lillqvist
Change-Id: I9a6b3fb05255630852a96ebf74d901b869a581f4 Reviewed-on: https://gerrit.libreoffice.org/45276 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-25Avoid a trivial typedefTor Lillqvist
Change-Id: Iea8700f168c5d13ebf1b98f435eeabea425b8259 Reviewed-on: https://gerrit.libreoffice.org/45275 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-25Revert "ofz#4406: don't increment ScCompressedArray::Iterator past end"Caolán McNamara
that doesn't help of course This reverts commit ab58e2e68f969e49b97b28b81225ed1e87df0855.
2017-11-25coverity#1422232 Improper use of negative valueCaolán McNamara
Change-Id: I79b24e175906ca5552f45b6af472c436ffde5fe3 Reviewed-on: https://gerrit.libreoffice.org/45247 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-25Convert ScConditionMode to scoped enumTor Lillqvist
Change-Id: I9b6aaa64e7f649581b27f0238e2908026ab6e0bd Reviewed-on: https://gerrit.libreoffice.org/45157 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-24tdf#97198 Make Calc function LEFT work with Unicode non-BMP charactersWinfried Donkers
Change-Id: I66d24dd962f824079fd0aea2827bd5b975ffe698 Reviewed-on: https://gerrit.libreoffice.org/44684 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#97198 Make Calc function SEARCH work with UniCode non-BMP characters.Winfried Donkers
Change-Id: I99b44dc88fe9c8d4d5d7a319bd6bc84dd2bff4f3 Reviewed-on: https://gerrit.libreoffice.org/45170 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#97198 Make Calc function LEN work with Unicode non-BMP characters.Winfried Donkers
Change-Id: Ibfdaaf1ed313257fc5f9036a7dd1ef51033fa8cd Reviewed-on: https://gerrit.libreoffice.org/44681 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#97198 Make Calc function FIND work with UniCode non-BMP characters.Winfried Donkers
Change-Id: I512199a5d29aa955b278f0666c7ce904e67d9767 Reviewed-on: https://gerrit.libreoffice.org/45171 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#97198 Make Calc function RIGHT work with Unicode non-BMP characters.Winfried Donkers
Change-Id: I775bc50d8e5948a2af9efb77dcea9d7cae7293d5 Reviewed-on: https://gerrit.libreoffice.org/44834 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#97198 Make calc function MID work with Unicode non-BMP characters.Winfried Donkers
Change-Id: Ic86344495490d6ca942f9fd2752874da22ef531f Reviewed-on: https://gerrit.libreoffice.org/45040 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2017-11-24tdf#45904 Move _XSheetAnnotationAnchor Java test to C++Jens Carl
Fixes #i109517 for sc.ScCellObj by adding a note/comment during the set up of the test in sccellobj.cxx and allows to re-enabled the test. Change-Id: Ic7aa6bcb2606f555a76612a6f50c200d738674db Reviewed-on: https://gerrit.libreoffice.org/44617 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jens Carl <j.carl43@gmx.de>
2017-11-24ofz#4406: don't increment ScCompressedArray::Iterator past endCaolán McNamara
Change-Id: I2adc204722a6143384bab8aecd16bed42cb5016a Reviewed-on: https://gerrit.libreoffice.org/45192 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-24drop unused ATTR itemsNoel Grandin
the table autoformat stuff does not use these values at all, it just writes the internal details of PoolItems in a pre-determined order Change-Id: Id6d8ae7508a8d19f5fd1e70f9a428d324082d3bf Reviewed-on: https://gerrit.libreoffice.org/45199 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-24consistently use sal_uInt32 for number formats in scNoel Grandin
instead of a mix of short/sal_uLong/sal_uInt32 Change-Id: Ie5bd26e1a6f716c0c4e174a6d560827084b3f421 Reviewed-on: https://gerrit.libreoffice.org/45159 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-23Document, and simplify the two uses of ScMutationGuard.Michael Meeks
Creates ScMutationDisable - used to ensure that no core data structure is mutated below this guard in any thread. This can also be used to disable access in the same thread now. Change-Id: I7e4e98d8ff986490ccd5064b3b9af56acd877b49 Reviewed-on: https://gerrit.libreoffice.org/45119 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2017-11-23related tdf#108757 small enhancements for speed upSerge Krot
- no strings reallocation - faster for-loops Change-Id: I98a355eb3bcb48219afd6334615c9c092ed1a352 Reviewed-on: https://gerrit.libreoffice.org/45142 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2017-11-23Add debug output operator<< for ScRangeListTor Lillqvist
Change-Id: I55f16428bbadd45d302ac31df9c1a691730c4a49 Reviewed-on: https://gerrit.libreoffice.org/45158 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-23crashtesting: crash loading fdo41403-1.xlsmCaolán McNamara
regression since... commit 3af8a4faf5e20ccae2386052697713da160afb16 Date: Wed Nov 15 13:10:40 2017 +0300 tdf#113861: Less aggressive expansion of chart headers. pre-change code had... FormulaToken* pToken = it2->second; if (pToken && nRow>=nHeaderRowCount) check, so restore that check for presence of FormulaToken Change-Id: Ia1b301e4f7fdf447b08de7fb22bd7fde8735bd52 Reviewed-on: https://gerrit.libreoffice.org/45155 Reviewed-by: Vasily Melenchuk <vasily.melenchuk@cib.de> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-23Resolves: tdf#113962 save detected filter outside file dialogCaolán McNamara
different file dialog impls work differently so store the detected filter outside the file dialog Change-Id: I144d8df06e5265fed8f11b284a2f40c17c3e09ef Reviewed-on: https://gerrit.libreoffice.org/45166 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-23Move the need for several ugly const_casts deeper down to just one placeTor Lillqvist
Change-Id: I2cdd66b9819f070548c2a0af7388d9bfbd3689c1 Reviewed-on: https://gerrit.libreoffice.org/45144 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2017-11-23TypedWhichId create custom get methods in ScPatternAttrNoel Grandin
Change-Id: Iaa326332f5806477dd81463e6b6004a962bac934 Reviewed-on: https://gerrit.libreoffice.org/45128 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-23tdf#50746 add paste unformatted to menu and context menuYousuf Philips
also unified menus and context menus in writer and calc Change-Id: I929df7a0fe2be7b8bf58cf048e53536ea89d5c9c Reviewed-on: https://gerrit.libreoffice.org/45095 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-11-23Disabled slot with Put leaks entry (tabvwsha/sc)Julien Nabet
Same as https://cgit.freedesktop.org/libreoffice/core/commit/?id=21dde7a09e2cfe7b48d2fec1edc7a94fa94af5c3 Change-Id: Ie4e3fcf097a6aa7b9a019947a677701723f98bf3 Reviewed-on: https://gerrit.libreoffice.org/45052 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2017-11-23Make loplugin:unnecessaryparen look through implicitStephan Bergmann
...similar to how <https://gerrit.libreoffice.org/#/c/45083/2> "Make not warning about !! in loplugin:simplifybool consistent" does for loplugin:simplifybool Change-Id: I23eef400af71c582d380c9bae6546ce06e8a1e18 Reviewed-on: https://gerrit.libreoffice.org/45122 Tested-by: Stephan Bergmann <sbergman@redhat.com> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-11-23tdf#45904 Move _XScenarioEnhanced Java test to C++Jens Carl
Change-Id: Ibc99dc0d50e612bc6cdebaf5f9c5f828e5af859f Reviewed-on: https://gerrit.libreoffice.org/45123 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jens Carl <j.carl43@gmx.de>
2017-11-23tdf#45904 Move _XSheetAuditing Java tests to C++Jens Carl
Change-Id: I057447494ff488546d43d2d3068a65d5c0d402e0 Reviewed-on: https://gerrit.libreoffice.org/45120 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jens Carl <j.carl43@gmx.de>
2017-11-23TypedWhichId for sc constantsNoel Grandin
Change-Id: I25a07d84ae24fdd3b4fbd303beea755169eb9acb Reviewed-on: https://gerrit.libreoffice.org/45084 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-23loplugin:simplifybool for negation of comparison operatorNoel Grandin
Change-Id: Ie56daf560185274754afbc7a09c432b5c2793791 Reviewed-on: https://gerrit.libreoffice.org/45068 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-22Avoid using the hideous std::stack -> deque inside ::InterpretMichael Meeks
dequeue loves to allocate and free memory crazily, vector is much saner. Change-Id: Idcd2c1d693594f280ce94423161651502f25dc2d Reviewed-on: https://gerrit.libreoffice.org/45086 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2017-11-22Don't re-create the interpreter context for each row.Michael Meeks
Change-Id: I272c9dc09806d47add26866f517e6ea07282c35f Reviewed-on: https://gerrit.libreoffice.org/45087 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2017-11-22Follow our naming conventions, tdf#39593 follow-upEike Rathke
Change-Id: Iee95b5bf0dd42169447163a41be5b2aa1899e83c
2017-11-22Make loplugin:unnecessaryparen warn about (x) ? ... : ... after allStephan Bergmann
...which had been left out because "lots of our code uses this style, which I'm loathe to bulk-fix as yet", but now in <https://gerrit.libreoffice.org/#/c/45060/1/> "use std::unique_ptr" would have caused an otherwise innocent-looking code change to trigger a loplugin:unnecessaryparen warning for pFormat = (pGrfObj) ? ... (barring a change to ignoreAllImplicit in compilerplugins/clang/unnecessaryparen.cxx similar to that in <https://gerrit.libreoffice.org/#/c/45083/2> "Make not warning about !! in loplugin:simplifybool consistent", which should also have caused the warning to disappear for the modified code, IIUC). Change-Id: I8bff0cc11bbb839ef06d07b8d9237f150804fec2 Reviewed-on: https://gerrit.libreoffice.org/45088 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2017-11-22clang-format on non-blacklisted file interpretercontext.hxxDennis Francis
which is a new header file introduced in calc formulagroup threading work. Change-Id: I8203ce392b36581f64457edb52449df3e84fb216 Reviewed-on: https://gerrit.libreoffice.org/45080 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2017-11-22turn some eof checks to eof or errorCaolán McNamara
Change-Id: I4592d7aa86c2cc2f420bccc622ffb95aa5cefa17 Reviewed-on: https://gerrit.libreoffice.org/45077 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-22replace check of eof and GetError with goodCaolán McNamara
Change-Id: I7d9f04262ab5420e9a14813fa1274bb9d01e3291 Reviewed-on: https://gerrit.libreoffice.org/45076 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-22drop duplicate methodCaolán McNamara
Change-Id: Idadd0a64e41cd02f5167b275081c3576a6224b12 Reviewed-on: https://gerrit.libreoffice.org/45075 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-22good() includes !IsEof() alreadyCaolán McNamara
Change-Id: I6eddd347333638c0a8ea924e6062f8e4d379a20e Reviewed-on: https://gerrit.libreoffice.org/45074 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-22tdf#39593 deduplicate code in ScOutputDatatagezi
Change-Id: Ie7efcdf23e65b2047af7972756c2ba11cc178672 Reviewed-on: https://gerrit.libreoffice.org/45064 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2017-11-22ofz#4361 Integer-overflowCaolán McNamara
Change-Id: I7b41a53622e2e87dc0998a4a181bb8e49ef8d277 Reviewed-on: https://gerrit.libreoffice.org/45046 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-22There appears to be no reason to ignore macros in these parts of...Stephan Bergmann
...loplugin:unnecessaryparen Change-Id: I473a1e16cf9f485a61af5477aca22798996253a3
2017-11-22Fix typoAndrea Gelmini
Change-Id: I591214b246f7702e52f71b95cdc2255bc296224b Reviewed-on: https://gerrit.libreoffice.org/45063 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-11-22TypedWhichId for SDRATTR* constants (3)Noel Grandin
Change-Id: Iea72cb3a4bbf693096de46269f58259b5952eedb Reviewed-on: https://gerrit.libreoffice.org/45024 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-22loplugin:simplifybool re-activate the !! warningNoel Grandin
Change-Id: Iac7d82a1c228734177be536e9a6c41803c03637b Reviewed-on: https://gerrit.libreoffice.org/45035 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-21ofz#4305 do not read past end of fileEike Rathke
Change-Id: I7e048c012296fc080434e527c533ea987184699b
2017-11-21Fix typosAndrea Gelmini
Change-Id: I4713dd5c590d331e599b7e9ab88f5cc65e8d7b87 Reviewed-on: https://gerrit.libreoffice.org/45045 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-11-21tdf#113970 : blacklist vlookup/hlookup for threading...Dennis Francis
... until the root issue is fixed. Change-Id: I8969a78fe252e0b12c5703519641f008d33dc02c Reviewed-on: https://gerrit.libreoffice.org/45039 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Dennis Francis <dennis.francis@collabora.co.uk>
2017-11-21coverity warningsCaolán McNamara
Change-Id: I456210188ab96e6be03e65c94eedb348cc93a8e1 Reviewed-on: https://gerrit.libreoffice.org/45023 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-21Introduce and use ScCellShell::HasClipboardFormat(), tdf#50746 follow-upEike Rathke
Intead of gathering all possible formats just to check one.. Change-Id: I80f3a2c957492c9f72084c69b867aaccfbdd0530