On 13 Feb 2001, Lars Gullik Bjønnes wrote:
> Lior Silberman <[EMAIL PROTECTED]> writes:
>
> | On Tue, 13 Feb 2001, Allan Rae wrote:
> |=20
> | > On 12 Feb 2001, Lars Gullik Bj=F8nnes wrote:
> | >=20
> | > > John Levon <[EMAIL PROTECTED]> writes:
> | > [...]
> | > > | 3) stl_string_fwd.h is old and can be deleted ?
> | > >
> | > > IMHO yes. The use of this non-standard header is a trick to reduce
> | > > compile times.
> | >=20
> | > This is used to allow use with STLport and SGI's STL. Without it those
> | > compiles fail when you try to use our string implementation and their S=
> TL.
> | > So I suppose you could say it reduces compile times from infinity down =
> to
> | > sometime today. Nothing in LyX includes it, it is the STL implementati=
> on
> | > that finds it. That's why it appears to be unused. It's old because it
> | > hasn't needed fixing.
> | >=20
> | > Allan. (ARRae)
> | >=20
> | >=20
> |=20
> | With a cross-platform project of this scope, there are bound to be
> | such easter eggs all over the place. But so many people are looking at the
> | code that source documentation is a must.
> |=20
> | I suggest taking the above 2 sentences (This is used ... and their STL)
> | and plugging them into a comment at the beginning of the file (unless
> | someone has a better wording). Perhaps also add "There is no need to
> | include this file, rather it is found by the STL implementation".
>
>
> But we really don't want to support STLport with our lyxstring we want
> to use theirs...
>
> Lgb
>
I am not qualified to discuss this last point, but I think that as long as
we keep the stl_string_fwd.h file, it should carry some explanation
(unless it's going to be abolished tomorrow)
Lior.
