John Levon <[EMAIL PROTECTED]> writes:

| This patch removes two unused methods for BV, and makes workarea_ use
| a scoped_ptr. This is necessary in order to use the virtual constructor
| factory idiom later, ie. in the GUII patch, a method is called into the
| frontends toolkits, which returns a WorkArea * which is really an
| XWorkArea * or QWorkArea *.
>
| This seems to be the simplest way of doing things, and doesn't have any
| obvious disadvantages imo (we already do this for LyXView).
>
| OK to apply ?

I am not sure why we need this to have XWorkArea and QWorkArea.

Why cannot a WorkArea_pimpl be used for this? Or in this case a gui
impl. I am wary of using pointer when we do not  have to.

-- 
        Lgb


Reply via email to