Comment #13 on issue 757 by tebu...@googlemail.com: Java 8 lambda definitions can break Guice's internal exception handling
http://code.google.com/p/google-guice/issues/detail?id=757

I explicitly left that part out, because in my reasoning I tried to express, that my patch is not about the state of integrated ASM and the Java 8 spec, but Guice robustness on providing meaningful error messages.

In my opinion Guice should _not fail_ on trying to be more precise about an error than needed. Instead it should always fail-safe back to what it is knows about the error instead of hiding it with a totally unrelated subsequent error.

Please have a look at the patch. It's really trivial and I think it's worth to be integrated, because everything else already seems to work for me under Java 8 and I assume the ASM reverse lookup for Errors might be always a candidate for unexpected and flaky failures.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"google-guice-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to google-guice-dev+unsubscr...@googlegroups.com.
To post to this group, send email to google-guice-dev@googlegroups.com.
Visit this group at http://groups.google.com/group/google-guice-dev.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to