Review Request: make parsing of fo:clip more strict, but also backward compatible

Friedrich W. H. Kossebau kossebau at kde.org
Thu Jul 12 19:52:51 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105528/
-----------------------------------------------------------

Review request for Calligra, Inge Wallin and C. Boemann.


Description
-------

As can be seen e.g. in odf/odt/OpenDocument-v1.1.odt from the calligratests, some OOo versions did not use commas as separators for the offset values in the value "rect(...)" for the attribute "fo:clip". So the parsing fails, for the given document e.g. with
--- 8< ---
KoUnit::parseValue: KoUnit::parseValue: Unit  "cm0cm0cm0cm"  is not supported, please report.
--- 8< ---

And while I created the attached patch with a workaround I changed the code to also:
    * detect "auto" values in rect(...)
    * check number of values in rect(...)
    * handle negative values in rect(...)

Okay to backport to 2.5?


Diffs
-----

  libs/flake/KoOdfWorkaround.h 18b801f 
  libs/flake/KoOdfWorkaround.cpp 9b8fce9 
  plugins/pictureshape/PictureShape.h 3937573 
  plugins/pictureshape/PictureShape.cpp 40c609b 

Diff: http://git.reviewboard.kde.org/r/105528/diff/


Testing
-------


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120712/4bec1a8d/attachment.htm>


More information about the calligra-devel mailing list