axis-c-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Samisa Abeysinghe <samisa.abeysin...@gmail.com>
Subject Re: [Axis2] Code freeze - Replacing double pointer env param to a single pointer env param
Date Thu, 01 Jun 2006 01:35:53 GMT
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 
> <mailto: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
>     <mailto:axis-c-dev-unsubscribe@ws.apache.org>
>     For additional commands, e-mail: axis-c-dev-help@ws.apache.org
>     <mailto:axis-c-dev-help@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


Mime
View raw message