[ 
http://issues.apache.org/jira/browse/XMLRPC-75?page=comments#action_12368078 ] 

Jochen Wiedmann commented on XMLRPC-75:
---------------------------------------

Walter,

I apologize, but I am unhappy with the overloading.

While I accept, that it might be suitable in certain situations, I do not think 
that it should be the default. In particular, I do not think that the selection 
of the right method via reflection should occur with any method invocation. 
That's something we had in version 2 and have just successfully managed to get 
rid of.

IMO, the AbstractReflectiveHandlerMapping should support a single method per 
name only. There is no reason for not adding more complicated subclasses, if 
you like that.

Jochen


> 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
>     Assignee: Jochen Wiedmann
>  Attachments: IntrospectionExample.java, xmlrpc-2.0-beta-introspection.patch, 
> xmlrpc3-metadata.diff, xmlrpc3-metadata.diff, xmlrpc3-metadata.diff
>
> 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

Reply via email to