hi,all
after a week's work,I finally finish this patch.it is the most complicated patch I had made.
as we know,the current cc use FindAIMatch function to get the matched tokens.people complain its low efficiency.
that is true because it use recursive calls.and I found it is hard to understand ,not mention to maintain it.at least for me,it is so.
so I rewrite FindAIMatch function.of course ,I abandon recursive calls.
I am not going to explain the whole search process, just started from what I changed.
1.first I use GenerateResultSet to get the tokens from search scope set.it is the same story as before.but a little different here.
I directly get the matched search text from the tree and then eliminate those which are not under the search scope set.
2.ResolveAutualType
this function is used to resolve the function/variable's actual type.
3.ResolveExpression
just the name said,it is used to resolve the expression. it is used to replace the FindAIMatch function.
if you want to ask what is the biggest different?
here is the answer.try the complained thing.you can see how fast it is. :lol:
string::append.append.append.append.append.append.
patch for it.
There is a typo? ResolveAutualType , should be ResolveActualType maybe?
Quote from: oBFusCATed on February 28, 2010, 12:05:59 PM
There is a typo? ResolveAutualType , should be ResolveActualType maybe?
you are right. :D :D :D
I have just applied in my local copy, and it works quite well.
Another update.
-make the search work fine for "this" now.
-better handle for operator overload(show or not)
e.g.
class::----------here we may want to show operator overload.
variable.--------here we do not want to show.
[attachment deleted by admin]
see the screen shot.
(http://i50.tinypic.com/ibjs54.jpg)
(http://i46.tinypic.com/24bns5v.jpg)
Nice work, I will test it soon!!!
BTW:The following messages are not relatived to current topic.but I put it here.just for reminder.
in this thread.http://forums.next.codeblocks.org/index.php/topic,11227.0.html (http://forums.next.codeblocks.org/index.php/topic,11227.0.html)
if you want to the function mentioned in the above thread,you need to use the following patch.
Index: src/plugins/codecompletion/codecompletion.cpp
===================================================================
--- src/plugins/codecompletion/codecompletion.cpp (revision 6185)
+++ src/plugins/codecompletion/codecompletion.cpp (working copy)
@@ -562,9 +562,9 @@
wxArrayInt already_registered;
std::set< wxString, std::less<wxString> > unique_strings; // check against this before inserting a new string in the list
TokensTree* tokens = parser->GetTokens();
+ m_SearchItem.clear();
for (TokenIdxSet::iterator it = result.begin(); it != result.end(); ++it)
{
- m_SearchItem.clear();
Token* token = tokens->at(*it);
if (!token || token->m_Name.IsEmpty())
continue;
Quote from: blueshake on March 03, 2010, 03:32:04 AM
BTW:The following messages are not relatived to current topic.but I put it here.just for reminder.
in this thread.http://forums.next.codeblocks.org/index.php/topic,11227.0.html (http://forums.next.codeblocks.org/index.php/topic,11227.0.html)
Ok, tested and works quite well.
Let me give a simple explanation.
For such code:
string aaaa;
aaaa.app
with autocompletion, you will get here, caret moved behind "append":
string aaaa;
aaaa.append|
Now, this this patch, you will get additional parenthesis added, so, you get caret in the "()".
string aaaa;
aaaa.append(|)
Ok,here comes another patch.
for better function args tip.
when we type "," this patch can show the tip now.quite simple here.
:D
Index: src/plugins/codecompletion/codecompletion.cpp
===================================================================
--- src/plugins/codecompletion/codecompletion.cpp (revision 6185)
+++ src/plugins/codecompletion/codecompletion.cpp (working copy)
@@ -2049,7 +2049,8 @@
++m_ActiveCalltipsNest;
ShowCallTip();
}
-
+ else if (ch == _T(','))
+ ShowCallTip();
// end calltip
else if (ch == _T(')'))
{
Quote from: ollydbg on March 03, 2010, 03:48:38 AM
Now, this this patch, you will get additional parenthesis added, so, you get caret in the "()".
string aaaa;
aaaa.append(|)
Such magic shall always be configurable. By experience I'd say that 70% of the CC users will ask how to turn that off.
whoaa so much code completion coding this is emotional, I have to compile the latest nightly to test all this goodies xD
Men i wish I could be programming in c++ again now Im all php, as3 xD