Skip to content
Snippets Groups Projects
Commit a7d3a5a1 authored by Marco Clemencic's avatar Marco Clemencic
Browse files

Merge branch 'AppMgrUI' into 'master'

Switch IAppMgrUI inheritance from Interface to INamedInterface and remove the pure virtual name()

IAppMgrUI inherits from IInterface, and adds a pure virtual name() method with the exact same signature as the one in INamedInterface.

As a result, the same effect can be obtained by inheritance from INamedIterface instead of IInterface.  The advantage is that if an implementation (virtually) inherits not only from IAppMgrUI but also another interface which (virtually) inherits from INamedInterface, there is no ambiguity in the definition of name(). With the 'old' code, one has to disambiguate between multiple name() members...

See merge request !18
parents 6a793d3d f3daa988
No related branches found
No related tags found
No related merge requests found
......@@ -3,7 +3,7 @@
// Include files
#include "GaudiKernel/StatusCode.h"
#include "GaudiKernel/IInterface.h"
#include "GaudiKernel/INamedInterface.h"
#include "GaudiKernel/StateMachine.h"
#include <string>
......@@ -17,7 +17,7 @@
@date 30/10/98
*/
class GAUDI_API IAppMgrUI: virtual public IInterface {
class GAUDI_API IAppMgrUI: virtual public INamedInterface {
public:
/// InterfaceID
DeclareInterfaceID(IAppMgrUI,3,0);
......@@ -34,9 +34,6 @@ public:
/// Process the next maxevt events
virtual StatusCode nextEvent(int maxevt) = 0;
/// The identifying name of the AppMgrUI object.
virtual const std::string& name() const = 0;
/// Start (from INITIALIZED to RUNNING).
virtual StatusCode start() = 0;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment