[ http://issues.apache.org/jira/browse/XMLRPC-75?page=comments#action_12361418 ]
Jochen Wiedmann commented on XMLRPC-75: --------------------------------------- I haven't had the time to carefully read through your patch. I do not know "Aaron Hamid's Introspection patch" not do I now the one from xmlrpc-c.sf.net. To me, it currently looks like yet another handler mapping. I wouldn't refuse the idea to add a different handler mapping to the source tree. The handler mapping API was written specifically with different mappings in mind. Otoh, not every mapping must be a port of the XML-RPC core distribution. What I currently would like to see, would be some sentences, that describe what the differences between the default handler mapping and your suggestions are. What advantages, what disadvantages, and so on. > Introspection Patch for XML-RPC, current to SVN 2005-12-29 (r359943) > -------------------------------------------------------------------- > > Key: XMLRPC-75 > URL: http://issues.apache.org/jira/browse/XMLRPC-75 > Project: XML-RPC > Type: New Feature > Components: Source > Versions: unspecified > Reporter: Walter Mundt > Attachments: IntrospectionExample.java, xmlrpc-2.0-beta-introspection.patch > > I've taken Aaron Hamid's Introspection patch (which was in turn derived from > the one here: > http://xmlrpc-c.sourceforge.net/hacks/helma-xmlrpc-introspection.diff ) and > basically rewritten it to work with the current XML-RPC library. > I tried to set it up so that it followed the existing coding style as much as > possible, and would not break any existing applications. For example, when I > needed a list of handlers supported by a particular mapping, I added a new > interface XmlRpcListableHandlerMapping that provides this information, made > the default handler implement it, and designed my code so that if it were not > implemented by a handler in use, everything would work except the > system.listMethods functionality that relies on it. > I hope you will be able to integrate this code into the next version of the > library; please contact me with any questions or issues. I will be attaching > the patch to this issue if I can so that the problem with his previous patch > will not occur. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira