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

List:       kde-commits
Subject:    branches/kdevelop/3.4/languages/cpp
From:       David Nolden <david.nolden.kde () art-master ! de>
Date:       2007-01-19 19:00:06
Message-ID: 1169233206.434945.7424.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 625278 by zwabel:

If a list of overloaded functions was found in one class, do not complete that list \
by searching the parents. That does not comply with the C++ Standard because one \
function in a child-class overloads all functions of that name in the parent-class. \
This patch should also finally fix all hangs that had to do with \
appendNextFunction(...), those were only workarounded before. BUG: 78641

 M  +0 -10     simpletype.cpp  


--- branches/kdevelop/3.4/languages/cpp/simpletype.cpp #625277:625278
@@ -248,16 +248,6 @@
       ///For functions, find all functions with the same name, so that overloaded \
functions can be identified correctly  TypePointer ret = mem.build();
       if ( ret && ret->asFunction() ) {
-        ///Search all bases and append all functions with the same name to it.
-        QValueList<LocateResult> bases = getBases();
-        for ( QValueList<LocateResult>::iterator it = bases.begin(); it != \
                bases.end(); ++it ) {
-          if( !safetyCounter ) break;
-          if ( ( *it ) ->resolved() ) {
-            TypeOfResult rt = ( *it ) ->resolved() ->typeOf( name );
-            if ( rt->resolved() )
-              ret->asFunction() ->appendNextFunction( SimpleType( rt->resolved() ) \
                );
-          }
-        }
         return TypeOfResult( LocateResult( ret->desc() ) );
       } else {
         ifVerbose( dbg() << "error, using old function-type-evaluation" << endl );


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