Bug in TransportsUtil.java

[Michenux]

Hi,

I think there's a bug in class TransportsUtil.java.

for (Object m : l) {
          return matchDecoder(e, m, nd, decodedObjects);
}

matchDecoder will be invoked only for the first element of the list and the following elements will be ignored.

Not sure but something like this would be better :

ArrayList<Object> list = new ArrayList<>();
for (Object m : l) {
   list.add(matchDecoder(e, m, nd, decodedObjects));
}
return list;

[jfarcand]

@Michenux Pull request welcomed :-)

[jfarcand]

Fixed https://github.com/Atmosphere/wasync/pull/153

[slovdahl]

This sounds similar to the issue I reported in #130, doesn't it? Maybe #130 wasn't properly fixed after all.

[Michenux]

Yes, the problem i had is the same as in #130. Works well with the fix but i tested it only with polling transport.

[thabach]

There were build test-failures, please reconsider the fix.