James Turner wrote:
> On 27 Jul 2008, at 21:02, Tim Moore wrote:
>
>>> If you're doing that work, I'd like you to follow a *really*
>>> minimalist stance
>> i.e., if a header file only contains pointers or references to
>> another class or
>> includes them as arguments in uninstantiated templates (e.g.,
>> osg::ref_ptr<foo>), then that classe's include file should not be
>> sucked in; a
>> local class declaration should be made instead.
>>
>> This will generate more work for you, but is really the way to go in
>> terms of
>> reducing compiler dependencies.
>
> For pointers, a forward declaration works - for template, auto_ptr and
> osg::ref_ptr, I'm not totally clear what you mean. With auto_ptr, the
The header file that defines the template should probably be included, although
a template declaration should suffice. But the argument to the template need
only be forward declared.
> following can cause problems with virtual destructors not being called
> on older (MSVC6, I think) versions of visual studio:
>
> [the following is from memory of several wasted days a couple of years
> ago, stepping through the generated assembly of MSVC6. I think I have
> the particular case right]
>
> class foo;
>
> class bar
> {
> // no explicit dtor!
>
> auto_ptr<foo> mFoo;
> }
>
> class wibble : public foo
> {
> virtual ~wibble()
> {
> // .... highly important stuff
> }
> }
>
> {
> bar a;
> a.mFoo.reset(new wibble);
> // a goes out out scope, its compiler-generated dtor gets run,
> destroying mFoo
> }
> // error, wibble's dtor was not run
>
I'm not sure what's going on in your example, as foo needs to be defined
somewhere in order for wibble to inherit from it. Otherwise there's serious bug
there.
> If we're requiring a never MSVC than that, I believe we're fine. And
> perhaps you meant something else entirely?
I'm comfortable saying that we need a more recent compiler than MSVC 6.
Tim
>
>>> In C++ code they should be <cstring> and friends in order to ensure
>>> that the
>> functions are all declared in the std namespace.
>
> Okay, when I'm bored I'll do the monster patches to fix that.
>
> James
>
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Flightgear-devel mailing list
> Flightgear-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/flightgear-devel
>
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel
