axis-c-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Subra A Narayanan" <aswat...@gmail.com>
Subject Re: axiom_data_handler_get_input_stream
Date Fri, 17 Aug 2007 01:46:06 GMT
Samisa,

Pls find attached the svn diff file.

Thanks!


On 8/16/07, Samisa Abeysinghe <samisa@wso2.com> wrote:
>
> Subra A Narayanan wrote:
> > Just some additional comments:
> >
> > I think we definitely need the length function because there is no
> > other way of retrieving the length of the input stream as the
> > definition of the axiom_data_handler struct is in data_handler.c file
> > and not in axiom_data_handler.h file. So my question is, should the
> > declaration of the axiom_data_handler struct be moved to
> > axiom_data_handler.h file?
> >
> > I have made these simple changes to my local copy of the files. If you
> > want I can send it over to you guys and you can check it in after
> > reviewing the changes.
> Please do 'svn diff' and send the diff. I will have a look and committ in.
> BTW, the reason to have the struct in the C file was for data hiding
> purposes. We can add an accessor method, so that it would be in sync
> with the rest of the code.
>
> Thanks,
> Samisa...
> >
> > Thanks!!
> > Subra
> >
> > On 8/16/07, *Subra A Narayanan* <aswathsn@gmail.com
> > <mailto:aswathsn@gmail.com>> wrote:
> >
> >     Hello folks,
> >
> >     I was looking at data_handler.c file and came across this function
> >
> >
> >     AXIS2_EXTERN axis2_byte_t * AXIS2_CALL
> >     axiom_data_handler_get_input_stream(axiom_data_handler_t
> >     *data_handler,
> >         const axutil_env_t *env)
> >     {
> >         return (axis2_byte_t *)"";
> >     }
> >
> >
> >     The above function always returns null. Should the above line be:
> >
> >     return (axis2_byte_t *)data_handler->buffer;
> >
> >
> >     Also, would it be useful to have a function called
> >     axiom_data_handler_get_input_stream_len?
> >
> >     Subra
> >
> >
>
>
> --
> Samisa Abeysinghe : WSO2 WSF/C
> "
> http://wso2.org/projects/wsf/c?WSO2&nbsp;Web&nbsp;Services&nbsp;Framework/C&nbsp;-&nbsp;Open&nbsp;source&nbsp;C&nbsp;library&nbsp;for&nbsp;providing&nbsp;and&nbsp;consuming&nbsp;Web&nbsp;services
> "
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-c-user-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-c-user-help@ws.apache.org
>
>

Mime
View raw message