Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [ice-dev] [ice-build] [eclipse/ice] b9aa17: Fixed Entry and VizEntry constructors
Robert,

Updating Entry is a very tricky thing and that particular bug may be more of a feature. Did you run a problem in ICE with something that uses Entry to make sure it still works OK?

If you change Entry then you have to test job launchers and models to make sure that they still store and update values properly.

In the long term, we actually need to fix the way the Entry works with content providers. The current method is unused and duplicates values in the XML.

Jay

On 09/11/2015 11:59 AM, GitHub wrote:
  Branch: refs/heads/next
  Home:   https://github.com/eclipse/ice
  Commit: b9aa177f3bde720f51a3add27dee89cba360b025
      https://github.com/eclipse/ice/commit/b9aa177f3bde720f51a3add27dee89cba360b025
  Author: Robert Smith <SmithRW@xxxxxxxx>
  Date:   2015-09-11 (Fri, 11 Sep 2015)

  Changed paths:
    M org.eclipse.ice.datastructures.test/src/org/eclipse/ice/datastructures/test/EntryTester.java
    M org.eclipse.ice.datastructures/src/org/eclipse/ice/datastructures/form/Entry.java
    M org.eclipse.ice.viz.service.test/src/org/eclipse/ice/viz/service/datastructures/test/VizEntryTester.java
    M org.eclipse.ice.viz.service/src/org/eclipse/ice/viz/service/datastructures/VizEntry.java

  Log Message:
  -----------
  Fixed Entry and VizEntry constructors

The Entry and VizEntry constructors had a bug in which overloaded
setup() methods may not have their changes applied if the constructor
taking a content provider as an arguement was used. All constructors now
properly invoke setup() once.

Signed-off-by: Robert Smith <SmithRW@xxxxxxxx>





_______________________________________________
ice-build mailing list
ice-build@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/ice-build


-- 
Jay Jay Billings
Oak Ridge National Laboratory
(865) 241-6308
Twitter Handle: @jayjaybillings

I only check email at 8:30, 12:30 and 16:30 every day. Please call me or send 
me a tweet if your message is urgent and I will take a look.

Back to the top