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

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


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


One more little thing, please try to use the same coding style as the surrounding code, else the file will become very ugly. In this specific case this means adding spaces around operators.

- 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