Re: [vtp-dev] Browser API Tutorial

[Dirk Schnelle <dirk.schnelle@xxxxxxxxxxxxx>]

Brent,

> For #1, sorry for the confusion. I think this needs to be clearer in
> the tutorial. setDefaults()/initializeFrom() are concepts lifted from
> Eclipse's ILaunchConfigurationTab. There is a tutorial that talks
> about the APIs here: 
> 
> http://www.eclipse.org/articles/Article-Launch-Framework/launch.html 
> 
> Its description of the two functions is: 
> 
> "In general terms, the first two methods copy values from the tab to a
> working copy, while the third method copies values from a working copy
> to the tab.  The setDefaults() method is called when a new config is
> created. " 
> 
> "The initializeFrom() method does the opposite of the performApply()
> method.  It reads attribute values out of the config argument, and
> sets these values in the corresponding GUI widgets" 
> 

thanks for the clarification. It was not obvious to me, since I am not
very familiar with eclipse.


> #2: Right now isValid is a simple boolean function and error messages
> would have to be inserted as widgets inside the configuration pane.
> This is something that I think needs better integration with the
> launch configuration UI, and I'd suggest you write the code with this
> in mind so that it would be simple to change the return type later as
> we modify the interface. I've created a bugzilla to track the
> enhancement: 
> 
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=136115 
> 

OK, I'll keep this in mind, for the design of the current prototype.
I think this enhancement is OK, although it will cover validation errors
only in sequential order, but this is OK.

/dirk