[poppler] Add trimbox support to pdftops

Albert Astals Cid aacid at kde.org
Thu Oct 28 13:16:23 PDT 2010


A Dijous, 28 d'octubre de 2010, Benjamin Adler va escriure:
> Hello!
> 
> I created a first patch to get what I need, and it *seems* to work fine.
> I'm not a C++ expert and haven't touched poppler before, so I'm prepared
> to work on the patch a little more; yes, I'll remove all the useless
> comments and debug output before sending a real patch.
> 
> Please tell me what you think!

 * Use capital first letter for the enum values.
 * Don't call the variables cropBox since it seems it contains the "cropBox", 
call them boxToCrop or something like that
 * It also would be cool if you detected -cropbox and -crop MediaBox together 
and told the user he is wrong and needs to fix his parameters.

Overval looks ok (though please send clean patches because all that //ben 
comments just make everything much harder to read)

Albert



> 
> cheers,
> ben


More information about the poppler mailing list