sim: Get rid of the non-const serialize() method
authorAndreas Sandberg <andreas.sandberg@arm.com>
Fri, 4 Dec 2015 09:48:48 +0000 (09:48 +0000)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Fri, 4 Dec 2015 09:48:48 +0000 (09:48 +0000)
The last SimObject using the legacy serialize API with non-const
methods has now been transitioned to the new API. This changeset
removes the serializeOld() methods from the serialization base class
as they are no longer used.

src/sim/serialize.cc
src/sim/serialize.hh
src/sim/sim_object.cc

index 90cb5e288112991f1dd6b9e0047229561299dfe7..e55107b793ccd6c79e0191877bb0834bb459000f 100644 (file)
@@ -580,13 +580,6 @@ Serializable::serializeSection(CheckpointOut &cp, const char *name) const
     serialize(cp);
 }
 
-void
-Serializable::serializeSectionOld(CheckpointOut &cp, const char *name)
-{
-    Serializable::ScopedCheckpointSection sec(cp, name);
-    serializeOld(cp);
-}
-
 void
 Serializable::unserializeSection(CheckpointIn &cp, const char *name)
 {
index 9a1170a9eef4c24682bef702c77a78691cd71478..376df9ec039f6c25608fa5883c974e3d3c6664d9 100644 (file)
@@ -323,32 +323,6 @@ class Serializable
         unserializeSection(cp, name.c_str());
     }
 
-    /**
-     * @{
-     * @name Legacy interface
-     *
-     * Interface for objects that insist on changing their state when
-     * serializing. Such state change should be done in drain(),
-     * memWriteback(), or memInvalidate() and not in the serialization
-     * method. In general, if state changes occur in serialize, it
-     * complicates testing since it breaks assumptions about draining
-     * and serialization. It potentially also makes components more
-     * fragile since they there are no ordering guarantees when
-     * serializing SimObjects.
-     *
-     * @warn This interface is considered deprecated and should never
-     * be used.
-     */
-
-    virtual void serializeOld(CheckpointOut &cp) {
-        serialize(cp);
-    }
-    void serializeSectionOld(CheckpointOut &cp, const char *name);
-    void serializeSectionOld(CheckpointOut &cp, const std::string &name) {
-        serializeSectionOld(cp, name.c_str());
-    }
-    /** @} */
-
     /** Get the fully-qualified name of the active section */
     static const std::string &currentSection();
 
index e87b7240fd7d1c2cfddaa95767183bd817c11a6c..af0ef3c2f5161cbc8841bb7c8cab8c71281fc0f3 100644 (file)
@@ -151,7 +151,7 @@ SimObject::serializeAll(CheckpointOut &cp)
         SimObject *obj = *ri;
         // This works despite name() returning a fully qualified name
         // since we are at the top level.
-        obj->serializeSectionOld(cp, obj->name());
+        obj->serializeSection(cp, obj->name());
    }
 }