Re: Suggested cleanup: DataObject/PersistenceState

From: Andrus (andru..bjectstyle.org)
Date: Sun Dec 01 2002 - 17:39:11 EST

  • Next message: Andrus: "Re: Recent mass update"

    At 08:07 AM 11/26/2002 +1300, Craig Miskell wrote:
    >On Tue, 2002-11-26 at 04:23, Holger Hoffstätte wrote:
    > >
    > > Suggestion:
    > >
    > > - deprecate & move persistenceStateSting() out of DataObject into
    > > PersistenceState
    >+1
    >
    > > - turn PersistenceState into a class instead of interface since there are
    > > no implementors anyway (it's just a place to keep the int symbols)
    >Sounds reasonable to me, but I'm wondering if Andrus had any particular
    >reason for doing it as an interface in the first place (or maybe it's
    >just a hangover from a previous idea).

    My vote on this is +0

    I noticed that the change has been done already, and I don't really have a
    preference one way or another. The reason PersistenceState was an interface
    was only because it had no behavior and only defined a few constants. Now
    that it does have methods, I have no problem converting it to a class.

    Andrus



    This archive was generated by hypermail 2.0.0 : Sun Dec 01 2002 - 17:38:17 EST