Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[dsdp-tm-dev] RC1 Code Review - persistence

Hi Dave,

I reviewed yesterday's changes and I have some questions
and requests:

* IRSEPersistableContainer.NO_CHILDREN
  should be explicitly declared public
  and please add Javadoc for it

* ISystemProfile.setActive()
  Javadoc: can you explain what the "active" state means?

* ISystemProfileManager.getActiveSystemProfilePosition()
  You deprecated this. Whenever you deprecate something, 
  please leave a note what other mechanism the client can
  use to accomplish what used to be accomplisehd by that

* PropertyList, RSEPersistableObject,
  Please use the "Fix Copyrights" tool. It has a slightly
  different default comment format built in than you are
  using. We should have a consistent comment format everywhere.

Can you please answer and/or address these.

Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member

Back to the top