Looking at the usage, it's not really that intuitive to me:

  @Load(name="example.wtkx") private Border border;
  @Bind(property="border") private Slider redSlider;

it's not obvious at first glance that the "property" value on @Bind
actually refers to which @load annotation is referenced.

I would suggest the following
- default to not needing to specify anything on the @Bind annotation
if there is only one @load annotation in the class.
- rename the property parameter on @Bind to something like
"loadProperty" or "loadField"

Also the "name" parameter on the @Bind annotation doesn't make sense
because it refers to the "id" in the wtkx file. Rather just call the
field "id".

But in general, these are just minor issues to me - I really like this feature!

Regards, Noel Grandin


On Wed, May 13, 2009 at 21:26, Todd Volkert <[email protected]> wrote:
> As one final follow-up on this discussion, I wanted to point out that
> this is now done and will be part of the 1.2 release.  I've
> temporarily posted the Javadoc for it here:
> http://people.apache.org/~tvolkert/doc/pivot/wtkx/Bindable.html.
> Thanks for the great idea Dominique!
>
> -T
>

Reply via email to