Lars Gullik Bjønnes wrote:
> | Do people think this is a clean solution? Should I move the class into
> | its own files rather than putting it in support/filetools.[Ch]?
> 
> Well... I think the boost::path stuff would do this...
> 
> | Comments please.
> 
> boost::filesystem.

My proposal:
        replace a string with a {string, bool} struct.
Your proposal:
        replace all of the lyx::support file stuff with boost filesystem.

Whilst your proposal is definitely valid, I prefer my incremental, one thing 
at a time approach. Let's get all references to external files as something 
type safe (ie, not a string) and then proceed.

-- 
Angus

Reply via email to