summaryrefslogtreecommitdiffstats
path: root/oox
Commit message (Collapse)AuthorAgeFilesLines
* improve passstuffbyref return analysisNoel Grandin2016-07-275-19/+19
| | | | | | | Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* oox: avoid pointless multiplication at int precision in WpsContextMiklos Vajna2016-07-211-1/+1
| | | | | | | | | When the output would be used at long precision anyway. Change-Id: I6eefda0533b8ee70b878f7ce7c6487f8b99a6025 Reviewed-on: https://gerrit.libreoffice.org/27360 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* oox: indentation fixesMiklos Vajna2016-07-082-2/+2
| | | | | | | These files had consistently no space between the "const" and "&", keep it that way. Change-Id: I5463c488d807344d67fb22ad544c049ea749be94
* loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann2016-07-077-13/+13
| | | | Change-Id: I19eaebdb25db8881d84625a4b9d0df0a588aab4f
* tdf#100179 correct cell merging in pptx.Mark Hung2016-06-301-2/+2
| | | | | | | Change-Id: I7a5bc6fb33c64d1ff398986eff5b960fe037df4d Reviewed-on: https://gerrit.libreoffice.org/25791 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
* RTF export: it's enough to pass these by const ref, no need to copyMiklos Vajna2016-06-301-1/+1
| | | | | | | Change-Id: I6854c3c8e9a2b5c6d478551c5f9bba19e114152c Reviewed-on: https://gerrit.libreoffice.org/26787 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* warn->info: its just a factCaolán McNamara2016-06-241-1/+1
| | | | Change-Id: Id3f468cc4bb4f6f0b34bcc8486817c3f9c994020
* tdf#100153 modify noSmoking shapeZhengqiang Wang2016-06-233-6/+6
| | | | | | | | | | | | | | | *modify customshapeproperties.cxx fix compile error : make oox dbglevel=2 The define of dumpCode() has been removed by c714b43b0137ad253f51e1c65417c9874b8509e5 *modify presetShapeDefinitions.xml fix noSmoking shape *modfiy oox-drawingml-cs-presets this file is generated by presetShapeDefinitions.xml Change-Id: I383d6b6961b546585035afd8c1a4398385a11914 Reviewed-on: https://gerrit.libreoffice.org/25801 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* simplifyNoel Grandin2016-06-211-1/+1
| | | | Change-Id: I67de532443cbb9e64bed67fe28ee94fef38f5eae
* fix indentationMarkus Mohrhard2016-06-181-10/+10
| | | | Change-Id: I3a65eae9a7d55aa42540567a19052d6339ecdbec
* tdf#42949: clean up includes in include/filter with iwyuJorenz Paragas2016-06-152-0/+3
| | | | | | | Change-Id: Id08a999764bfeeff3bb3409b46f48b278009ccbe Reviewed-on: https://gerrit.libreoffice.org/26238 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Fix typosAndrea Gelmini2016-06-131-1/+1
| | | | | | | Change-Id: Ic3efae53430fbb2c1eb7962c3cf69bf99315e55b Reviewed-on: https://gerrit.libreoffice.org/26202 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* Clean up uses of Any::getValue() in ooxStephan Bergmann2016-06-102-86/+77
| | | | Change-Id: I76d83ad472eb0cd44b593a86dac1f8c1ee8c3585
* tdf#42949: clean up includes in include/oox/vml with iwyuJorenz Paragas2016-06-103-0/+3
| | | | | | | Change-Id: I753880ce146fef6523561290b8e13310d9f7219c Reviewed-on: https://gerrit.libreoffice.org/26142 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Revert "Slideshow: Add 'Oval' Shape Transition variant"jan iversen2016-06-092-18/+0
| | | | | | | | | | | | | The patch caused problems with the export filter tests, to check that you need to add --with-export-validation to your autopen.input see https://cgit.freedesktop.org/libreoffice/contrib/dev-tools/tree/export-validation for more info on how to set it up This reverts commit 248c5ea771255b54e64394458a321ccf829bbd02. Change-Id: Ib3b8fa7bf80630feeca1f24dfb1ceb5a945d7162 Reviewed-on: https://gerrit.libreoffice.org/26114 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* Slideshow: Add 'Oval' Shape Transition variantMayank Gupta2016-06-092-0/+18
| | | | | | | | Change-Id: Ibc3d617d3bb94bdd0702bb4d60ce5fbe2eea8e24 Reviewed-on: https://gerrit.libreoffice.org/23661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* tdf#94698 cleanup Makefiles. Get rid of udkapi and offapiGleb Mishchenko2016-06-093-12/+3
| | | | | | | | | | | | | | | | | | | | This patch changes occurences in makefiles from $(eval $(call gb_CppunitTest_use_api,comphelper_test_config, \ udkapi \ offapi \ )) to $(eval $(call gb_CppunitTest_use_sdk_api,comphelper_test_config)) (corrected instead of abandon, jani) Change-Id: Ic96ec65d82d7452e288f05a8b6d576ef543b068e Reviewed-on: https://gerrit.libreoffice.org/23426 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* Let loplugin:passstuffbyref also look at fn defn not preceded by any declStephan Bergmann2016-06-081-1/+1
| | | | Change-Id: I752bc96d2d521d790e919283cabb14b6526626f4
* tools: rename SvStream::Read/Write to ReadBytes/WriteBytesMichael Stahl2016-06-062-6/+6
| | | | | | | Change-Id: Ib788a30d413436aa03f813aa2fddcbc4d6cd2f9a Reviewed-on: https://gerrit.libreoffice.org/25972 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#93883 don't add space to empty paragraph when bullet is turned off.Mark Hung2016-05-311-1/+4
| | | | | | | Change-Id: Ia1bad61cb5585dae0501e8cd657fc84bfea60ab2 Reviewed-on: https://gerrit.libreoffice.org/25679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Convert GfxLinkType to scoped enumNoel Grandin2016-05-311-9/+9
| | | | | | | Change-Id: Iae074ef0927a55f1fbf401cb66f4f183213f547c Reviewed-on: https://gerrit.libreoffice.org/25656 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* teach refcounting clang plugin about uno::ReferenceNoel Grandin2016-05-311-2/+2
| | | | | | | | | | | | | | uno::Reference is only allowed to used with classes that have a ::static_type member. So convert all those places to rtl::Reference. Maybe we need some LIBO_INTERNAL_ONLY constructors on rtl::Reference and uno::Reference to make this a little smoother? Change-Id: Icdcb35d71ca40a87b1dc474096776412adbfc7e3 Reviewed-on: https://gerrit.libreoffice.org/25516 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert GRAPHIC to scoped enumNoel Grandin2016-05-271-2/+2
| | | | | | | Change-Id: I1fd09a729cbda00f99841532e0dd3fa66bce7bea Reviewed-on: https://gerrit.libreoffice.org/25534 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#42949: clean up includes in include/oox/ppt with iwyuJorenz Paragas2016-05-2610-1/+11
| | | | | | | Change-Id: Ib2cf76418e8406b925113ae7a9e3c5b07ffd2de2 Reviewed-on: https://gerrit.libreoffice.org/25011 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:unusedreturntypesNoel Grandin2016-05-261-2/+1
| | | | | | | | | and clean up the python script Change-Id: I0a7068153290fbbb60bfeb4c8bda1c24d514500f Reviewed-on: https://gerrit.libreoffice.org/25439 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* oox: replace boost::array with std::arrayMichael Stahl2016-05-251-4/+2
| | | | Change-Id: Id9a8f7dbbf13890e7c9787d9b0737f1087e4a509
* warning C4334: '<<': result of 32-bit shift implicitly converted to 64 bitsNoel Grandin2016-05-201-1/+1
| | | | | | | | in VisualStudio reported by Luke Benes Change-Id: I908039750a3e89e4d1b77631f7d309191735c765
* clang-tidy modernize-make-sharedNoel Grandin2016-05-1814-35/+34
| | | | | | | Change-Id: I3fa866bfb3093fc876474a9d9db29fe05dc2af3a Reviewed-on: https://gerrit.libreoffice.org/25056 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#42949: clean up includes in include/oox/ole with iwyuJorenz Paragas2016-05-132-0/+6
| | | | | | | Change-Id: I9cbdc78b540b0f735c451ac561f0f0ccdaf7efbd Reviewed-on: https://gerrit.libreoffice.org/24911 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#42949: clean up includes in include/oox/mathml with iwyuJorenz Paragas2016-05-111-0/+2
| | | | | | | Change-Id: Iefb4e7184009318c0b8859aa895e0b25fc9d9213 Reviewed-on: https://gerrit.libreoffice.org/24871 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#42949: clean up includes in include/oox/helper with iwyuJorenz Paragas2016-05-1088-5/+150
| | | | | | | | | ...and fix the many compiler errors that occurred as a result. Change-Id: I497c326272b2f02737ca3765720d6815b735423b Reviewed-on: https://gerrit.libreoffice.org/24735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Silence Clang -Wimplicit-fallthrough in gperf-generated codeStephan Bergmann2016-05-101-3/+2
| | | | | | ...in preparation of eventually enabling -Wimplicit-fallthrough for Clang. Change-Id: Idca352d011505e83dbe275cb19626c8b9d8560c8
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-105-6/+25
| | | | | | | | | | | | | | | | ...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
* Insert explicit "break" when falling through to empty next caseStephan Bergmann2016-05-104-0/+4
| | | | | | | ...which itself only contains a "break" (or nothing at all at the end of the "switch"), as otherwise Clang -Wimplicit-fallthrough would warn about these. Change-Id: I25c1cf2ca74dfeba7ca0385ca8f1c1bf30bbf91b
* loplugin:rangedforcopyTor Lillqvist2016-05-101-2/+2
| | | | | | Loop variable passed by value, pass by reference instead. Change-Id: I12d1cb639cbe7a5686a1854168362b85322fa2e5
* remove STATIC_ARRAY_END and usagesJochen Nitschke2016-05-104-93/+93
| | | | | | | | | | | | used in loops, rewrite to range based loop oox/source/drawingml/color.cxx has some mappings in plain arrays make them arrays of pairs which can be used in range based loops Change-Id: Ib6693197d890f595c27ca24b9f9b4e0763747f4c Reviewed-on: https://gerrit.libreoffice.org/24809 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
* basegfx,oox,slideshow: replace boost::next/prior with std::next/prevMichael Stahl2016-05-062-26/+24
| | | | Change-Id: I38fde54d0fdbb9c61e3df004242a70e14429f52f
* tdf#42949: clean up includes in include/oox/export with iwyuJorenz Paragas2016-05-062-0/+3
| | | | | | | | | ...and fix the usual compiler errors that occurred as a result. Change-Id: I54781d3f01e9f5a87fd008870c825b2d8154fd59 Reviewed-on: https://gerrit.libreoffice.org/24686 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* teach passstuffbyref plugin to check for..Noel Grandin2016-05-062-2/+2
| | | | | | | | | | | | unnecessarily passing primitives by const ref. Suggested by Tor Lillqvist Change-Id: I445e220542969ca3e252581e5953fb01cb2b2be6 Reviewed-on: https://gerrit.libreoffice.org/24672 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* clang-tidy modernize-loop-convert in oox to saxNoel Grandin2016-05-055-18/+18
| | | | | | | Change-Id: If0d87b6679765fc6d1f9300c6972845cf3742b9c Reviewed-on: https://gerrit.libreoffice.org/24674 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#42949: clean up includes in include/oox/drawingml with iwyuJorenz Paragas2016-05-0319-0/+23
| | | | | | | | | Add includes in other files to fix compiler errors. Change-Id: I566e2e081094cb074d20afbe7ceee49f759c725b Reviewed-on: https://gerrit.libreoffice.org/24510 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#99030: PPTX import: Fixed lost slide background colorMatus Uzak2016-04-292-2/+4
| | | | | | | | | Regression from commit f3d1ac7 Change-Id: I5cb9fe1bb6c753c34b49e72194a9fbe4c10c1654 Reviewed-on: https://gerrit.libreoffice.org/23930 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
* tdf#90736: PPTX import: Improved table border line style supportMatus Uzak2016-04-291-1/+41
| | | | | | | | | | | Map preset line dash values from ooxml to the smaller set of LO's predefined table border line styles. Map custom dashing scheme to a simple dashed line style. Change-Id: Ia81d94cb7e20b05f1c2cc0c20c8ad398c5613d6c Reviewed-on: https://gerrit.libreoffice.org/24048 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
* teach stylepolice plugin about ref-counted-pointer namingNoel Grandin2016-04-282-5/+5
| | | | | | | Change-Id: I6e91d22fc1826038c05ddb6fc065563c6a250752 Reviewed-on: https://gerrit.libreoffice.org/24459 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#80999 Canceling password prompt should abort detectionMaxim Monastirsky2016-04-271-5/+13
| | | | | | | | | | | | | | | ... instead of continuing the detection loop and being "detected" as plain text. The detection API will from now return a type based on the file extension only, which is far more useful than "plain text" anyway. Plus the media descriptor has a flag to indicate that the detection wasn't completed, which can be also used by the loading code to abort the loading process. Change-Id: Ida37e2687bd5cd86b5780620724e9bce82e11946 Reviewed-on: https://gerrit.libreoffice.org/24428 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
* tdf#42949: clean up includes in include/oox/crypto with iwyuJorenz Paragas2016-04-265-3/+20
| | | | | | | | | | | | | | | | | The includes in CryptTools.hxx that are within #if/#endif directives are not altered. iwyu suggested replacing the includes for nss.h, pk11pub.h, and sechash.h with hasht.h, pkcs11t.h, seccomon.h, and secmodt.h. I decided not to because it might make CryptTools.hxx harder to refactor in exchange for little gain. Before creating this commit, I ran 'make check' after passing --with-tls=openssl to ./autogen.sh to ensure that building with either OpenSSL or NSS (the default except on iOS and Android) works. Change-Id: I20260d18f073ffd8077bbcc597e7a8e4954e2ec5 Reviewed-on: https://gerrit.libreoffice.org/24386 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* update loplugin stylepolice to check local pointers varsNoel Grandin2016-04-268-39/+39
| | | | | | | | | | | | are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Some more sal_Bool -> boolStephan Bergmann2016-04-251-1/+1
| | | | Change-Id: Iaae2c92e011c55f8739ecb7972ef9a3f71300765
* tdf#42949: clean up includes in include/oox/core using iwyuJorenz Paragas2016-04-2568-2/+159
| | | | | | | | | | | | | This allows the headers in that folder to stand on their own and not depend on any transitively included headers. All changes to other files are needed to fix compiler errors related to missing includes that occurred afterwards. Change-Id: I2083b30763f0b2c1aacdff00226e5f567ae52db4 Reviewed-on: https://gerrit.libreoffice.org/24355 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* Avoid reserved identifiersStephan Bergmann2016-04-222-3/+3
| | | | Change-Id: Ie1664e06dc02f7070e4ef77155e6541c70b2f8e8
generated by cgit (git 2.34.1) at 2024-08-09 05:55:05 +0000