Re: svn commit: r732453 [1/3] - in /incubator/shindig/branches/1.0.x-incubating: ./ java/common/ java/common/src/main/java/org/apache/shindig/auth/ java/common/src/main/java/org/apache/shindig/common/ java/common/src/main/java/org/apache/shindig/comm

[Henning Schmiedehausen]

I  -1 this patch unless you can sufficiently explain why e.g.

> -    tokenData = new HashMap<String, String>(5,1);
> +    tokenData = Maps.newHashMapWithExpectedSize(5);

is better readable to someone with knowledge of the standard Java APIs
and no knowledge of Google Collections.

All that this huge patch IMHO does is making the code harder to read
for no particular reasons.

Please back the patch out again and let's discuss whether it makes sense.

    Thanks
       Henning