Review Request: When declaration not found, look at the similar ones

David Nolden zwabel+reviewboard at gmail.com
Sun Apr 19 22:18:17 UTC 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/613/#review970
-----------------------------------------------------------


Just to give you some more explanation: This does not yet work for all cases. If you use that "searchFrom" context, and there is no container context, then you have to consider all declarations that are visible from within that context(An extended version DUContext::allDeclarations() is needed), not only the local declarations of that context. But when you do that, you will have a very long list, and this will be too slow for being done in the foreground thread.

- David


On 2009-04-18 20:13:08, Aleix Pol wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/613/
> -----------------------------------------------------------
> 
> (Updated 2009-04-18 20:13:08)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> Haven't you ever mistyped a class member and got an error message for that?
> Well, this patch tries to help the user by searching similar names and giving a hint.
> 
> It also works for variables.
> 
> Hope you like it :)
> 
> The code used to get the difference was got from the wikipedia (was pseudocode i translated it into c++).
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdevelop/languages/cpp/cppduchain/missingdeclarationassistant.cpp 955802 
> 
> Diff: http://reviewboard.kde.org/r/613/diff
> 
> 
> Testing
> -------
> 
> class MyClass
> {
> 	public:
> 		void aaa();
> };
> 
> int main()
> {
> 	MyClass c;
> 	c.aab(); //Here aaa is proposed
> 	
> 	int xxx;
> 	int k=xxy; //Here xxx is proposed
> 
> 	return 0;
> }
> 
> 
> Thanks,
> 
> Aleix
> 
>





More information about the KDevelop-devel mailing list