summaryrefslogtreecommitdiff
path: root/svgio
AgeCommit message (Collapse)AuthorFilesLines
2016-01-10Fix typosAndrea Gelmini4-5/+5
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2015-12-11tdf#69977: uno::Sequence is expensiveNoel Grandin40-240/+228
when used as a mutable data-structure. Plain std::vector halves the time taken to display the chart dialog Create a class to represent the std::vector we are going to be passing around, and move some of the utility methods into it to make the code prettier. Also create an optimised append(&&) method for the common case of appending small temporaries. Change-Id: I7f5b43fb4a8a84e40e6a52fcb7e9f974091b4485
2015-11-15Fast PCH generator and optimized PCH filesAshod Nakashian1-50/+46
Ported update_pch.sh to Python with improved performance and features. The new script is invoked from the same update_pch.sh which calls it for each library in parallel, although it can be invoked directly. The ported script (update_pch) updates all PCH files in ~15 seconds where the old script took ~4500 seconds. In addition, the new script supports 3-tiered headers (system, module, and local) and is very flexible to support other improvement. It has a per-library optimal configuration settings that can be updated using another new scripts (update_pch_autotune.sh) which finds optimal per-PCH settings. PCH files have been generated using the new scripts which builds significantly faster (2-3x, depending on module and configuration) and the intermediate binaries are noticably smaller (by several GBs). The new script stamps each generated PCH file with the command that generated it to make it trivial for users to update them, and also adds the command to invoke another script (update_pch_bisect) that helps find missing headers or conflicting headers that may break the build after updating the PCH. Finally update_pch has built-in unit-tests for makefile parsing and other core functionality. Change-Id: Ib933b50e50374d7e2e7e3e95ba8799b0cc8a27fa Reviewed-on: https://gerrit.libreoffice.org/19965 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann42-115/+115
Change-Id: I0164a67073395b865fa3e3ce342665f6497dc5b8
2015-11-09new loplugin: oncevarNoel Grandin1-3/+1
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann14-14/+14
Change-Id: I987fa3f29cb578e44e4022b6c33ae21a0094bfd8
2015-11-02SVG: add import test for tdf#87309Xisco Fauli2-0/+19
Change-Id: I0f899fbab73ae43e09ca2afc18d956f42ec8b0a7 Reviewed-on: https://gerrit.libreoffice.org/19537 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
2015-10-29com::sun::star->css in starmath,stoc,svgio,svlNoel Grandin7-23/+21
Change-Id: If4308b358a55351f6e951ebf055df076ce4ad4ce Reviewed-on: https://gerrit.libreoffice.org/19667 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-20loplugin:defaultparamsStephan Bergmann7-19/+19
Change-Id: Idad9ea385a9b83ad88792cc489413eeb6e6fcd9b
2015-10-14make test more readable - use Primitive2DSequenceTomaž Vajngerl1-10/+9
Change-Id: I8e49fce5a4d8f5b46b9bd1dd3f71a8c7dc90c20d
2015-10-13cppcheck:variableScopeNoel Grandin1-7/+6
Change-Id: I7cbd5a9e9bb5417f754d4e2445df309140fd40af Reviewed-on: https://gerrit.libreoffice.org/19329 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann29-80/+80
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-09-30Fix typosAndrea Gelmini3-4/+4
Change-Id: Ic3d2171521e308261c354d8f699b44b26c95f1ea Reviewed-on: https://gerrit.libreoffice.org/18978 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-30Fix typosAndrea Gelmini1-1/+1
Change-Id: Icd2055ee1f941383d0338ea3515a9fbb6db1712a Reviewed-on: https://gerrit.libreoffice.org/18979 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-15Resolves: tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe1-1/+2
with the variadic variants. Change-Id: I7154f9472f02fdf47d27ba715db55bb1ec669a8a Reviewed-on: https://gerrit.libreoffice.org/18580 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-29o3tl/cow_wrapper: remove boost dependencyDaniel Robertson1-0/+1
Remove boost dependencies from ::o3tl::cow_wrapper, and add the necessary includes to files including checked_delete and noncopyable that do not already include the necessary files. Change-Id: Iedae4232002792724226829a5d5cf9d16ffd0686 Reviewed-on: https://gerrit.libreoffice.org/18125 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-08-20loplugin: defaultparamsNoel Grandin5-14/+14
Change-Id: Ibf0ea539d2a95073a12f84964759d9c7a94baa7b
2015-07-16loplugin:unusedmethods svgioNoel Grandin9-21/+1
Change-Id: I0dd601429b70dc09780e31079a6f7c0570652fe9 Reviewed-on: https://gerrit.libreoffice.org/17114 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-15cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: Id438b987f72ae57bd4fa882e01fba17d3fa5b95b
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-06remove some unnecessary typedefs around uno::ReferenceNoel Grandin1-8/+8
Not very useful these days, since we have the css:: prefix to reduce verbosity. Found with a search like: git grep -nP 'typedef .*uno::Reference.*<.*>\s+\w+;' | grep -v 'std::' Change-Id: I16f41c53837ec5c613545ae45df38b0daddcbadd Reviewed-on: https://gerrit.libreoffice.org/16730 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-04Fix typosAndrea Gelmini2-2/+2
Change-Id: Ic6415423f46aaee7ba90239a617c318cf92ae222 Reviewed-on: https://gerrit.libreoffice.org/16711 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-26loplugin:stringconstant: handle OUString+=OUString(literal)Stephan Bergmann1-1/+1
Change-Id: I367a880669138406e715e8574543b1d54754b0ba
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-17Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe1-2/+2
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-15remove unnecessary check for null when calling deleteNoel Grandin17-22/+22
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-15Fix typosAndrea Gelmini2-4/+4
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-13Typo: corected->correctedJulien Nabet1-1/+1
Change-Id: I9a41d801b8807ed122a6737a924ba6792b449c45
2015-06-04convert B2DLineJoin to scoped enumNoel Grandin1-3/+3
Change-Id: I4fb8f2c9e2080e3aa542d2ccf838e6784baa8429
2015-06-01loplugin:loopvartoosmallNoel Grandin3-6/+6
Change-Id: I86ff38a90018a2ddfb2db3babf67168b0e6257a5
2015-05-05loplugin:staticmethodsNoel Grandin2-3/+3
Change-Id: Iea68c4e466dd8e1cb7164959a680e5e52542166b
2015-03-27fixed typos in commentsChr. Rossmanith1-3/+3
Change-Id: I6f0b90f7a76ddc69e158502af760466ab7ce2f91
2015-03-25TyposJulien Nabet1-1/+1
Change-Id: I434be0034b26ac4719590238817538ecd90b8923
2015-03-17tdf#89119: SVG - accelerated import of huge paths without fillChr. Rossmanith1-2/+6
the sample attached to the bug can be rendered within seconds instead of minutes Change-Id: I64671e9619f78931da2c11461a6884f2a1c8d66f Reviewed-on: https://gerrit.libreoffice.org/14883 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Christina Roßmanith <ChrRossmanith@web.de>
2015-03-08TyposJulien Nabet1-1/+1
Change-Id: Ibc378fa5f515de61bb768b4ef082638b40c94e00
2015-02-26TyposJulien Nabet1-1/+1
Change-Id: Ic03fb2ee6470eac6034be13908dd9ec92cf175b6
2015-02-06tdf#87309: SVG - use black as default currentColor if no color was specifiedChr. Rossmanith2-4/+16
improves import of attachment https://bugs.documentfoundation.org/attachment.cgi?id=110836 from issue 62284 Change-Id: Icb056774746ce270138a54e9dfe9ca6987a64769 Reviewed-on: https://gerrit.libreoffice.org/14242 Reviewed-by: Katarina Behrens <bubli@bubli.org> Tested-by: Katarina Behrens <bubli@bubli.org>
2015-01-26new loplugin: change virtual methods to non-virtualNoel Grandin2-3/+3
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-07fdo#84938: convert STREAM_ #defines to 'enum class'Noel Grandin2-2/+2
Change-Id: Ibbf14c7e9a5c1883c1311d4c86f948f74f8e473e
2015-01-04boost::unordered_map->std::unordered_mapCaolán McNamara6-9/+8
Change-Id: I82f668ef72e916d2ff11df5cda2a02653999f66f
2014-12-18svgio: Use appropriate OUString functions on string constantsStephan Bergmann1-4/+1
Change-Id: Ic47d4a855b948f399f67d051fa8dbe874b939676
2014-12-15svgio: Use appropriate OUString functions on string constantsStephan Bergmann2-7/+3
Change-Id: Idce7768fd43453703ba758d9fddf26e0b7e5cc76
2014-11-20SVG: improve handling of visibility propertyChr. Rossmanith2-14/+15
visible children of an invisible parent are now visible Change-Id: I2eafbd15d22f54ec39f12bfd32175925ab8a6184 Reviewed-on: https://gerrit.libreoffice.org/12504 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava1-1/+4
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-05fdo#38835 strip out OUString globalsNoel Grandin4-166/+165
they are largely unnecessary these days, since our OUString infrastructure gained optimised handling for static char constants. Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
2014-11-03svgio: two of the token names were not being mapped to tokensNoel Grandin1-0/+2
Change-Id: I2be280ff8ebbf1046047a5bb4463191462172e24
2014-11-01SVG: handle visibility propertyChr. Rossmanith5-8/+56
improve import of masking-path-08-b.svg of the W3C svg test suite when using insert->media->image Change-Id: Ib4d48165f982e394e2171ac82e6cc06911553904 Reviewed-on: https://gerrit.libreoffice.org/12168 Reviewed-by: Christina Roßmanith <ChrRossmanith@web.de> Tested-by: Christina Roßmanith <ChrRossmanith@web.de>
2014-10-30fixed typoChr. Rossmanith1-1/+1
Change-Id: I2be2caae32a4b33facbf9e933f4537e12e576023
2014-10-30use isEmpty() instead of getLength() where appropriateChr. Rossmanith1-2/+2
Change-Id: I2ac660a6035791e68af9bde9470bbf8c1d008dee
2014-10-24Match gb_CppunitTest_CppunitTest,* and CppunitTest_*.mkStephan Bergmann1-12/+12
...so that "make CppunitTest_svgio" work. Change-Id: I7f9ad01c4e96f6b99b55531801970dc7a258389f