On 30 Jan 2014 16:38, "Matt Benson" <gudnabr...@gmail.com> wrote: > > How do we feel about changing the type of a thrown RTE? Specifically, we > have agreed to standardize on the idea that Validate.notNull() throws > NullPointerException, but the recently added > FieldUtils#removeFinalModifier() method validates the incoming field > parameter with Validate.isTrue(field != null), throwing > IllegalArgumentException. I prefer the simplicity of Validate.notNull() > but changing this now would mean that consumers of the method who are > conceivably now catching IllegalArgumentException would now fail to catch > the thrown NPE. > > Thoughts? > > Matt
If this is the new normal, we should update the developer guide on the website as it currently says the opposite. Duncan
