Commit aa848126cb6ceb00bd93e71d0ce53d93ee824897
1 parent
92fbc6fd
Comment about QPDFMatrix and QPDFObjectHandle::Matrix
Showing
2 changed files
with
4 additions
and
3 deletions
TODO
| @@ -97,8 +97,6 @@ Comments appear in the code prefixed by "ABI" | @@ -97,8 +97,6 @@ Comments appear in the code prefixed by "ABI" | ||
| 97 | * Consider having addArrayItem, replaceKey, etc. return the new value | 97 | * Consider having addArrayItem, replaceKey, etc. return the new value |
| 98 | so you can say | 98 | so you can say |
| 99 | auto oh = dict.replaceKey("/Key", QPDFObjectHandle::newSomething()); | 99 | auto oh = dict.replaceKey("/Key", QPDFObjectHandle::newSomething()); |
| 100 | -* Do something about QPDFObjectHandle::Matrix and QPDFMatrix. They | ||
| 101 | - should be the same thing. | ||
| 102 | 100 | ||
| 103 | Page splitting/merging | 101 | Page splitting/merging |
| 104 | ====================== | 102 | ====================== |
include/qpdf/QPDFObjectHandle.hh
| @@ -267,7 +267,10 @@ class QPDFObjectHandle | @@ -267,7 +267,10 @@ class QPDFObjectHandle | ||
| 267 | double ury; | 267 | double ury; |
| 268 | }; | 268 | }; |
| 269 | 269 | ||
| 270 | - // Convenience object for transformation matrices | 270 | + // Convenience object for transformation matrices. See also |
| 271 | + // QPDFMatrix. Unfortunately we can't replace this with QPDFMatrix | ||
| 272 | + // because QPDFMatrix's default constructor creates the identity | ||
| 273 | + // transform matrix and this one is all zeroes. | ||
| 271 | class Matrix | 274 | class Matrix |
| 272 | { | 275 | { |
| 273 | public: | 276 | public: |