+1 for change.

nandika

On 5/31/06, Samisa Abeysinghe <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Reply via email to