On Monday 21 July 2003 11:39 am, Andre Poenitz wrote:
> +     void absFilename(string const abs_filename);
> string const &?

Umm. Yes. Thanks.

> @@ -144,7 +144,10 @@ void InsetGraphicsParams::Write(ostream
>       // Do not write the default values
>
>       if (!filename.empty()) {
> -             os << "\tfilename " << MakeRelPath(filename, bufpath) << '\n';
> +             string const name = filename.saveAbsPath() ?
> +                     filename.absFilename() :
> +                     MakeRelPath(filename.absFilename(), bufpath);
> +             os << "\tfilename " << name <<  '\n';
>
> maybe this kind of logic could go into the FileName class somehow.
> It's used more than once and it could simplify 'client code' a bit...

Good idea.
Thanks for the feedback.
Angus

Reply via email to