* Dekel Tsur <[EMAIL PROTECTED]> [010729 19:02]:
> On Sun, Jul 29, 2001 at 02:42:48PM +0300, Baruch Even wrote:
> > Adds the scale method as request by Michael.
> > Fixes the EPS detection part, non hacky revision (Garst please test).
> > 
> > The scale option is added to the horizontal part but affects both
> > horizontal and vertical, bad UI, I know. Consider this a proof of
> > concept, and if someone won't do the UI fix I'll do it some other time.
> 
> Looking at the patch, it seems that if you have foo.epsi, lyx will try to
> convert it to foo.eps ...

The latest patch will avoid this since it will detect it to be an eps
file and avoid converting it, the detectFormat function will return the
same suffix and so the if (suffix == fmt_to_convert_to) return; will
just return with no conversion.

> Also, why did you define epsi format in your previous patch ?
> One argument in Converter::Convert is the format of the from_file, so
> it can be used as follows:
>    converters.Convert(..,"foo.epsi",..., "eps", "pdf")

gotcha, can you finally write doc strings for converter?

Everytime I need to use it and want to know how to do something I need
to read the goddamn code, and it's not that easy to follow.

If you'll document it's features and how to use it I'll know how to use
it better and will not need such hacks.

-- 
Baruch Even
http://baruch.ev-en.org/

Reply via email to