summaryrefslogtreecommitdiff
path: root/oox
AgeCommit message (Collapse)Author
2015-02-19Remove newly unused includeStephan Bergmann
Change-Id: I3a6e668b930cb5f65abe345fdf5b5e4b1b511440
2015-02-19"no matching function for call to 'get_pointer'," use lambda insteadStephan Bergmann
Change-Id: I40ea3b781ec35171706a277acff6eb2eab29c297
2015-02-19boost->stdCaolán McNamara
Change-Id: Icae6b6f07ad8dbd287fdfc689739187883a07775
2015-02-19there is no x error bar for bar charts, related fdo#66455Markus Mohrhard
e.g. fdo#66455-1.ods exported to OOXML Change-Id: I81c65071cef820076b3f4788b45152cb48eb608d
2015-02-19document one more place where MSO 2007 export wrong valuesMarkus Mohrhard
Change-Id: Ic35f1188e66cf89f203b34f354a2c4b11f1f65a2
2015-02-19this should be [-90, 90] and not (-90, 90), tdf#89454Markus Mohrhard
Change-Id: I943be2cf660d2dc77eebd776208af96c0b5f67a4
2015-02-19start documenting problems around OOXML specMarkus Mohrhard
We implement in many places wrong default values based on how MSO 2007 behaves. Newer MSO versions produce correct files so our import is wrong. Instead of changing the default value we need to be able to use default values based on the document producer. Change-Id: I6a2a0b30885ebccc384d17f896709ecdd475d786
2015-02-18no need for that anymoreMarkus Mohrhard
Change-Id: I53002c0b97da66c44429b5ace0fc709b5e8a325f
2015-02-18better way to set default for chart area fill style, related tdf#89451Markus Mohrhard
Change-Id: Iee7fea0f55cf54d43a349b6cc0ffa25995069e40
2015-02-18implement fill based on style also for chart area, tdf#89451Markus Mohrhard
Change-Id: I7b488e222cd44e75d1e0ff7f8e545e9399f418b5
2015-02-17oox: use std::enable_shared_from_thisMiklos Vajna
Change-Id: I184a7a3579bebd81d5d81f64b4ace74bde8e87fb
2015-02-17fix indentationMarkus Mohrhard
Change-Id: Ifab81cb6e54e177aaf31aceeddc1b6cd3c0cc888 Reviewed-on: https://gerrit.libreoffice.org/14513 Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-02-10Properly check for Clang with static initializer_list bugStephan Bergmann
Change-Id: I98060f1adae0ba8ec03b2f0d6b0db6d5a1c0385c
2015-02-10It is apparently the static initializer_list that's confusing old ClangStephan Bergmann
...so drop "static" from all Clang for now (and revert back to OUStringLiteral) Change-Id: I33a3fca7f529005529f083d3e89ddfbd80bbc9bb
2015-02-09Some old Clang appear unhappy with the initializer_list<OUStringLiteral> codeStephan Bergmann
...so try whether initializer_list<OUString> would keep them happy Change-Id: I009eb601057066030addb409aa7996679d5d5922
2015-02-09oox: replace legacy DBG_WARNINGsMichael Stahl
Change-Id: If2c47bbd0c12cc006b38972c5f71eb00d3b03460
2015-02-07coverity#1268301 Dereference null return valueCaolán McNamara
Change-Id: I487009503866b6910049824efab987066dd72023
2015-02-07loplugin:deletedspecialStephan Bergmann
Change-Id: I37eb6c976779aeec829d312920ab1d18884c0f2f
2015-02-06fdo#75757: Remove inheritance from std::vectorMatthew Pottage
Deprecated comphelper/sequenceasvector.hxx. Rewritten code using it. Using instead the functions containerToSequence and sequenceToContainer, found in include/comphelper/sequence.hxx. One class that inherits from it (in framework/inc/stdtypes.h), and the code using that has been left. Signed-off-by: Michael Stahl <mstahl@redhat.com> Conflicts: writerfilter/source/dmapper/DomainMapper.cxx writerfilter/source/dmapper/TblStylePrHandler.hxx writerfilter/source/dmapper/WrapPolygonHandler.hxx Change-Id: Ice61c94ffb052e389281aebb7cdf185134221061
2015-02-06Make OUStringLiteral more usefulStephan Bergmann
...don't dare make it non-explicit, yet. Along the way, introduce SAL_CONSTEXPR. Change-Id: Ia3179d0d5e001fd7aa92237c97437e9b74366ee1
2015-02-06static const to avoid re-init all the time + c++11.Jan Holesovsky
Change-Id: I24ac6405f03e2d55f832d1ffe0c71e575883c5c8
2015-02-06bnc#637947 improve DrawingML export of custom shapesAndras Timar
Change-Id: Iaa880528cf3c899ce66e4349c6d989dfbe5cbeb6 Reviewed-on: https://gerrit.libreoffice.org/14346 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-02-05Updated all precompiled headers.Ashod Nakashian
Change-Id: I955c8ac4dbe002d23531df7eb10fb4444d6b5157 Reviewed-on: https://gerrit.libreoffice.org/14292 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-02-02oox: 89 DPI is not such a great defaultMiklos Vajna
Using the DPI from Application::GetDefaultDevice() is a much better idea, especially that now oox::GraphicHelper::GraphicHelper() and oox::drawingml::DrawingML::WriteSrcRect() are in sync. Should fix the testCropPixel() failure in CppunitTest_sw_ooxmlexport that appears on HiDPI systems. Also, fix all the rounding problems that now became visible when the DPI is the same for both import and export. Change-Id: Iceb34a8a5a1eaa8ce0824491521ad6b4d2f6949c Reviewed-on: https://gerrit.libreoffice.org/14280 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-02-02fdo#88826: replace MM100toEMU() by convertHmmToEmu()Juan Picca
Change-Id: I9ca79dfa0c214e3f58b7f45a78ad0877d4c7d10a Reviewed-on: https://gerrit.libreoffice.org/14266 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-01-30chisel off some more unused codeCaolán McNamara
Change-Id: I62cafc8e10e9c9dab3fde1785b826ba8ff68569f
2015-01-29callcatcher: large newly detected unused methods post de-virtualizationCaolán McNamara
i.e lots now able to be detected after... commit b44cbb26efe1d0b0950b1e1613e131b506dc3876 Author: Noel Grandin <noel@peralex.com> Date: Tue Jan 20 12:38:10 2015 +0200 new loplugin: change virtual methods to non-virtual Where we can prove that the virtual method is never overriden. In the case of pure-virtual methods, we remove the method entirely. Sometimes this leads to entire methods and fields being eliminated. Change-Id: I605e2fa56f7186c3d3a764f3cd30f5cf7f881f9d
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin
Where we can prove that the virtual method is never overriden. In the case of pure-virtual methods, we remove the method entirely. Sometimes this leads to entire methods and fields being eliminated. Change-Id: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae
2015-01-24coverity#1265791 Unchecked return valueCaolán McNamara
Change-Id: I6577fe7a9eeeaadd42db5fe667d0418af9899978
2015-01-21fdo#87488 Wrong text rotation inside a preset shape from docxGary Houston
Preset shape text rotation values are read in a few places and stored in moRotation. With these changes, moRotation is always read unchanged, but multiplied by -1 when it's used. Change-Id: I633d665c21daa69e15fa828a43300f10d2bf2054 Reviewed-on: https://gerrit.libreoffice.org/13561 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-01-20Some more loplugin:cstylecast: ooxStephan Bergmann
Change-Id: If2f8b9ff2577f2c8e9f15d1f174d50e5254fe7d8
2015-01-07fdo#84938: convert STREAM_ #defines to 'enum class'Noel Grandin
Change-Id: Ibbf14c7e9a5c1883c1311d4c86f948f74f8e473e
2015-01-03fdo#83058 File corrupt due to data label position is invalid in chart.xmlPriyankaGaikwad
- File getting corrupted after RT due to data label position is invalid in chart.xml - LO must not export invalid label position property if chart type doesn't support that positioning in MS Office. - For clusterd bar chart XML Difference chart1.xml Original file <c:barChart> <c:grouping val="clustered" /> <c:ser> <c:dLbls> ... <c:dLbl> <c:dLblPos val="outEnd" /> </c:dLbl> ... </c:dLbls> </c:ser> </c:barChart> Roundtrip file <c:barChart> <c:grouping val="clustered" /> <c:ser> <c:dLbls> ... <c:dLbl> <c:dLblPos val="r" /> </c:dLbl> ... </c:dLbls> </c:ser> </c:barChart> Change-Id: I9cae11b8d198aaa70fb05cb3bf329cd898519170 Reviewed-on: https://gerrit.libreoffice.org/11141 Reviewed-by: Kohei Yoshida <libreoffice@kohei.us> Tested-by: Kohei Yoshida <libreoffice@kohei.us>
2015-01-02boost::unordered_map->std::unordered_mapCaolán McNamara
you can get debug stl this way Change-Id: Ia70a3e7c7c452390e8bee34975d296c9318e4a19
2014-12-22ERROR: Invalid content was found starting with element 'p:pic'.Miklos Vajna
Again, as pointed out by CppunitTest_sd_export_tests. Sadly it seems that the spec and PowerPoint is not in sync here, so just do what makes both happy. This fixes the --with-export-validation build. Change-Id: I49e22d154d30106f91f1c972012f99a863784d23
2014-12-22ERROR: Attribute 'spid' must appear on element 'p:oleObj'.Miklos Vajna
As pointed out by CppunitTest_sd_export_tests. Change-Id: I8ecbc3ef4ea9dc84ebf88506967d02de04ded852
2014-12-22bnc#862510: PPTX import: Wrong text vertical adjustment inside shape.Zolnai Tamás
When alignment is not specified, it should be set to the default "left" value just on time, when properties are pushed to the property set, otherwise uninitialized value is better. Change-Id: Ic294c5d7cc477f7515aea1c071ad9b67275c012d
2014-12-22bnc#862510: PPTX import: Wrong text color inside shapeZolnai Tamás
When theme index is 0, it means it's unset so we should not apply that. Change-Id: I62a9cd2a9b4c19f5acffc334d5e8263fe24fc8fd
2014-12-22bnc#822341: PPTX export of embedded text documentsZolnai Tamás
-Make embedded text documents exported by analogy with spreadsheet embedded documents. -Convert MS ole objects to LO sepcific objecst to allow exporting those ole objects too. Plus use gb_CppunitTest_use_rdb instead of wasting time on finding out which missing components cause test failing. Especcially when we have platform dependent components (e.g. related to embedded object) Change-Id: Ic76d659b51274777a1669c215344a1169ebcba81
2014-12-22Refactor building perfect hash tabels for tokensDaniel Sikeler
moved the defines to be usable in more modules moved perl-script to be usable in more modules Change-Id: If893e4b7e9dc670646bdf097cb6fa5710e45f1ac
2014-12-19fdo#39440 reduce scope of local variablesMichael Weghorn
This addresses some cppcheck warnings. Change-Id: Idcedd908c653d5a5700884f233ad134dde8be018 Reviewed-on: https://gerrit.libreoffice.org/13540 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-19WaE: variable 'nModuleCount' set but not usedTor Lillqvist
Change-Id: If7893fcfbac978710c664cfd013910ece77cda59
2014-12-19defining a macro for two lines of code introduces more noiseNoel Grandin
.. than it solve Change-Id: Ic60021aa428289fffb26bc349425e9b8be520d0a
2014-12-19remove operator>> and operator<< methodsNoel Grandin
in favour of ReadXXX/WriteXXX methods Change-Id: I75fcb04cb9e2981a10146a96de27916ca7f6bc57
2014-12-19remove operator>> and operator<< methodsNoel Grandin
in favour of ReadXXX/WriteXXX methods Change-Id: I849fd10c689fb9834ae9974e430dc337adc68755
2014-12-18cppcheck: Same expression on both sides of '||'Julien Nabet
Change-Id: Ibdadac46483ca3b17a12e3d1ae31f75a869b8dbc
2014-12-18oox: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: I63db342676e1116c2ed1eef4e27599add0a318c9
2014-12-17Introduce rtl::OUStringLiteral1Stephan Bergmann
...to use single ASCII character literals "more directly" in the OUString API (instead of having to go via an intermediary OUString ctor call). Especially useful for character literals that are defined as const variables or via macros ("direct" uses of character literals in the OUString API can often simply be replaced with single-character string literals, for improved readability). (The functions overloaded for OUStringLiteral1 are those that are actually used by the existing LO code; more could potentially be added. The asymmetry in the operator ==/!= parameter types is by design, though---writing code like 'x' == s is an abomination that shall not be abetted.) Change-Id: Ic5264714be7439eed56b5dfca6ccaee277306f1f
2014-12-15Resolves: fdo#87281 Crash on importing diagram in .docx fileGary Houston
Change-Id: Ic2c7326ab83258e2e9dcbac88aa9f76f771a9b12
2014-12-15no point checking for null before deleteCaolán McNamara
Change-Id: I58cbb075a56518d71874137180ad92ae764ec4bf