You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@axis.apache.org by Samisa Abeysinghe <sa...@gmail.com> on 2006/06/01 03:35:53 UTC
Re: [Axis2] Code freeze - Replacing double pointer env param to a
single pointer env param
I forgot to mention, guththila too would be broken with these changes.
Got to fix it.
Samisa...
Given the scale of change, I would like to call a code freeze to
facilitate this change.
I would like to freeze the code from now on till the task is complete.
Thanks,
Samisa...
Nandika Jayawardana wrote:
> +1 for change.
>
> nandika
>
> On 5/31/06, *Samisa Abeysinghe* <samisa.abeysinghe@gmail.com
> <ma...@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
> <ma...@ws.apache.org>
> For additional commands, e-mail: axis-c-dev-help@ws.apache.org
> <ma...@ws.apache.org>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-c-dev-help@ws.apache.org