Foodback from IM discussion.
http://gwt-code-reviews.appspot.com/827802/diff/4001/5007 File user/src/com/google/gwt/rpc/client/impl/SimplePayloadSink.java (right): http://gwt-code-reviews.appspot.com/827802/diff/4001/5007#newcode76 user/src/com/google/gwt/rpc/client/impl/SimplePayloadSink.java:76: // ETypeSeedName~"9~FieldName // ETypeSeedName~IOrdinal~ http://gwt-code-reviews.appspot.com/827802/diff/4001/5007#newcode80 user/src/com/google/gwt/rpc/client/impl/SimplePayloadSink.java:80: accept(new IntValueCommand(x.getValue().ordinal())); Use appendTypedData here. The use of accept() is because Strings can be encoded as a backreference. http://gwt-code-reviews.appspot.com/827802/diff/4001/5008 File user/src/com/google/gwt/rpc/server/SimplePayloadDecoder.java (right): http://gwt-code-reviews.appspot.com/827802/diff/4001/5008#newcode201 user/src/com/google/gwt/rpc/server/SimplePayloadDecoder.java:201: // ETypeSeedName~"9~FieldName // ETypeSeedName~IOrdinal~ http://gwt-code-reviews.appspot.com/827802/diff/4001/5008#newcode212 user/src/com/google/gwt/rpc/server/SimplePayloadDecoder.java:212: x.setValue(enumConstants[ordinal]); Please add a TODO here to the effect of: This is prone to subtle corruption or an ArrayIndexOutOfBoundsException if the client and server have drifted. http://gwt-code-reviews.appspot.com/827802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
