This code needs comments. I'm having a hard time figuring out what it does.
http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java File dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java (right): http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode45 dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:45: * Collapses derived properties. Expand this comment to roughly describe the process implemented for future maintainers. http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode52 dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:52: // find collapsed properties For each input property, determine if it has a collapsed-value equivalence set containing the associated input value. http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode68 dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:68: Map<String, ArrayList<String>> dependencies = new TreeMap<String, ArrayList<String>>(); Make this a Map<BindingProperty, List<String>>. Add doc: A map of binding properties to the names of the properties that must be evaluated prior to computing the value of the key object. http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode72 dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:72: if (deps.size() > 0) { !deps.isEmpty() http://gwt-code-reviews.appspot.com/1424803/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode76 dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:76: for (Map.Entry<String, ArrayList<String>> e : dependencies.entrySet()) { What is this loop doing? http://gwt-code-reviews.appspot.com/1424803/diff/1/user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc File user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc (right): http://gwt-code-reviews.appspot.com/1424803/diff/1/user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc#newcode26 user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc:26: permutations into a single compiled unit. As a consequence, collapsed units into a single output file http://gwt-code-reviews.appspot.com/1424803/diff/1/user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc#newcode39 user/src/com/google/gwt/user/tools/templates/sample/_srcFolder_/_moduleFolder_/_moduleShortName_.gwt.xmlsrc:39: By default, GWT will collapse permutations of older and less used browsers. s/GWT/this template/ http://gwt-code-reviews.appspot.com/1424803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
