I would like to announce that the code freeze is over.
I have successfully updated the double pointer to a const pointer for env.
All the samples are working fine. This means util, axiom and the core
modules are building and working successfully.
Unfortunately, despite the code freeze, there had been a commit on
xml_schema and hence that would be broken. Please fix that.
Woden stuff is also changed, but could not build and test as xml_schema
is broken.
Thanks,
Samisa...
Samisa Abeysinghe wrote:
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* <[EMAIL PROTECTED]
<mailto:[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]
<mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: [EMAIL PROTECTED]
<mailto:[EMAIL PROTECTED]>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
