Re: CropBox should default to MediaBox if not specified

[Michal H. <ms...@gm...>]

On Mon, Jan 28, 2008 at 09:40:43PM -0500, Gwurk wrote:
> Hi again,

Hi.

> 
> In cpage.cc, you use the default A4 rectangle for the CropBox if the
> CropBox is not specified. However, the CropBox should default to the
> MediaBox instead. (See section 3.6.2, subsection "Page Objects" in the
> PDF spec.)

You are right. I am not sure but I think that this function was done by
me. I have checked specification down to version 1.3 and all of them
states the same that default for crop box (if not present) is media box
so don't understand the comment that specification is unclear in that
way...

> 
> Can fillInheritedPageAttr() be changed to use a copy of the MediaBox
> for the CropBox if the CropBox isn't specified?

See attached patch.
Jozo, can you review and give it ack?

> 
> Thanks!

Thanks for reporting.

-- 
Michal Hocko