[poppler] [PATCHES] Support for the NoRotate annotation flag

Adam Reichold adamreichold at myopera.com
Tue Apr 16 11:17:16 PDT 2013


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello Fabio,

Am 16.04.2013 19:50, schrieb Fabio D'Urso:
> FabioOn Sunday, April 07, 2013 08:10:32 PM  D'Urso wrote:
>> Modified patches attached. They are not final yet: I still have
>> to update the documentation in the second-to-last patch.
> 
> Final patches attached. Adam, can you have a look at 0006 and check
> if the documentation on FixedRotation is clear enough?

First of all, sorry for not posting anything on the rendering API yet,
despite your previous patch set. I hope to get something done on the
weekend.

I had a look at patch #6 and I think the documentation is rather helpful.

One thing I noticed in patch #5 is that you could maybe reuse MTX in
fillTransformationMTX, i.e. always call "fillNormalizationMTX( MTX,
pageRotate );" first and modify MTX using the QTransform if necessary
instead of first creating MTXnorm on the stack.

Best regards, Adam.

> Differences from initial patches:
> 
> 0001 (core: Support for rendering annotations with flagNoRotate) 
> The counter-rotation now does *not* depend on the rendering
> context. NoRotate annotations are always positioned as if rendering
> rotation was zero.
> 
> 0002 (poppler_qt4viewer: Add combobox to select rotation) and 0003
> (pdftops: Added "-rasterize" flag to force rasterization) These two
> patches have not changed. I had posted them because they were 
> useful to test context-dependent rotations, but I've removed such 
> rotations, therefore feel free to drop them if you wish.
> 
> 0004 (core: Remove geometry-related arguments from annotation 
> constructors) and 0005 (qt4: FixedRotation annotations' coordinate 
> conversion) No changes
> 
> 0006 (qt4: Some documentation about annotations) I've partly
> rewritten the doc section about FixedRotation annotations
> 
> 0007 (qt4: Free some temporary memory in 
> TextAnnotationPrivate::createNativeAnnot) No changes
> 
> Fabio
> 
> 
> 
> _______________________________________________ poppler mailing
> list poppler at lists.freedesktop.org 
> http://lists.freedesktop.org/mailman/listinfo/poppler
> 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)

iQEcBAEBAgAGBQJRbZWsAAoJEPSSjE3STU34Pu4H/1sm/aZbQ3kpgxWZbn4OJm9m
Sfoo3FOjxskpHW8LKF8vOHEF+JoDMiCqQhFp2dvfBqJDWu/LdtqY2/46U2DoRPXy
GP9XfZq+rs58ccwJfjtuChemtn2b5UQFwvs5YD9kNw9iVFTl9bw+NHutXxZWB3uu
9n+5K+GbDu345lUxXaCpYwnhORXylgEhkq8DgWh98n3iCaN1Km/a1duChShUUG14
ZBR7NEtSqem6qg9bGkaISEW23p3Y+pMW/WxtSbvdcWupbama+YcTNnJT0hyum53O
hR9PhdPIBqTj2jb5m18Vprf209cEqIPEAaMvAmGvwmJHyEzhp3UdPInZk0JbzEI=
=1xsd
-----END PGP SIGNATURE-----


More information about the poppler mailing list