9867f9c2142355ae958f9eeb8fb96811082c8812 |
|
02-Feb-2015 |
sameb <sameb@google.com> |
Implement my old patch for issue #366, with some tweaks. This does the following: * Fixes @Provides injection so that parameters are checked for nullability. By default this will error. The flag is named: guice_check_nullable_provides_params and can be set to ERROR, WARNING or IGNORE. * Adds InjectionPoint.forMethod to build an InjectionPoint off an arbitrary method. * Adds Binder.getProvider(Dependency) to a get a Provider for a given dependency (with all its nullability & injection points maintained). * Update ProviderLookup to accept a Dependency in addition to a Key. This is in preparation for two things: 1) Allowing multibindings/mapbindings/optionalbindings to be specified as annotations on methods in a module. 2) Adding a dagger compatibility module. ... the general idea will be that I'll also add a hook into ProvidesMethodModule somehow to look at arbitrary other annotations and let folks process them specially. ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=85353820
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
4daa205dc4fe1d339f91155dde65f3941cbd144c |
|
03-May-2014 |
Sam Berlin <sameb@google.com> |
Ignore synthetic/bridge methods when determining if a method should be injected. This broke injections for different reasons before & after java8, but the fix is the same for both. Add a test with method injection points & super/sub classes that change method visibility, to confirm that synthesized/bridged methods don't artificially change what's injected or the parameters of the injections. ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=66136797
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
4d5804923d55353b214b8e2a45a6416a75665a4d |
|
09-Aug-2013 |
Christian Edward Gruber <cgruber@google.com> |
Fix random bitwise | to be a real ||. ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=50288042
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
c756777526309dc6c9ff2b5df80a8f811d390c09 |
|
01-Jun-2012 |
Sam Berlin <sameb@google.com> |
Add Binder.requireAtInjectOnConstructors, to force Guice to require @Inject annotations on constructors. Revision created by MOE tool push_codebase. MOE_MIGRATION=4906
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
ee443bce9731bb8da1eb60202f68de4c4c298698 |
|
09-Sep-2011 |
guice.mirrorbot@gmail.com <guice.mirrorbot@gmail.com@d779f126-a31b-0410-b53b-1d3aecad763e> |
Fix issue 644. Print a better error message when requestStaticInjection is called on an interface, fix it so that static injection errors show the source of the static injection. Revision created by MOE tool push_codebase. MOE_MIGRATION=3070 git-svn-id: https://google-guice.googlecode.com/svn/trunk@1578 d779f126-a31b-0410-b53b-1d3aecad763e
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
b7a02b02d81c830d148355c90bc309bcd66fb592 |
|
08-Jul-2011 |
sberlin <sberlin@d779f126-a31b-0410-b53b-1d3aecad763e> |
* Remove unused imports * Sort imports * Convert tabs to spaces * Fix the ant no_aop build Revision created by MOE tool push_codebase. MOE_MIGRATION=2532 git-svn-id: https://google-guice.googlecode.com/svn/trunk@1572 d779f126-a31b-0410-b53b-1d3aecad763e
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
d9c913acca55023ef5d76a32c3d4a51ee6b420cb |
|
26-Jun-2011 |
sberlin <sberlin@d779f126-a31b-0410-b53b-1d3aecad763e> |
switch Guice from manually repackaging Guava to depending on Guava. it's still jarjar'd right now, which is causing a ~400k increase in guice-snapshot.jar. next step is to switch to ProGuard to remove the unnecessary code and cut it back down (even further?!). this will let people build from Guice source and depend directly on Guava code without having to worry about hiding internal/util. git-svn-id: https://google-guice.googlecode.com/svn/trunk@1558 d779f126-a31b-0410-b53b-1d3aecad763e
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
dfee9fd5f78c5f1f769713b603e4e92a0b7b01e3 |
|
30-Oct-2010 |
sberlin <sberlin@d779f126-a31b-0410-b53b-1d3aecad763e> |
issue 531 -- don't let c.g.i.internal.util reference outside its package. git-svn-id: https://google-guice.googlecode.com/svn/trunk@1328 d779f126-a31b-0410-b53b-1d3aecad763e
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|
74d714c1e72bb60706ed9df4d1982971c88c10aa |
|
21-Oct-2010 |
mcculls <mcculls@d779f126-a31b-0410-b53b-1d3aecad763e> |
Issue 552: (step 1) move core code to its own subdirectory git-svn-id: https://google-guice.googlecode.com/svn/trunk@1300 d779f126-a31b-0410-b53b-1d3aecad763e
/external/guice/core/src/com/google/inject/spi/InjectionPoint.java
|