Alan,

Thanks for sticking with me - your comments and support have definitely improved my code, and I am grateful to you as a result.

I have updated the version at...

https://jdk-collaboration.dev.java.net/servlets/ProjectForumMessageView?messageID=20129&forumID=1463

...to include:

a) toString(ParameterSeparator)
b) Better JavaDoc around the 'parse' methods for ParameterSeparator

What we have left appear to be some fairly small details. If I may summarise, you are saying...

1) java.net.URL is discouraged, therefore we shouldn't support it in our API nor follow its naming convention (even though we'll sit alongside it in the java.net package)
2) We should move the 'apply' method into java.net.URI
3) Keeping the method names familiar to javax.servlet.ServletRequest developers is less important than neatness of code during method chaining

I'm happy either way on 1 and 2, though I disagree with you on 3, but none of these seem like they should be major obstacles to CCC approval? With your permission, then, I'd like to leave them to the larger consensus?

Regards,

Richard.

Reply via email to