On Fri, Sep 6, 2013 at 3:31 PM, Carsten Ziegeler <[email protected]> wrote: > ...The constants should really be defined in the BVP class, this is the > service declaring them...
Good point, done in revision 1521017 > >... I don't understand what you mean by "changes to existing code" - this is > new, isn't it?... Yes but existing code does use BindingsValuesProvider, and BindingsValuesProvidersByContext returning a Collection causes less changes to that existing code than the addContextBindings method that we discussed. I don't think one is better than the other anyway, it's just slightly different styles. -Bertrand
