On 16/03/2015 14:50, Raghavendran N wrote: >> diff --git a/java/org/apache/tomcat /websocket/server/WsServerContainer.java >> b/java/org/apache/tomcat/websocket/server/WsServerContainer.java >> index d2c824c..a663e8e 100644 >> --- a/java/org/apache/tomcat/websocket/server/WsServerContainer.java >> +++ b/java/org/apache/tomcat/websocket/server/WsServerContainer.java >> @@ -166,7 +166,7 @@ public class WsServerContainer extends >> WsWebSocketContainer >> * must be called before calling this method. >> * >> * @param sec The configuration to use when creating endpoint >> instances >> - * @throws DeploymentException if the endpoint can not be published as >> + * @throws DeploymentException if the endpoint cannot be published as >> * requested >> */ >> @Override >>
>Both are technically correct. Which to use is a style choice although >cannot is certainly the more common. I thought cannot was a more common usage and sent the patch. I also wanted to keep it very small and make a small beginning. >There were rather more instances than the one you highlighted. I have no >string preference for one over the other so I did a global search and >replace for 9.0.x. Thank you for taking it up. I did a git pull and saw your commit :) Also for fixing the other occurrences. I wanted to send a very small patch and I picked this one. I guess I should have picked all the occurrences. >Personally, I don't think it is worth back-porting. If someone wants to >do that work, got for it. >Mark Thank you Mark. -raghav --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
