Re: [PyX-devel] [PATCH 1/2] add hiresbbox parameter to epsfile.epsfile constructor

[Michael S. <m-s...@us...>]

Salut,

On 22/07/11, Michael J Gruber wrote:
> Joerg Lehmann venit, vidit, dixit 22.07.2011 15:24:
> > On 22.07.11, Michael J Gruber wrote:
> >> hiresbbox=0 is the default, unchanged behaviour.
> >> hiresbbox=1 means that the high resolution bounding box (HiResBoundingBox)
> >> is used rather than the normal one (BoundingBox).
> > 
> > Thanks! I wonder whether one should not do this automatically, i.e., if there
> > is a HiResBoundingBox present, use that one instead of BoundingBox.

+1 for defaulting to HiRes, with fallback to ordinary BB if HiRes not
present. In my opinion, the ordinary BB is the default only for
historical reasons.

BTW: If I compare inclusion of eps and pdf in latex/pdflatex, then eps
defaults to BB, and in pdf there is only a HiResBB. This made me use
always the HiResBB, also if I include eps in latex.

> Well, that would change existing behaviour, which would mean a no-go in
> another project I'm working with. That attitude somewhat stuck to me ;)
> 
> Note that with my implementation, hirebbox=1 even errors out when there
> is no hires bbox. This is intentional (user asks for it but there is
> none), but I don't mind changing it.

Cordialement,
  Michael