axis-c-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Samisa Abeysinghe (JIRA)" <j...@apache.org>
Subject [jira] Commented: (AXIS2C-54) include files in src folders
Date Fri, 03 Feb 2006 00:42:03 GMT
    [ 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


Mime
View raw message