[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdepim/kode/kwsdl
From:       Tobias Koenig <tokoe () kde ! org>
Date:       2005-06-07 16:05:12
Message-ID: 1118160312.871108.17944.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 423129 by tokoe:

Fixed two small bugs:
 - don't add output functions twice
 - using lower case names for variables


 M  +9 -2      converter.cpp  


--- trunk/KDE/kdepim/kode/kwsdl/converter.cpp #423128:423129
@@ -769,8 +769,8 @@
   for ( it = parts.begin(); it != parts.end(); ++it ) {
     QString lowerName = (*it).name();
     lowerName[ 0 ] = lowerName[ 0 ].lower();
-    code += "Serializer::marshal( doc, method, \"" + (*it).name() + "\", " + \
                mNameMapper.escape( (*it).name() ) + " );";
-    code += "delete " + mNameMapper.escape( (*it).name() ) + ";";
+    code += "Serializer::marshal( doc, method, \"" + (*it).name() + "\", " + \
mNameMapper.escape( lowerName ) + " );"; +    code += "delete " + mNameMapper.escape( \
lowerName ) + ";";  }
 
   code += "qDebug( \"%s\", doc.toString().latin1() );";
@@ -789,6 +789,13 @@
   messageName[ 0 ] = messageName[ 0 ].lower();
   KODE::Function respSignal( messageName, "void", KODE::Function::Signal );
 
+  /**
+    If one output message is used by two input messages, don't define
+    it twice.
+   */
+  if ( newClass.hasFunction( respSignal.name() ) )
+    return;
+
   const Message::Part::List parts = message.parts();
   Message::Part::List::ConstIterator it;
   for ( it = parts.begin(); it != parts.end(); ++it ) {


[prev in list] [next in list] [prev in thread] [next in thread]