The CCC request has received initial approval,
which simply means that the API change is accepted in principle.
But, we need to finalize the specification and submit it to the CCC
for final approval.

Due to the (large) amount of discussion that has already
taken place, I don't anticipate that this will take too long.

The following is a comment from the final CCC reviewer. The comments
are based on an older version of the API, but they are still
applicable to the current version.

- Michael

A couple of quick comments:

 - The names of the elements of the URIQueryString.ParameterSeparator
   enum should be in UPPER_CASE, per convention.  (Actually, defining
   an enum for this seems like overkill to me; I'd be tempted to accept
   a string argument and throw an IllegalArgumentException for anything
   except "&" and ";").

 - Using string arrays in the URIQueryString API is just wrong; these
   should be List<String>.  (Mimicing the ServletRequest API is of
   marginal value at best, and so not an argument for using String[].)

Reply via email to