[ 
https://issues.apache.org/jira/browse/AXIS2C-786?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Samisa Abeysinghe resolved AXIS2C-786.
--------------------------------------

    Resolution: Won't Fix

This was done in the past for API consistency. If we are to change this, we 
have to do it in 2.0 release, but not in 1.x family as it is a considerable API 
change.

> axiom_element_set_text() and other functions that do not make use of an 
> axiom_element_t* object in its parameters
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: AXIS2C-786
>                 URL: https://issues.apache.org/jira/browse/AXIS2C-786
>             Project: Axis2-C
>          Issue Type: Improvement
>          Components: xml/soap
>    Affects Versions: Current (Nightly)
>            Reporter: Edward Liu
>            Assignee: Dinesh Premalal
>            Priority: Minor
>
> There are some functions, such as axiom_element_set_text(), that take in an 
> axiom_element_t* object as one of its parameters, but does not make use of it 
> (after viewing the Axis2-C source code).  It is pretty inconvenient and 
> unnecessary (in my opinion) to create an axiom_element_t* object, pass it 
> into axiom_element_set_text(), and then free it, without using this object 
> anywhere else except as an argument to a function that doesn't even use the 
> object itself.  I think that this creation and freeing of an axiom_element_t* 
> object could be avoided.  After talking with one of the developers, it is 
> recommended that the axiom_element_t* argument be removed from those 
> functions that have it as a parameter but make no use of it.  I believe that 
> passing NULL into these affected functions is a temporary workaround, not a 
> permanent solution, for this issue.  Thanks!

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to