okay seems fine to me.

On Thu, Mar 29, 2012 at 9:57 PM, Eduardo Silva <[email protected]> wrote:

> Hi,
>
> we focus in performance and the string comparison is expensive and not
> necessary.
>
> If the developer using Duda API wants to have a friendly method to
> retrieve the parameters he could do:
>
> #define PARAM_AA   0
> #define PARAM_BB   1
> #define PARAM_CC   2
>
> and then use the macro/constant in the param get method
>
> regards,
>
> On Thu, Mar 29, 2012 at 9:20 AM, Sourabh Chandak <[email protected]>
> wrote:
> >
> > This patch changes the method of retrieving the parameters from
> > the web service URL. Instead of taking index of the parameter as
> > the argument, duda_param_get function now takes a char *, specifying
> > the name of the parameter.
> >
> > In terms of performance it will be a bit slower beacuse of an
> > increased string comparision loop to find the parameter name, but
> > is a good modification for ease of access of parameters.
> > ---
> >  plugins/duda/duda_api.h   |    2 +-
> >  plugins/duda/duda_param.c |   32 +++++++++++++++++++++++++++-----
> >  plugins/duda/duda_param.h |    2 +-
> >  plugins/duda/webservice.c |    1 +
> >  4 files changed, 30 insertions(+), 7 deletions(-)
> >
> >
> > _______________________________________________
> > Monkey mailing list
> > [email protected]
> > http://lists.monkey-project.com/listinfo/monkey
> >
>
>
>
> --
> Eduardo Silva
> http://edsiper.linuxchile.cl
> http://www.monkey-project.com
>



-- 
Sourabh Chandak
_______________________________________________
Monkey mailing list
[email protected]
http://lists.monkey-project.com/listinfo/monkey

Reply via email to