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

List:       kdevelop-devel
Subject:    Re: C++ duchain unit tests
From:       Nicolás_Alvarez <nicolas.alvarez () gmail ! com>
Date:       2010-06-20 0:27:13
Message-ID: AANLkTimnXUch4IOQaK7Qsarcn0QZmsQT3ymTIlLv1Dtx () mail ! gmail ! com
[Download RAW message or body]

On 6/19/10, David Nolden <zwabel@googlemail.com> wrote:
> I guess the main problem is that the old contexts either aren't
> removed from the duchain at the end of the tests, or the
> remove(topContext) doesn't work properly. This needs to be found out.
>
> The first problem could be solved by using something similar to my
> recently added "InsertIntoDUChain" class, which automatically releases
> the top-context in the destructor.

I just modified both testLocalNamespaceAlias and
testDeclareUsingNamespace to use InsertIntoDUChain (I had to remove
EnvironmentManager::init() from the test initialization to avoid an
assert about double init() call).

The problem persists. Running both tests gives different results than
running them separately.

-- 
Nicolas

-- 
KDevelop-devel mailing list
KDevelop-devel@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
[prev in list] [next in list] [prev in thread] [next in thread]