Hi Roger,
What about adding a bit of leeway in the reason for which
IAE can be thrown. Here is the text from your webrev:
359 * @throws IllegalArgumentException If any of the
following is true:
360 * <ul>
361 * <li> if a limit is missing the name or the name is
not one of
362 * "maxdepth", "maxrefs", "maxbytes", or "maxarray"
363 * <li> if the value of the limit can not be parsed by
364 * {@link Long#parseLong Long.parseLong} or is negative
365 * <li> if the pattern contains "/" and the module name
is missing
366 * or the remaining pattern is empty
367 * <li> if the package is missing for ".*" and ".**"
368 * </ul>
could it be amended to something like:
@throws IllegalArgumentException if the pattern string is illegal or
malformed and cannot be parsed.
In particular, an IllegalArgumentException will be thrown
if any of the following is true: ...
best regards,
-- daniel
On 06/12/16 22:04, Roger Riggs wrote:
Please review a few additional clarifications to the ObjectInputFilter
specification for generating
a filter from a pattern and use in ObjectInputStream plus related test
updates.
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-createfilter-8170287/
Thanks, Roger