xiaodududu / google-guice

Automatically exported from code.google.com/p/google-guice
Apache License 2.0
0 stars 0 forks source link

ServletModule needs better error msg if calling serve(..) or filter(..) outside of configureServlets #571

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
right now just throws a vague NPE.

Original issue reported on code.google.com by sberlin on 10 Nov 2010 at 10:06

GoogleCodeExporter commented 9 years ago
i actually hit this in the 2.0 => 3.0 migration.  fact is, the same case exists 
for AbstractModule....its binder is set to null after configure() exits.

probably a Preconditions.checkNonNull(...) with a bit that stores whether 
configure*() has exited will do the trick.

Original comment by ffa...@google.com on 27 Jan 2011 at 7:02

GoogleCodeExporter commented 9 years ago

Original comment by sberlin on 22 Feb 2011 at 1:45

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago

Original comment by dha...@gmail.com on 27 May 2014 at 9:23