Review Request: Few fixes to pathshape/enhancedpath

Lassi Taneli Nieminen lassniem at gmail.com
Mon Dec 27 21:17:14 GMT 2010


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

(Updated 2010-12-27 21:17:13.787173)


Review request for Calligra.


Changes
-------

The changes are to T, not to W as described earlier.


Summary (updated)
-------

Few fixes to pathshape/enhancedpath
1. In T the parameters are w/h, to get radiuses they should be divided by two
2. Allow T to have sweep angle to both directions as odf spec does not limit it to one direction.
3. The move after T had wrong y axis direction
4. When loading odf, load viewBox before evaluating path. This because path may have references to w,h,l,r,t,b and those are defined
from viewBox.

Feel free to ignore changes to rest of the files.


Diffs
-----

  filters/kpresenter/pptx/PptxXmlSlideReader.cpp 206ccdf 
  filters/libmsooxml/ComplexShapeHandler.h d1c78e3 
  filters/libmsooxml/ComplexShapeHandler.cpp 622af85 
  filters/libmsooxml/MsooXmlCommonReaderDrawingMLImpl.h eb03cf9 
  filters/libmsooxml/MsooXmlCommonReaderDrawingMLMethods.h d5732fc 
  filters/libmsooxml/MsooXmlCommonReaderImpl.h 138e797 
  filters/libmsooxml/MsooXmlCommonReaderMethods.h e99c747 
  filters/libmsooxml/generated/generatedShapes.h f8b3b2a 
  filters/libmsooxml/generated/generator.cpp 72e8c0a 
  filters/words/docx/DocxXmlDocumentReader.h d7fb2f5 
  filters/words/docx/DocxXmlDocumentReader.cpp 231a3e7 
  plugins/pathshapes/enhancedpath/EnhancedPathCommand.cpp 7fcd453 
  plugins/pathshapes/enhancedpath/EnhancedPathShape.cpp 2c26294 

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


Testing
-------


Thanks,

Lassi Taneli

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20101227/5ac8afdd/attachment.htm>


More information about the calligra-devel mailing list