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

Dinesh Premalal updated AXIS2C-786:
-----------------------------------

    Fix Version/s:     (was: Current (Nightly))
                   1.2.0
         Assignee: Dinesh Premalal

> 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
>             Fix For: 1.2.0
>
>
> 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