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