more comments

1 files changed, 2 insertions, 15 deletions

diff --git a/cpp/demo/Ice/session/SessionI.h b/cpp/demo/Ice/session/SessionI.h
index abe4eb73b6b..998e8c92c31 100755
--- a/cpp/demo/Ice/session/SessionI.h
+++ b/cpp/demo/Ice/session/SessionI.h
@@ -21,8 +21,8 @@ public:
 
     virtual Demo::HelloPrx createHello(const Ice::Current&);
     virtual void refresh(const Ice::Current&);
-    virtual void destroy(const Ice::Current&);
     virtual std::string getName(const Ice::Current&) const;
+    virtual void destroy(const Ice::Current&);
 
 private:
 
@@ -37,21 +37,8 @@ private:
     const std::string _name;
     IceUtil::Time _timestamp; // The last time the session was refreshed.
 
-    //
-    // XXX This needs to be a static, otherwise hello objects from
-    // different client have the same ID.
-    //
-    // The reason I didn't make the _nextId static in the first place
-    // was because the client also displays the index of the hello
-    // object. I think displaying a different id on the client &
-    // server will be confusing, and I didn't think it was worth
-    // making the client keep a real id->hello object client map
-    // because a) it would make the client more complicated and b) it
-    // would require the user to type bigger and bigger numbers all
-    // the time which is a pain in the ass.
-    //
     int _nextId; // The per-session id of the next hello object. This is used for tracing purposes.
-    std::list< Demo::HelloPrx> _objs; // List of per-client allocated Hello objects.
+    std::list< Demo::HelloPrx> _objs; // List of per-session allocated hello objects.
     bool _destroy;
 };