summaryrefslogtreecommitdiff
path: root/xmloff
AgeCommit message (Collapse)AuthorFilesLines
2015-07-11write trailing text subformat also to Excel .xls and .xlsx, tdf#92457Eike Rathke1-3/+1
... without generating 0;;;@ from 0;@ that has different semantics. Introduce css::util::NumberFormat::EMPTY to properly flag empty subformats and distinguish from UNDEFINED, everything else would be an ugly hack. SvNumberformat::GetMappedFormatstring() now correctly supports the trailing text subformat, so exports using it should get that automatically. Change-Id: If9a1bcc5ec5dfcf46688035e2b1428ab4747a68d
2015-07-11store trailing text format in ODF, e.g. 0;@ tdf#92457Eike Rathke1-20/+51
A clumsy kludge, but since there is no "all others" condition.. Change-Id: Ie89b786585fdee6688f66f5a7af47ec84409eefa
2015-07-11the last condition is "all other numbers", not the third, tdf#92457Eike Rathke1-3/+3
Change-Id: Ic9571d938c4a8837c5712bafbfb2ebf9f32011df
2015-07-10do not include boost/utility.hppMichael Stahl1-1/+1
It just includes a bunch of other boost headers; mostly we need boost/noncopyable.hpp so include that directly. This eliminates 831 MB(!) of boost/preprocessor/seq/fold_left.hpp completely, which is the 2nd biggest header after ustring.hxx. Change-Id: I3df55770adcb46e56f389af828e8ba80da2dc1f2
2015-07-08Avoid loplugin:staticmethods in non-debug buildStephan Bergmann1-0/+2
Change-Id: I9f98e86556ed4435d24f49b32b848eb9f9a66008
2015-07-08Fix typosAndrea Gelmini1-1/+1
Change-Id: I020c7ce4d9aa93cb0292378a410cdb9f7788e920 Signed-off-by: Michael Stahl <mstahl@redhat.com>
2015-07-03Fix typosAndrea Gelmini16-18/+18
Change-Id: I98b4dfb0f72f315065693335e2f882bb2eed3afe Reviewed-on: https://gerrit.libreoffice.org/16713 Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-30coverity#1308431 wiggle code to see if this silences coverityCaolán McNamara1-2/+4
Change-Id: I44eae735b710332d87f2f642911e988c796457cd
2015-06-30Fix typosAndrea Gelmini2-2/+2
Change-Id: I1c5e4970d4e90bdea68a901294594104de5e189e Reviewed-on: https://gerrit.libreoffice.org/16582 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Andras Timar <andras.timar@collabora.com>
2015-06-28Fix typosAndrea Gelmini1-1/+1
Change-Id: I80995af6459af9f647e66d1c85da2eedad2647b9 Reviewed-on: https://gerrit.libreoffice.org/16523 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2015-06-28Fix typosAndrea Gelmini2-2/+2
Change-Id: Ifd126a535f18db79a18aaec61b76cdf26ab33dff Reviewed-on: https://gerrit.libreoffice.org/16522 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2015-06-27coverity#1308433 Uncaught exceptionCaolán McNamara2-2/+2
Change-Id: I32750eebbcd5a54d3767d77f4f4b8c54c0888078
2015-06-26loplugin:stringconstant: handle OUString+=OUString(literal)Stephan Bergmann3-8/+5
Change-Id: I44c30ef19661d3274a4131dd05a5c786263fae59
2015-06-25loplugin:stringconstant: Flag more inefficienciesStephan Bergmann1-1/+1
Change-Id: I81a60eb512356d19cb746145fdf0dc1a3c676ba1
2015-06-25remove our custom RTTI from xmloff moduleNoel Grandin60-198/+1
not being used, anyhow Change-Id: I0eb0d6377efe324d9a442a70fc2046a58ed2f3c4
2015-06-24Simplify uses of comphelper::string::equals with constant argumentStephan Bergmann1-1/+1
Change-Id: I7b12dd04f6811f71b776ee9168703434217b59c8
2015-06-23Revert "Typo: iff->if"Julien Nabet3-3/+3
This reverts commit cf92da3d6e1de14756efe3f1ee79f393a2f3787d. iff can mean "if and only if" so not a typo
2015-06-23Typo: iff->ifJulien Nabet3-3/+3
Change-Id: I3fc60856b5a56e71d70b55c89323be074bdec3b3
2015-06-23move sorting code closer togetherNoel Grandin1-88/+77
to make it easier to understand. Also switch iteration to range-based for loops. Change-Id: I39a3f565f40458fd5598bb2e97fe71a0fec1c09c
2015-06-22Fix typosAndrea Gelmini1-1/+1
Change-Id: I60ed5eb658d50cbc7dc572facb5463b7527b4d9b Reviewed-on: https://gerrit.libreoffice.org/16408 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-19tdf#42374 some small optimisations for opening this PDF fileNoel Grandin1-15/+13
makes it about 10% faster Change-Id: I145faed5aa7c312372f08cc651df5afcf10c70ab
2015-06-18Fix typosAndrea Gelmini7-8/+8
Change-Id: Ibfa971952cd330ddf430e7e951c5c235d2ae6bd8 Reviewed-on: https://gerrit.libreoffice.org/16314 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-06-17Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe2-3/+3
This may reduce some degree of dependency on boost. Done by running a script like: git grep -l '#include *.boost/scoped_array.hpp.' \ | xargs sed -i -e 's@#include *.boost/scoped_array.hpp.@#include <memory>@' git grep -l '\(boost::\)\?scoped_array<\([^<>]*\)>' \ | xargs sed -i -e 's/\(boost::\)\?scoped_array<\([^<>]*\)>/std::unique_ptr<\2[]>/' ... and then killing duplicate or unnecessary includes, while changing manually m_xOutlineStylesCandidates in xmloff/source/text/txtimp.cxx, extensions/source/ole/unoconversionutilities.hxx, and extensions/source/ole/oleobjw.cxx. Change-Id: I3955ed3ad99b94499a7bd0e6e3a09078771f9bfd Reviewed-on: https://gerrit.libreoffice.org/16289 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-06-16Fix typosAndrea Gelmini1-1/+1
Change-Id: Id8c7cf460cba5af89891c470e1aaa61b227d356e Reviewed-on: https://gerrit.libreoffice.org/16310 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-16Fix typosAndrea Gelmini1-1/+1
Change-Id: I528752dfabeb31d14c350f79819b521537ab9b56 Reviewed-on: https://gerrit.libreoffice.org/16300 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin5-58/+36
Idea originally from caolan. Found using the following command: find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;' Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
2015-06-15cppcheck:unreadVariableNoel Grandin1-2/+0
Change-Id: I03981ceba67280e8ed98a9add7f24b3bd958d522
2015-06-15Fix typosAndrea Gelmini1-1/+1
Change-Id: Id05ea8cf8aed878bedfe73c274df3c67b64bad37 Reviewed-on: https://gerrit.libreoffice.org/16263 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-14cppcheck: redundantCondition [part2]Julien Nabet2-3/+3
Change-Id: I23e8a4834fa7858adb292ce0a4dfa5dab1ab5f00
2015-06-13TyposJulien Nabet1-1/+1
Change-Id: I21c352a63d668c174eef212dbfbe6346c678ce4d
2015-06-11convert expressions like 'size() == 0' to 'empty()'Noel Grandin1-2/+2
Change-Id: Ia5c8c0f38a347f398d587970a22e03f29ffd37af
2015-06-11convert 'it.begin() == it.end()' to 'it.empty()'Noel Grandin1-1/+1
Change-Id: I244a9eb6bce6b1c649653ec38ebb9a39f8b4c145 Reviewed-on: https://gerrit.libreoffice.org/16212 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann14-28/+28
Change-Id: I6fa6ee6d9eaf98a00624ad353dc6d2c5687b0547
2015-06-08some spelling fixes, mostly in Java codeNoel Grandin1-1/+1
Change-Id: Ib9d0522a1f371404ee990348712e2d196049f69f
2015-06-04coverity#1302607 Logically dead codeCaolán McNamara1-4/+8
also coverity#1302606 Logically dead code Change-Id: I510f17e93f7caf5f1c35becdf6b7d626c0160b66
2015-06-04uno::Sequence provides now begin and endNoel Grandin3-6/+5
use begin() and end() when calling std::copy on an uno::Sequence Inspired by commit b34b648fc3262c5d9aa295f621e8fe9c97d4c6b2 "uno::Sequence provides now begin and end" Change-Id: I08e8c3fd6144e77b95a26f85bc0daf6a9edeeb0b Reviewed-on: https://gerrit.libreoffice.org/16057 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-02loplugin:cstylecast: deal with those that are (technically) const_castStephan Bergmann1-1/+1
Change-Id: Idd4f5a66667e2c13d2cb691960cfd1f568310674
2015-06-02cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I8ae8623252546ca94f65fc04b331dd9cafa4fc92
2015-06-01xmloff: don't add FIELD namespace twiceMichael Stahl1-6/+1
The first place already checks SvXMLExportFlags::CONTENT, so the second Add is unnecessary (and for versions <= 1.2 it shouldn't be exported). Change-Id: If2e471b50fcf21ad8977de48c433b628c5c03305
2015-05-29loplugin:loopvartoosmallNoel Grandin5-7/+7
Change-Id: Icb31e51575f7fffd36be73bbd87a3c5e56c3aa26
2015-05-28add field namespace to styles.xml when saving fdo85232-1.docx to .odtCaolán McNamara1-1/+7
when >= SvtSaveOptions::ODFVER_012 Change-Id: I0414bc4a83ae4787e10cd8a215555f249da7e65a
2015-05-24cppcheck: noExplicitConstructorCaolán McNamara51-82/+82
Change-Id: I8c9e970d5c5931de1a7526cbff3de626ef30bd74
2015-05-22tdf#91140: tweak fix a bit, turns out xmloff was also passing empty URLMichael Stahl1-1/+2
Change-Id: Ia6e1fbe18e72c9c06915e1b437076a1f56a6c206
2015-05-22kde250524-4.odt (etc) are missing some namespace declarations in meta.xmlCaolán McNamara1-1/+1
Change-Id: Ie6a86262f60a4d44423bf8673d3847594e25e296
2015-05-21loplugin:literaltoboolconversionStephan Bergmann1-1/+1
Change-Id: I4b9299ef87686fee4e2771255ed1fcbc16374db9
2015-05-20xmloff: convert legacy assertions in SvXMLNamespaceMapMichael Stahl1-9/+5
Change-Id: I9771bf3b727168bad6aeb07a2eb55eb08b1b6408
2015-05-20xmloff: convert legacy assertions in DocumentSettingsContextMichael Stahl1-10/+11
Change-Id: I2d13b43b6689e75185426854d749760f820884f9
2015-05-20xmloff: convert legacy assertions in PageMasterExportPropMapperMichael Stahl1-18/+16
Make the properties optional, like XMLTextExportPropertySetMapper already does. Change-Id: Iee5d8ac113360937d8f3a3b867a6e6c7a18bbfa4
2015-05-20xmloff: convert legacy assertions in XMLFootnoteSeparatorExportMichael Stahl1-10/+4
Change-Id: I4f9d429e055df0d7bc01e710334259d737d2d4f3
2015-05-18Related tdf#88056: use better-fitting <text:page-name>Katarina Behrens6-74/+72
Implemented Regina's suggestions: * this is a text field, therefore it belongs to text namespace * page-name (evtl. slide-name) describes much better what this field really contains This (among other changes) reverts commit 6609de8856519e0e9 Change-Id: Idab3b7c291839a9137f80d325a7d3f5ef0ff2636