summaryrefslogtreecommitdiff
path: root/svgio
AgeCommit message (Collapse)Author
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-07fdo#84938: convert STREAM_ #defines to 'enum class'Noel Grandin
Change-Id: Ibbf14c7e9a5c1883c1311d4c86f948f74f8e473e
2015-01-04boost::unordered_map->std::unordered_mapCaolán McNamara
Change-Id: I82f668ef72e916d2ff11df5cda2a02653999f66f
2014-12-18svgio: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: Ic47d4a855b948f399f67d051fa8dbe874b939676
2014-12-15svgio: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: Idce7768fd43453703ba758d9fddf26e0b7e5cc76
2014-11-20SVG: improve handling of visibility propertyChr. Rossmanith
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 Srivastava
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 Grandin
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 Grandin
Change-Id: I2be280ff8ebbf1046047a5bb4463191462172e24
2014-11-01SVG: handle visibility propertyChr. Rossmanith
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. Rossmanith
Change-Id: I2be2caae32a4b33facbf9e933f4537e12e576023
2014-10-30use isEmpty() instead of getLength() where appropriateChr. Rossmanith
Change-Id: I2ac660a6035791e68af9bde9470bbf8c1d008dee
2014-10-24Match gb_CppunitTest_CppunitTest,* and CppunitTest_*.mkStephan Bergmann
...so that "make CppunitTest_svgio" work. Change-Id: I7f9ad01c4e96f6b99b55531801970dc7a258389f
2014-10-24Beware gb_CppunitTest_use_library_files plus gb_CppunitTest_use_componentStephan Bergmann
...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 McNamara
Change-Id: I96b4d44e44389245f3ccc4a08198676461c6a38b
2014-10-18cppcheck: Prefer prefix ++/-- operators for non-primitive typesJulien Nabet
Change-Id: I0dd2f1d2efef7995aaf2ab7656c24178a02eb9cd
2014-10-13Resolves: #i125325# added code to handle block comments...Armin Le Grand
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 Grand
(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 Grand
(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 Grand
(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. Rossmanith
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 Nabet
Change-Id: Ib27b3e125581884b90efc9cc33b6b2aadab73851
2014-08-16SVG test: check content of Primitive2DSequence with the XML dumperTomaž Vajngerl
Change-Id: I12954cd52cece2d2fe99b7b4ca1e60cee3b430f0
2014-08-07loplugin:passstuffbyrefStephan Bergmann
Change-Id: Ib2b244f466d21835229a5a42e6decf4cf4aba653
2014-08-07loplugin:saloverrideStephan Bergmann
Change-Id: Ifbcd602203708519f7b4416cf31d94d45b552556
2014-08-07Related: #i125293# More unified (still simple) CssStyles and solversArmin Le Grand
(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ž Vajngerl
Change-Id: Ib7b0d2e8160a938821940645ea6e6471188bcc16
2014-07-30error: reference to "Primitive2DSequence" is ambiguousMatúš Kukan
Change-Id: Ic01f84b2603c93985f0b28fc9aced7eeac941b53
2014-07-29SVG test that sequences are equalJoren De Cuyper
Change-Id: I58178ac9942dc52083dd44565669769761652ada Reviewed-on: https://gerrit.libreoffice.org/10601 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-26drop rtl:: and staticsCaolán McNamara
Change-Id: I6108efad77cf1c21dea3826ae149f3750552b0ed
2014-07-26Related: #i125293# Further corrections for CssStyle handling in SVG importerArmin Le Grand
(cherry picked from commit f00d6d2ea3e73e687edb1da4c32c0960eaf5010b) Conflicts: svgio/source/svgreader/svgnode.cxx svgio/source/svgreader/svgstyleattributes.cxx svgio/source/svgreader/svgstylenode.cxx svgio/source/svgreader/svgsvgnode.cxx Change-Id: I335b292fcdf1ebac925c9fd76ad624efb923447d
2014-07-25fdo#48065 SVG: correctly clamp fill-opacityJoren De Cuyper
See W3 test http://www.w3.org/Graphics/SVG/Test/20061213/htmlObjectHarness/full-painting-fill-05-b.html which is also attached to the bug document. This fixes the behavior of the insert > image > from file behavior. The open behavior is already fixed by 79a6e40e6f19a896dbc25640deb3d4507eddad95 Change-Id: I9e3a3755bac25ca596d418c229a78cb23e07d3f3 Reviewed-on: https://gerrit.libreoffice.org/10522 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-07-24Related: #i125292#, #i125293# enhanced CssStyle handling in SVG importArmin Le Grand
(cherry picked from commit 7fc272837995079247594e173d977a2e541a7042) Conflicts: svgio/source/svgreader/svgnode.cxx Change-Id: I1896d9859e07a8edfb46ede6a237efa3edce5ab1
2014-07-22Fix build: incorrect revertion svgioJoren De Cuyper
Incorrect revertion in commit 3d3401a6397e893808309ec374f5d8f890144906 Change-Id: If4abac2670b083ebbc75c2fe06ae9e100072ef00 Reviewed-on: https://gerrit.libreoffice.org/10466 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-07-22Revert "fdo#74743 avoid infinite loop when gathering "svg" element styles"Joren De Cuyper
This reverts commit 3b7472b284131c09d91b69f26d5d26d54648f939. Commit 0879a639bc7c734f0847f74b965809f9107b3195 didn't revert all changes, so we ran into problems getting the css style correctly. Conflicts: svgio/source/svgreader/svgstyleattributes.cxx svgio/source/svgreader/svgsvgnode.cxx Change-Id: I4d17c6d599482de005c009638d03ca851491f700 Reviewed-on: https://gerrit.libreoffice.org/10455 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-16Resolves: #i125258# reworked some of the style hierarchy stuffArmin Le Grand
(cherry picked from commit 3b13e15a7174f5177700fdcd4864b64fbf0b3535) Conflicts: svgio/inc/svgio/svgreader/svgstylenode.hxx svgio/source/svgreader/svggnode.cxx svgio/source/svgreader/svgstyleattributes.cxx svgio/source/svgreader/svgstylenode.cxx svgio/source/svgreader/svgsvgnode.cxx Change-Id: I5cfe6871ab235305f206d83d643884b493901dfe
2014-07-15Avoid infinite loop when gathering "g" element stylesJoren De Cuyper
Same way of how fdo#74743 is fixed 3b7472b284131c09d91b69f26d5d26d54648f939 Change-Id: If6cc8eb6ff89b0081f27ff75a9b60a3e81cff1bd Reviewed-on: https://gerrit.libreoffice.org/10326 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-15Cleanup of switch case flowRoot in svg endelementJoren De Cuyper
We don't need this switch case due the fact we already check it before the switch itself. I did introduce this in commit 5bd241b99b76ae7f4b3c1d4f2bcbaf7c487bb339 but it is redundant. Change-Id: I24e8c946ef8b8f094daa248c701772e54790c9d1 Reviewed-on: https://gerrit.libreoffice.org/10317 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-15fdo#50114 ingore flowRoot element during svg importJoren De Cuyper
This element is not specified in any released svg specification. It might be mentioned in svg1.2 but since this is not yet released/ or will even not be released ever -> ignore it. Change-Id: Iaf5a392893070fda9e7a420bc951c8565bcfb37f Reviewed-on: https://gerrit.libreoffice.org/10312 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-14fdo#78554 - SVG: Text from SVG no longer displayed in LibreOfficeJoren De Cuyper
regression of 3b7472b284131c09d91b69f26d5d26d54648f939 Change-Id: I95a30acbf4b2684dda9684f5b51b887356a940e1 Reviewed-on: https://gerrit.libreoffice.org/10302 Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2014-07-04Resolves: #i124852# Corrected mask and clip polygons for userSpaceOnUseArmin Le Grand
(cherry picked from commit 247d5bc4d13c8ab299f8ca72768946f59815dca4) Change-Id: I328a55e940db64d3dad04902bf81ede56a0dfe73
2014-06-30Fix svgio buildmatteocam
Change-Id: Ief84952f2b1dc077e871e42e419f41d99810af26
2014-06-13loplugin:staticcallStephan Bergmann
Change-Id: I7f26acf82f0b945bf49449dd082f79ea5b5b9d1b
2014-06-04update_pch: add a bunch of pch filesThomas Arnhold
desktop: 1m51s -> 54s framework: 1m55s -> 1m04s package: 32s -> 16s sdext: 1m31s -> 47s svgio: 32s -> 15s uui: 49s -> 20s vbahelper: 1m44s -> 27s xmlscript: 15s -> 10s xmlsecurity: 45s -> 23s Change-Id: Ia437969c091bf877983ababc5ea2d044bbc0bee0
2014-06-04DeInitVCL at end of testsStephan Bergmann
This required some changes to the framework: * Init-/DeInitVCL is no longer done per individual test in BootstrapFixture, but once per CppunitTest invocation in a new vclbootstrapprotector (similarly to the exisiting unobootstrapprotector). CppunitTests that need VCL now need to declare gb_CppunitTest_use_vcl. * For things to work properly, the UNO component context needs to be disposed from within DeInitVCL (cf. Desktop's Application::DeInit called from DeInitVCL). The easiest solution was to introduce an Application::setDeInitHook (where the hook is called from DeInitVCL) specifically for vclbootstrapprotector to call. * PythonTests don't (yet) call DeInitVCL; they still hook into BootstrapFixture's original test_init functionality (to call InitVCL), and do not make use of the vclbootstrapprotector. Change-Id: I4f3a3c75db30b58c1cd49d81c51db14902ed68b2
2014-05-29cppcheck: consecutive return/break/...Julien Nabet
Change-Id: I171df68539fc41046b706157c04ab1e8cc1e60ca
2014-05-16fix-includes.pl: svgioThomas Arnhold
Change-Id: I660f0b2da8f5e6fd8de37e22fd4edd2bd8d21ef8
2014-05-16clean up, remove eof commentsThomas Arnhold
Change-Id: I166e781cb54181a8e5d08021cf5f6dfcd1fde5cb
2014-05-15Resolves fdo#70681: fixincludeguards.pl: all that's leftThomas Arnhold
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5 Reviewed-on: https://gerrit.libreoffice.org/9360 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2014-05-01Update filter README and add a new one.Julien Nabet
See http://nabble.documentfoundation.org/About-Svgreader-td4107050.html Thank you Christina for these information Change-Id: I56c4f5fdeec91e7e35990bffcf19451baa541552