axis-c-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nandika Jayawardana" <jayaw...@gmail.com>
Subject Re: [Axis2]Replacing double pointer env param to a single pointer env param
Date Wed, 31 May 2006 05:11:25 GMT
+1 for change.

nandika

On 5/31/06, Samisa Abeysinghe <samisa.abeysinghe@gmail.com> wrote:
>
> If there are no objections, I would go ahead and make this change either
> today or tomorrow.
>
> Thanks,
> Samisa...
>
> Samisa Abeysinghe wrote:
>
> > Hi All,
> >    For almost all the method calls, we pass the environment as a
> > double pointer. This was originally done thinking that we could create
> > an environment in case the user did not supply a correct pointer.
> > However, after the code evolved, now we realize that this view is
> > flawed. We do not need a double pointer here.
> >
> >    For new users, use of a double pointer would look strange.
> > Moreover, it is inefficient to use a double pointer, especially given
> > that fact that all the methods are using it (though bulk of the getter
> > functions do not need it)
> >
> >    The better and the correct approach would be to change it to "const
> > axis2_env_t *".
> >
> >    This change is very time consuming and also would affect those that
> > are already using the API. However, we better change this now, rather
> > than waiting till late.
> >
> > Thanks,
> > Samisa...
> >
> >
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-c-dev-help@ws.apache.org
>
>

Mime
View raw message