Archived
This forum has been archived. Please start a new discussion on GitHub.
Demos from the former book (now Wiki) use virtual inheritance for no reason
Hi,
A co-worker noticed that we use virtual public inheritance from Ice::Application in our non-IceBox applications. I checked your documentation and noticed that the examples from the former book (now part of the wiki) all use virtual public inheritance from Ice::Application like in:
You can find those in the source tree
as well as in the Wiki, e.g.:
Example of a File System Server in C++ - Ice 3.5 - ZeroC
Since there is no reason to use virtual here I'd suggest to update the documentation.
Cheers,
Michael
A co-worker noticed that we use virtual public inheritance from Ice::Application in our non-IceBox applications. I checked your documentation and noticed that the examples from the former book (now part of the wiki) all use virtual public inheritance from Ice::Application like in:
class FilesystemApp : virtual public Ice::Application { ... };
You can find those in the source tree
grep -R virtual * | grep Ice::Application cpp/demo/book/simple_filesystem/Server.cpp:class FilesystemApp : virtual public Ice::Application cpp/demo/book/map_filesystem/Server.cpp:class FilesystemApp : public virtual Ice::Application cpp/demo/book/lifecycle/Server.cpp:class FilesystemApp : virtual public Ice::Application cpp/demo/book/evictor_filesystem/Server.cpp:class FilesystemApp : virtual public Ice::Application
as well as in the Wiki, e.g.:
Example of a File System Server in C++ - Ice 3.5 - ZeroC
Since there is no reason to use virtual here I'd suggest to update the documentation.
Cheers,
Michael
0
Comments
-
Hi,
Thanks, we will fix this.
Cheers,
Benoit.0