Daniel L. Rall wrote:
- return base64Codec.decode(cdata.getBytes()); + return base64Codec.decode((Object) cdata.getBytes());
This cast looks unnecessary. What's the thinking here?
No, it's required, because the compiler would pick the old method otherwise.
Why is that a problem? I was under the impression that this patch was to make XML-RPC work across different versions of Commons Codec. Do these Codec methods have different implementations with wildly different semantics?