You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by na...@apache.org on 2006/03/29 07:41:39 UTC

svn commit: r389680 - in /webservices/axis2/trunk/c: include/axis2_om_stax_builder.h modules/util/qname.c modules/xml/om/om_stax_builder.c

Author: nandika
Date: Tue Mar 28 21:41:36 2006
New Revision: 389680

URL: http://svn.apache.org/viewcvs?rev=389680&view=rev
Log:
om_stax_builder.c set_document removed , memory leak fixed 

Modified:
    webservices/axis2/trunk/c/include/axis2_om_stax_builder.h
    webservices/axis2/trunk/c/modules/util/qname.c
    webservices/axis2/trunk/c/modules/xml/om/om_stax_builder.c

Modified: webservices/axis2/trunk/c/include/axis2_om_stax_builder.h
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/include/axis2_om_stax_builder.h?rev=389680&r1=389679&r2=389680&view=diff
==============================================================================
--- webservices/axis2/trunk/c/include/axis2_om_stax_builder.h (original)
+++ webservices/axis2/trunk/c/include/axis2_om_stax_builder.h Tue Mar 28 21:41:36 2006
@@ -92,20 +92,6 @@
 		get_document)(struct axis2_om_stax_builder *builder,
                       axis2_env_t **env);
         /**
-         *  set the document associated with the builder
-         * @param builder pointer to builder struct
-         * @param env environment , MUST NOT be NULL.
-         * @param document pointer to document struct that needs to be associated 
-         *                  with builder
-         * @return status of the op. AXIS2_SUCCESS on success and AXIS2_FAILURE 
-         *                  on error.
-         */                                          
-
-        axis2_status_t  (AXIS2_CALL *
-		set_document)(struct axis2_om_stax_builder *builder,
-                      axis2_env_t **env,
-                      axis2_om_document_t *document); 
-        /**
          * builder is finished building om structure
          */        
                                              
@@ -152,9 +138,7 @@
 /** free op of the builder */
 #define AXIS2_OM_STAX_BUILDER_FREE(builder,env) \
         ((builder)->ops->free(builder,env))
-/** associate a document with a builder */       
-#define AXIS2_OM_STAX_BUILDER_SET_DOCUMENT(builder,env,document) \
-        ((builder)->ops->set_document(builder,env,document))
+
 /** get the document associated with the builder */  
 #define AXIS2_OM_STAX_BUILDER_GET_DOCUMENT(builder,env) \
         ((builder)->ops->get_document(builder,env))

Modified: webservices/axis2/trunk/c/modules/util/qname.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/util/qname.c?rev=389680&r1=389679&r2=389680&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/util/qname.c (original)
+++ webservices/axis2/trunk/c/modules/util/qname.c Tue Mar 28 21:41:36 2006
@@ -363,7 +363,7 @@
         if(NULL != temp_string3)
         {
             AXIS2_FREE((*env)->allocator, temp_string2);
-            temp_string2 = NULL;
+            temp_string3 = NULL;
         }
     }     
     return qname_impl->qname_string;        

Modified: webservices/axis2/trunk/c/modules/xml/om/om_stax_builder.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/xml/om/om_stax_builder.c?rev=389680&r1=389679&r2=389680&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/xml/om/om_stax_builder.c (original)
+++ webservices/axis2/trunk/c/modules/xml/om/om_stax_builder.c Tue Mar 28 21:41:36 2006
@@ -43,12 +43,6 @@
 axis2_om_stax_builder_get_document (axis2_om_stax_builder_t *builder,
                                     axis2_env_t **env);
                                             
-                                            
-axis2_status_t  AXIS2_CALL
-axis2_om_stax_builder_set_document(axis2_om_stax_builder_t *builder,
-                                    axis2_env_t **env,
-                                    axis2_om_document_t *document);                           
-   
 axis2_bool_t  AXIS2_CALL
 axis2_om_stax_builder_is_complete(axis2_om_stax_builder_t *builder,
                                    axis2_env_t **env);
@@ -144,10 +138,7 @@
         
 	builder->om_stax_builder.ops->free = 
 	        axis2_om_stax_builder_free;
-	
-    builder->om_stax_builder.ops->set_document = 
-	        axis2_om_stax_builder_set_document;
-	
+
     builder->om_stax_builder.ops->get_document = 
             axis2_om_stax_builder_get_document;
     
@@ -842,20 +833,7 @@
     AXIS2_ENV_CHECK(env,NULL);
     return AXIS2_INTF_TO_IMPL(builder)->document;
 }
-/**
-    This is an internal function
-*/
-axis2_status_t  AXIS2_CALL
-axis2_om_stax_builder_set_document(axis2_om_stax_builder_t *builder,
-                                    axis2_env_t **env,
-                                    axis2_om_document_t *document)
-{
-    AXIS2_ENV_CHECK(env,AXIS2_FAILURE);
-    AXIS2_PARAM_CHECK((*env)->error, builder, AXIS2_FAILURE);
-    AXIS2_PARAM_CHECK((*env)->error, document , AXIS2_FAILURE);
-    AXIS2_INTF_TO_IMPL(builder)->document = document ;
-    return AXIS2_SUCCESS;
-}
+
 /**
     This is an internal function 
 */