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

List:       kde-commits
Subject:    KDE/kdelibs/nepomuk/query
From:       Allen Winter <winter () kde ! org>
Date:       2010-11-21 19:42:35
Message-ID: 20101121194235.61F49AC8A0 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1199398 by winterz:

don't reuse the method argument "term" in the foreach loops.
legal, but generates a unnecessary warning and I find it confusing anyway.



 M  +4 -4      queryserializer.cpp  


--- trunk/KDE/kdelibs/nepomuk/query/queryserializer.cpp #1199397:1199398
@@ -116,16 +116,16 @@
 
         case Term::And:
             xml.writeStartElement( QLatin1String("and") );
-            Q_FOREACH( const Term& term, term.toAndTerm().subTerms() ) {
-                doSerializeTerm( xml, term );
+            Q_FOREACH( const Term& myterm, term.toAndTerm().subTerms() ) {
+                doSerializeTerm( xml, myterm );
             }
             xml.writeEndElement();
             break;
 
         case Term::Or:
             xml.writeStartElement( QLatin1String("or") );
-            Q_FOREACH( const Term& term, term.toOrTerm().subTerms() ) {
-                doSerializeTerm( xml, term );
+            Q_FOREACH( const Term& myterm, term.toOrTerm().subTerms() ) {
+                doSerializeTerm( xml, myterm );
             }
             xml.writeEndElement();
             break;
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic