On Thursday 18 April 2002 6:15 pm, Marco Morandini wrote:
> Angus Leeming wrote:
> > On Thursday 18 April 2002 5:03 pm, Marco Morandini wrote:
> >
> > In ControlGraphics.C, are you sure you need to #include "converter.h"?
>
> It defines the class Format and declares the external formats variable.
>
> > and #include "insets/insetexternal.h" ?
>
> No, I don't need it. I've added it while I was lookig for a method to
> get the formats variable.
>
> > You're right about the ugliness of the "|" stuff. Really we should pass
> > the vector to browseRelFile and let it deal with this. For now though,
> > you should use:
> >
> > #include "support/lstrings.h"
> >
> > string const answer =
> > "*.(" + getStringFromVector(browsable_formats, " | ") +"|gz)|";
> >
> > I think. (Don't forget the gz).
>
> Thanks.
> "|" and not " | ", or it will not work.
>
> > Could you test it for me?
>
> Seems to work.
>
>
> Thanks again,
>
> Marco
And thank you. Patch verified and applied.
Angus
