Reviewers: rjrjr, jlabanca,

Message:
Actual changes marked in the review, the rest is autoformatting.

The basis for this change is to prevent properties that depend on other
collapsed properties from ballooning the number of hard (emitted)
permutations.  For example, the html5-related deferred binding
properties prevent a simple <collapse-property name="user.agent" /> from
actually reducing the number of permutations emitted.

The rule:
  A dependency on a collapsed property forms an equivalence set for the
dependent value.  In other words, "Collapsing is contagious."


http://gwt-code-reviews.appspot.com/1525804/diff/1/dev/core/src/com/google/gwt/dev/cfg/BindingProperty.java
File dev/core/src/com/google/gwt/dev/cfg/BindingProperty.java (right):

http://gwt-code-reviews.appspot.com/1525804/diff/1/dev/core/src/com/google/gwt/dev/cfg/BindingProperty.java#newcode313
dev/core/src/com/google/gwt/dev/cfg/BindingProperty.java:313: void
normalizeCollapsedValues() {
Actual change is here.

http://gwt-code-reviews.appspot.com/1525804/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/1525804/diff/1/dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java#newcode230
dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java:230:
public String toString() {
Actual change here.

Description:
Collapse deferred-binding property values that depend on collapsed
property values.
Patch by: bobv
Review by: rjrjr


Please review this at http://gwt-code-reviews.appspot.com/1525804/

Affected files:
  M dev/core/src/com/google/gwt/dev/cfg/BindingProperty.java
  M dev/core/src/com/google/gwt/dev/cfg/PropertyPermutations.java
  M dev/core/test/com/google/gwt/dev/cfg/ModuleDefTest.java


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to