I don't think that anybody would ever check this return value; a good method would be to throw an IllegalArgumentException if newVal == null, and to document for param newVal, that it must not be null