summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-12-04tdf#104227: fix subtypes DECIMAL/NUMERIC in resultsetJulien Nabet
bt here: https://bug-attachments.documentfoundation.org/attachment.cgi?id=129298 See also https://bugs.documentfoundation.org/show_bug.cgi?id=104227#c13 for more details Change-Id: Ied497cfff2f71e67161d541b90fe48bb3e2456b9 Reviewed-on: https://gerrit.libreoffice.org/31593 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tamás Bunth <btomi96@gmail.com> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2016-12-04Remove write-only fieldUlrich Gemkow
Change-Id: I45502d73939e99cd3d72c9d8aaa17dc820e78207 Reviewed-on: https://gerrit.libreoffice.org/31590 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-04tdf#88206 replace cppu::WeakImplHelper* in svtools and svxJochen Nitschke
and remove unused implbase*.hxx and compbase*.hxx includes Change-Id: Iedf6b6dce09b5baf714a1b3394bb7fb3526cca82 Reviewed-on: https://gerrit.libreoffice.org/31594 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-04gbuildtoide: builddir is not necessarily same as srcdirChristian Lohmaier
Change-Id: Ied9f9abcc75f2edcd518ac247907f573aa21f35e Reviewed-on: https://gerrit.libreoffice.org/31158 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04basic: cleanup test_math_methodsArnaud Versini
Change-Id: I11b3ca4c55151de7403ff9980ada9958d8ef12dc Reviewed-on: https://gerrit.libreoffice.org/31592 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Laurent BP <laurent.balland-poirier@laposte.net>
2016-12-04tdf#97361 remove = None assignments in check_fieldskerem
Change-Id: I7696f28e6b35e79e98614b295c749adebf33878b Reviewed-on: https://gerrit.libreoffice.org/31439 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: I5c0b9007e5a84643ccbe3c5b6b6a913a43667e18 Reviewed-on: https://gerrit.libreoffice.org/31573 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: I1ee686ff96615f7b8aa69c0cd4caafaae07318b7 Reviewed-on: https://gerrit.libreoffice.org/31493 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: I0c6610308f75a57e517b233db66c66b34eb25189 Reviewed-on: https://gerrit.libreoffice.org/31492 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: Ibeb0aff07aec223f274932f2b3a492e58db09a02 Reviewed-on: https://gerrit.libreoffice.org/31488 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: Ibcdc6a492abe5e26fb9b5244119be5dacc01b7b4 Reviewed-on: https://gerrit.libreoffice.org/31487 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04use proper message passingBjoern Michaelsen
Change-Id: I55ce4f6a9b033dc064beed8e70d35a5b5097846d Reviewed-on: https://gerrit.libreoffice.org/31451 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-04tdf#93430: Get run attributes of text objectsTamás Zolnai
.. which are not paragraphs. Note: For some reason in character attributes sequence we have a wrong Any value for "CharPosture" property. Change-Id: Ia45526c01cc381d3d6a1b56dbf4f03fdd38a0989 Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/31591
2016-12-03Calc test - Financial and Logical functions. move to new directoryZdeněk Crhonek
Change-Id: Ie94c190f53904372aebfa15749b65c807052fcc7 Reviewed-on: https://gerrit.libreoffice.org/31589 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-03oss-fuzz runs under dockerCaolán McNamara
and builds as root in that env, at least the default out of the box experience is that for me, I suppose I could faff around getting it add a user specifically for the build like I do for my coverity in docker build, or just drop this. This effectively reverts commit 466257699559390237615f651ce0f30f07543434 Author: Tor Lillqvist <tml@collabora.com> Date: Thu Apr 14 09:01:48 2016 +0300 149 Change-Id: I63f055756ff5f571d5ebe89aa4aee52d8ae2c510 I see your 149 and raise you a 1172 Change-Id: I3e81a725cdab1ff5ead0484dbfee11ac8fe303a9
2016-12-03cppcheck style fix for noExplicitConstructor in writerfilterJochen Nitschke
make ctors with one parameter explicit Change-Id: I08e753cea9f5ea17d654c889bad37ab7d0f4611e Reviewed-on: https://gerrit.libreoffice.org/31588 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2016-12-03Calc functions tests - move datetime functions to new directoryZdeněk Crhonek
Change-Id: I5b2c9455da5f700d0c4da59ee3d48e5aba486a55 Reviewed-on: https://gerrit.libreoffice.org/31583 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-03sc tests files - move array functions to the new directoryZdeněk Crhonek
Change-Id: Ia701f2b0a3c21e19b2de95c8012cc5a37ed88359 Reviewed-on: https://gerrit.libreoffice.org/31580 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-03Remove unnecessary indirections.Kohei Yoshida
Especially in the oox code there are many... Change-Id: I16915cc207c274e5bcdb5d1f4f8708db5a0479a1 Reviewed-on: https://gerrit.libreoffice.org/31578 Reviewed-by: Kohei Yoshida <libreoffice@kohei.us> Tested-by: Kohei Yoshida <libreoffice@kohei.us>
2016-12-03starmath: Prefix members of SmPolyLineNode etc.Takeshi Abe
Change-Id: I6fa81689e14e605cd7449526e44245d4fdebb298 Reviewed-on: https://gerrit.libreoffice.org/31536 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-12-03add the SalAbort message to the crash reporterMarkus Mohrhard
Change-Id: Ibe08034bcf2a73fb12562c87c2898db1e974376c Reviewed-on: https://gerrit.libreoffice.org/31577 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-03first cut at adding a GDI handles attribute to the windows crash report.Michael Meeks
Change-Id: I4e6fee65a1416398f6f59feb8734b3b00c0aed3b Reviewed-on: https://gerrit.libreoffice.org/31570 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-12-03Fix the formatting style a bit, to be more consistent.Kohei Yoshida
Change-Id: Iaca5837e5502ef8eed4cddbca1d00e00c71c53c2 Reviewed-on: https://gerrit.libreoffice.org/31576 Reviewed-by: Kohei Yoshida <libreoffice@kohei.us> Tested-by: Kohei Yoshida <libreoffice@kohei.us>
2016-12-03sc tests of functions - move information functions to the new directoryZdeněk Crhonek
Change-Id: Ica03ff03aa32d446335b6f6b8e6ec10f3c2c19db Reviewed-on: https://gerrit.libreoffice.org/31569 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-02Require HarfBuzz 0.9.42Khaled Hosny
This is the first version where hb_buffer_set_cluster_level() was introduced. The layout engine depends on this feature in a few places and though we can build without it, the result will be wrong in some cases (i.e. any where we assume combining marks will have their own cluster number). Change-Id: I89187f317052e7933b83ad0350cace239333510b Reviewed-on: https://gerrit.libreoffice.org/31521 Reviewed-by: Rene Engelhard <rene@debian.org> Reviewed-by: Khaled Hosny <khaledhosny@eglug.org> Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-12-02No component condition for a package file listMatúš Kukan
This is to fix a build after 67398a8d9746dbed743beb702a8cff2eac2bbf99 No idea if packages should contain conditions too, but at least gid_File_Bin_UnpackUpdate did not. Change-Id: I56e1b23f6eeca9c62cdadd8d93de6ccda89b20c4
2016-12-02remove now unused SvtSysLocaleOptions::GetLocaleConfigString()Eike Rathke
Which just lead to confusion as could be seen in two earlier commits.. Change-Id: I172127cf08bcd45c3c53e0cf00cf11a41e0dd3a6
2016-12-02use SvtSysLocaleOptions::GetLanguageTag() instead of GetLocaleConfigString()Eike Rathke
Change-Id: Ifcfc4588d5bd6f713a40bd2300740f7a1c52337c
2016-12-02use SvtSysLocaleOptions::GetRealUILanguageTag(), not GetLocaleConfigString()Eike Rathke
Assuming that the screenshots directory suffix * should match the configured UI locale instead of the work locale * should not be empty if configuration says "Default - ..." * actually ScreenshotTest::implSaveScreenshot() explicitly tests (maCurrentLanguage == "en-US") and then sets the suffix to empty, so no other locale should exhibit the same Change-Id: Ie4561315b67c3ade20f3947772af45e471d0752f
2016-12-02use SvtSysLocaleOptions::GetRealLanguageTag(), not GetLocaleConfigString()Eike Rathke
Testing for GetLocaleConfigString() starting with a language code could never have worked if configured was "Default - ..." in which case the config string is an empty string. Change-Id: I3a68a168a0f8e53c8c389fc243e40053f5d45e1f
2016-12-02scp2: Bin not defined gids and not used definesMatúš Kukan
And a little cleanup. Change-Id: Id066b4d37e217b8ab21ea162c23a999566a9e190
2016-12-02tdf#90753: AutoInstall readlicense_oo packagesMatúš Kukan
Change-Id: I5bd3a5f6168b20c925f5713085fcb0918bcccc8e
2016-12-02tdf#90753: AutoInstall setup_native packagesMatúš Kukan
Change-Id: Iecf059b974b996fceb57b04197c7dac9b7c8dd47
2016-12-02sufficient to get us building in the oss-fuzz environmentCaolán McNamara
we're presented with... CXX="/usr/local/bin/clang++" CC="/usr/local/bin/clang" CXXFLAGS="-g -stdlib=libc++" and out of the box we fail to build, but if we reshuffle to... CXX="/usr/local/bin/clang++ -stdlib=libc++" CC="/usr/local/bin/clang -stdlib=libc++" CXXFLAGS="-g" LDFLAGS="-lpthread" then with this distro-config we build to completion Change-Id: Ib6bee4dd69e0609e3c1238d86837a1680483ffa4
2016-12-02Simplify FontManager a bitKhaled Hosny
There does not seem to be any need for that atom thing as we are perfectly happy using plain OUStrings in the same struct, not to mention that font names are supposed to be unique so I don’t see what we are saving here. As this was the only use for unotools/atom, it goes with it. Change-Id: If9d58d84fff0403f9b2c41fe594b99028b30c2f4 Reviewed-on: https://gerrit.libreoffice.org/31520 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
2016-12-02tdf#104333 revert ww8import: set table keep/split if emulatedJustin Luth
tables that span multiple pages, and rows greater than one page caused problems, so just removing the enhancement that tried to emulate what the original odt looked like. reverts commit 129f93e46c29b388d38e9097869fd3e72dc40a5e from bug tdf#91083 Change-Id: Iac31116d731e1e4aa60ee6b8de973f7f9291e47a Reviewed-on: https://gerrit.libreoffice.org/31551 Reviewed-by: Justin Luth <justin_luth@sil.org> Tested-by: Justin Luth <justin_luth@sil.org>
2016-12-02starting skeleton for oss-fuzz configCaolán McNamara
gets vcl built anyway Change-Id: I521a53019f54d5e42c931043c758143f2f8bdb1d
2016-12-02xmlsecurity mscrypto PDF verify: implement support for non-detached signaturesMiklos Vajna
This was the last unit test that was disabled on Windows due to missing implementation. Change-Id: Ia7d84f72bcdf79267c7de17cd8822ed02c378642 Reviewed-on: https://gerrit.libreoffice.org/31552 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-12-02warning C4018: '<' : signed/unsigned mismatchStephan Bergmann
Change-Id: I5e6a6debf01886c283f5a2d6b9cacc00a44f7134
2016-12-02fix display of DMG background on Mac OS X SierraChristian Lohmaier
Change-Id: Ib553e085337972f74857db0aad9bc885712d5ea3 Reviewed-on: https://gerrit.libreoffice.org/29788 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>
2016-12-02SwDrawContact, SwFlyDrawContact: move code from Modify to SwClientNotifyBjoern Michaelsen
- also move from deprecated OSL_ENSURE to SAL_ macros - also flattened and refactored the deeply nested copypasta Change-Id: Ib7333ae9589b93bca999c8b1d003b712c59920ee Reviewed-on: https://gerrit.libreoffice.org/31391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
2016-12-02usageinfo causes one of our more frequent fedora 25 crashesCaolán McNamara
https://retrace.fedoraproject.org/faf/problems/bthash/?bth=17ba9a57f40d5ca29778866a9f96c1c0dfc7593c&bth=2f1801e44e9cb0be2b013624521fb0959dc2c73a&bth=5e8a1e07503f53b13b3a6779084f8b0637aaf5a7&bth=8033013a2ce586bccc23245583fa86ef4a761dce&bth=975271167132418534cb07336601efb5b0c711f0&bth=de75c2ad7917bc889d6307b152f79d34e79ce74d Change-Id: Ie93afa476eca9fdbf7ea17ff0189fbaa493a5531
2016-12-02get vertices count without jumping through hoopsDavid Tardon
Change-Id: I0cab6b2f9fffc8cf9fc5059b8dc48fa9407e42b5
2016-12-02rename to avoid further misunderstandingsDavid Tardon
Change-Id: Ibcfae1b3ec118d74921312a80eb56c97c623f4a9
2016-12-02avoid valgrind warningDavid Tardon
Change-Id: I8ffb08bdab452f6a8618eaf466b722e7c8d226d9
2016-12-02xmlsecurity PDF verify: don't hide signatures where digest match is uncertainMiklos Vajna
Use case: the bugdoc has 2 signatures, one normal one and one with SubFilter=ETSI.RFC3161. By not hiding the second signature it's possible to counter-sign the document, even if we don't handle the contents of the second one. Change-Id: I580e1211072ec9839f01b529b569c98b702b6534 Reviewed-on: https://gerrit.libreoffice.org/31539 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-12-02coverity#1394668 Inferred misuse of enumCaolán McNamara
just keep the current logic and silence coverity about it Change-Id: I397632eab8ce983e362380b83bbf78c1fbf8e038 Reviewed-on: https://gerrit.libreoffice.org/31535 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2016-12-02Lok: correct tile invalidation mergingAshod Nakashian
Rectangles that are empty (i.e. IsEmpty() returns true) were incorrectly considered to mean equivalent to "EMPTY". The latter means full-area, while the former mean zero-area. This fixes the issue by restrict full-area to rectangles with 2 billion units on the each side (roughly INT_MAX) or more, and using this new check rather than IsEmpty(). Change-Id: I12aca17267f5dd33b2932012d1d9db3545f9af6f Reviewed-on: https://gerrit.libreoffice.org/31458 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-12-02Postfix rather than prefix locale to directory nameKatarina Behrens
as this is what vnd.libreoffice.image:// expects for localized images Change-Id: I454ff86b86e11248f47ab65f682fb57e52d0f7e5 Reviewed-on: https://gerrit.libreoffice.org/31057 Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de> Tested-by: Katarina Behrens <Katarina.Behrens@cib.de>
2016-12-02oops, wasn't supposed to commit thatNoel Grandin
snuck in with commit 4d2c210c74567d9af6bededf3fae6bfd62406f14 "use early return to simplify flow" Change-Id: I2a5cbff01c5f40d879d320fd3934bc0856756d37