Jean-Marc Lasgouttes wrote:

>>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:
> 
> Angus> I dug this out of my store of things half-implemented. As you
> Angus> can see from the SigC references, it's very old ;-)
> 
> Angus> The idea was to prevent the code in src/frontends from looking
> Angus> in the different GUI dirs. Would anybody be upset if I brought
> Angus> this up to date and applied it?
> 
> Hmm, what is it designed to replace?

frontends/Timeout.C:
        #include "Timeout_pimpl.h"

That is a very nasty way of specifying a common interface. It means that 
compilation of Timeout.C now depends on GUI-specific code.

The language has the concept of abstract base classes for exactly this 
problem and is the 'proper' way to specify an interface. 

Anyway, I took the silence as consent and just committed it...

-- 
Angus

Reply via email to