fine. just be sure tests work.
 
thanks gwurk, again :)




> Date: Tue, 29 Jan 2008 10:03:34 +0100
> From: mstsxfx@gmail.com
> To: gwurk@figpost.com
> CC: pdfedit-devel@lists.sourceforge.net; misutkajunior@hotmail.com
> Subject: Re: CropBox should default to MediaBox if not specified
>
> 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



Need to know the score, the latest news, or you need your HotmailŪ-get your "fix". Check it out.