Changes made, committing

http://codereview.appspot.com/18041/diff/1/11
File
../trunk/java/common/src/main/java/org/apache/shindig/protocol/DefaultHandlerRegistry.java
(right):

http://codereview.appspot.com/18041/diff/1/11#newcode193
Line 193: sortedSet = Sets.<RestPath>newTreeSet();
On 2009/02/10 23:31:32, awiner wrote:
<RestPath> is unnecessary here

Done.

http://codereview.appspot.com/18041/diff/1/11#newcode286
Line 286: RpcInvocationHandler handler;
On 2009/02/10 23:31:32, awiner wrote:
final

Done.

http://codereview.appspot.com/18041/diff/1/11#newcode411
Line 411: private String operationPath;
On 2009/02/10 23:31:32, awiner wrote:
List<Part>, where Part has a PartType and a String?  All these could
be made
final also (albeit with some code tweaks)

Done.

http://codereview.appspot.com/18041

Reply via email to