Re: Patch for SHINDIG-757

[awiner]

http://codereview.appspot.com/13061/diff/1/2
File
java/common/src/main/java/org/apache/shindig/common/xml/XmlUtil.java
(right):

http://codereview.appspot.com/13061/diff/1/2#newcode313
Line 313: try {
why not just:

  try {
    parse(xml);
  } catch (XmlException e) {
    throw new RuntimeException(e);
  }
?

http://codereview.appspot.com/13061/diff/1/8
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/DefaultGadgetSpecFactory.java
(right):

http://codereview.appspot.com/13061/diff/1/8#newcode58
Line 58: private final Logger logger =
Logger.getLogger(DefaultGadgetSpecFactory.class.getName());
usually static.  @Singleton, so little practical effect.

http://codereview.appspot.com/13061/diff/1/9
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/spec/GadgetSpec.java
(right):

http://codereview.appspot.com/13061/diff/1/9#newcode56
Line 56: this.checksum = HashUtil.checksum(doc.toString().getBytes());
doc.toString() strikes me as an expensive operation.

http://codereview.appspot.com/13061