Hi,

thanks for your patch, it looks pretty good, well done!

We just need to adjust some minor things...

   - params->params_xxx is redundant. please set the methods like
params->method_name(..)
   - fix some lines that have \t in the middle (we use spaces instead of tabs)

please resubmit the patch with the changes proposed so we are ok... so
far, good job :)

cheers

On Wed, Mar 21, 2012 at 7:12 AM, Sourabh Chandak <[email protected]> wrote:
> Hi,
>
> To access the parameters specify the parameter index(starting from 0).
> For example check out the example webservice.
>
> Regards.
>
>
> On Wed, Mar 21, 2012 at 6:38 PM, Sourabh Chandak <[email protected]>
> wrote:
>>
>> ---
>>  plugins/duda/duda_api.c      |   26 +++++++++
>>  plugins/duda/duda_api.h      |    6 ++
>>  plugins/duda/example/hello.c |  117
>> +++++++++++++++++++++++++++++++++++++++---
>>  plugins/duda/webservice.c    |    3 +-
>>  plugins/duda/webservice.h    |    3 +-
>>  5 files changed, 144 insertions(+), 11 deletions(-)
>>
>
>
>
> --
> Sourabh Chandak
>
>
>
>
> _______________________________________________
> Monkey mailing list
> [email protected]
> http://lists.monkey-project.com/listinfo/monkey
>



-- 
Eduardo Silva
http://edsiper.linuxchile.cl
http://www.monkey-project.com
_______________________________________________
Monkey mailing list
[email protected]
http://lists.monkey-project.com/listinfo/monkey

Reply via email to