[ 
http://issues.apache.org/jira/browse/AXIS2C-54?page=comments#action_12365021 ] 

Samisa Abeysinghe commented on AXIS2C-54:
-----------------------------------------

I thought WSDL module is an independant module. Hence wsdl related headers 
should never refer to description related headers. Hence " 
axis2_wsdl_compopent.h refers to description/axis2_description.h " should be 
solved.

axis2_phase_holder.h refers to wsdl/axis2_wsdl.h  also does not make sense. 
Only description should be using wsdl, phase should only deal with description. 
By the way, the fact that description structs have the wsdl related structs 
appearing in the header would be a problem here. We have to strictly enforce 
the data hiding in that case and move the  wsdl related structs to impl 
description structs in c source.

I hope "axis2_transport_out_desc.h refers to 
transport/axis2_transport_sender.h" could be solved using 'void*'

> include files in src folders
> ----------------------------
>
>          Key: AXIS2C-54
>          URL: http://issues.apache.org/jira/browse/AXIS2C-54
>      Project: Axis2-C
>         Type: Bug
>   Components: core/description
>     Versions: M0.3
>  Environment: windows
>     Reporter: nabeel yoosuf

>
> axis2_wsdl_compopent.h refers to description/axis2_description.h
> axis2_phase_holder.h refers to wsdl/axis2_wsdl.h
> axis2_transport_in_desc.h refers to transport/axis2_transport_receiver.h
> axis2_transport_out_desc.h refers to transport/axis2_transport_sender.h
> These header files should be moved to include dir.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to