summaryrefslogtreecommitdiff
path: root/svgio
AgeCommit message (Collapse)AuthorFilesLines
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
2014-10-24Beware gb_CppunitTest_use_library_files plus gb_CppunitTest_use_componentStephan Bergmann1-1/+2
...it causes the library's objects to be loaded twice, causing potential trouble with static data. So introduce gb_Cppunit_set_componentfile, processing the library's source component file into an additional workdir/CppuntiTest/ component file referencing the test library. (This required to drop the gb_Helper_abbreviate_dirs bric-a-brac, as it would do $W replacement into file URLs contained in the resulting .component file.) Change-Id: I57847e549817d25cca53a13effd7d408e0c99f9a
2014-10-22callcatcher: update unused codeCaolán McNamara2-9/+0
Change-Id: I96b4d44e44389245f3ccc4a08198676461c6a38b
2014-10-18cppcheck: Prefer prefix ++/-- operators for non-primitive typesJulien Nabet1-1/+1
Change-Id: I0dd2f1d2efef7995aaf2ab7656c24178a02eb9cd
2014-10-13Resolves: #i125325# added code to handle block comments...Armin Le Grand3-1/+65
in Css style definitions (cherry picked from commit 7b071b828a5f602cc30c17ddd871a75426a53faf) Conflicts: svgio/source/svgreader/svgtools.cxx Change-Id: I094228ea398686c5b39f5f279a964a2df7b00368
2014-10-13Resolves: #i125326# accept CssStyles with missing type attributeArmin Le Grand1-1/+17
(cherry picked from commit bb72d5b3ecbe6a03d3d3dfe101ad67ff4650a9f9) Conflicts: svgio/source/svgreader/svgdocumenthandler.cxx Change-Id: I5395c2cada92d5abcde9f6cc18bc237a58e77992
2014-10-09Resolves: #i125329# Take care of Css selector '*'Armin Le Grand1-0/+9
(cherry picked from commit f73813d9e0f13e3bdf735f8626dbf540701a1900) Conflicts: svgio/source/svgreader/svgnode.cxx Change-Id: Ifc5df8bed47d69709ef590eced19635b6b9580d0
2014-10-08Resolves: #i125332# made css style stuff aware that it is case independentArmin Le Grand30-82/+128
(cherry picked from commit 2b179313ac95f4ae9dc2cf8f6b6d577ea893c7a8) Conflicts: svgio/inc/svgio/svgreader/svgstyleattributes.hxx svgio/inc/svgio/svgreader/svgtoken.hxx svgio/inc/svgio/svgreader/svgtools.hxx svgio/source/svgreader/svgdocumenthandler.cxx svgio/source/svgreader/svgnode.cxx svgio/source/svgreader/svgstyleattributes.cxx svgio/source/svgreader/svgtoken.cxx svgio/source/svgreader/svgtools.cxx Change-Id: I8bac5a185feb0c7a54aa7bf816ab7aabd71a2e19
2014-10-05use isEmpty() instead of getLength() where appropriateChr. Rossmanith2-7/+7
Change-Id: I87bb3473af631dc321c5a8c2b06b0b5ccdf8e73b Reviewed-on: https://gerrit.libreoffice.org/11813 Reviewed-by: Christina Roßmanith <ChrRossmanith@web.de> Tested-by: Christina Roßmanith <ChrRossmanith@web.de>
2014-08-31Typo: childs->childrenJulien Nabet1-1/+1
Change-Id: Ib27b3e125581884b90efc9cc33b6b2aadab73851
2014-08-16SVG test: check content of Primitive2DSequence with the XML dumperTomaž Vajngerl2-13/+41
Change-Id: I12954cd52cece2d2fe99b7b4ca1e60cee3b430f0
2014-08-07loplugin:passstuffbyrefStephan Bergmann2-2/+2
Change-Id: Ib2b244f466d21835229a5a42e6decf4cf4aba653
2014-08-07loplugin:saloverrideStephan Bergmann1-1/+1
Change-Id: Ifbcd602203708519f7b4416cf31d94d45b552556
2014-08-07Related: #i125293# More unified (still simple) CssStyles and solversArmin Le Grand6-125/+250
(cherry picked from commit b760428400bbc7ab3db4d5de6239589e79981a06) Conflicts: svgio/inc/svgio/svgreader/svgdocument.hxx svgio/inc/svgio/svgreader/svgnode.hxx svgio/inc/svgio/svgreader/svgstylenode.hxx svgio/source/svgreader/svgdocument.cxx svgio/source/svgreader/svgnode.cxx svgio/source/svgreader/svgstylenode.cxx Change-Id: Ifcfc665df18f56e2cbc359411b633271c3b8d4bb
2014-08-02Change RectWithStylesByGroup to be same as other style testsTomaž Vajngerl2-19/+23
Change-Id: Ib7b0d2e8160a938821940645ea6e6471188bcc16