Started to review... but ouch... Rietveld doesn't show moves as such.  I
am defeated.


http://codereview.appspot.com/14066/diff/1/51
File java/common/pom.xml (right):

http://codereview.appspot.com/14066/diff/1/51#newcode110
Line 110: <version>2.2</version>
versions are usually specified in the root pom.xml (otherwise you run
the risk of multiple versions)

http://codereview.appspot.com/14066/diff/1/69
File
java/common/src/main/java/org/apache/shindig/protocol/model/Exportablebean.java
(right):

http://codereview.appspot.com/14066/diff/1/69#newcode32
Line 32: public @interface Exportablebean {}
ExportableBean, while you're moving

http://codereview.appspot.com/14066

Reply via email to