tree cd3d2424edabb3dfde3ddc8279c3190e12d4b483
parent 4c01fd73cad04a7f247d4c01eb84667ce11cf18a
author limpbizkit <limpbizkit@d779f126-a31b-0410-b53b-1d3aecad763e> 1225602895 +0000
committer limpbizkit <limpbizkit@d779f126-a31b-0410-b53b-1d3aecad763e> 1225602895 +0000

Hopefully the last of the big exceptions refactorings. I went through all of the places we're adding context to our Errors object and made sure we're never doubling-up -- specifing the same injection point or key multiple times.

The new errors have nice 'at' lines for parameters, fields and linked bindings. Hopefully this makes it easier to follow the stacktraces. Additional context (such as the binding's origin in a module) could be added later if desired. Currently we have module-specific sources in CreationExceptions, and plain old binding sources elsewhere. The end result is messages that can look like this:

com.google.inject.ProvisionException: Guice provision errors:

1) Error injecting constructor, java.lang.UnsupportedOperationException
  at com.google.inject.ProvisionExceptionTest$RealD.<init>(ProvisionExceptionTest.java:284)
  at binding for com.google.inject.ProvisionExceptionTest$RealD.class(ProvisionExceptionTest.java:284)
  at binding for com.google.inject.ProvisionExceptionTest$D.class(ProvisionExceptionTest.java:1)

1 error

git-svn-id: https://google-guice.googlecode.com/svn/trunk@652 d779f126-a31b-0410-b53b-1d3aecad763e