Comment #5 on issue 395 by sberlin: InjectorBuilder
http://code.google.com/p/google-guice/issues/detail?id=395

True, you could definitely call stage() twice on the builder -- but atleast the builder isn't distributed among different classes/authors, so you see the error in front of you. That's the big pitfall of Binder, I think -- it scales *so* well that it even scales to different authors & classes, which is where these building options
fail to make sense.

Agreed on separating InjectorBuilder.  Guice exposes it right now because
InjectorBuilderImpl needs to be in internal (because it uses package-private code). If Guice is deprecated, InjectorBuilder can become a class (right now it's an
interface) and delegate to the internal InjectorBuilderImpl (as opposed to
InjectorBuilderImpl implementing InjectorBuilder right now).

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"google-guice-dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/google-guice-dev?hl=en.

Reply via email to