Some strange behaviors

[GeO]

Currently i found some strange behaviors using the new CC Branch of Code::Blocks:
1) if i enter

Code Select Expand

while () and then press "." it will be autocompleted as

Code Select Expand

while ().while
2) if i enter "ret" and then press "CTRL-SPACE" (for autocompletion) it shows me: return ,SHORT ,short_f.....
(see attachment)

[attachment deleted by admin]

[ollydbg]

I can confirm the first issue.
But, if there's no space between "while" and "(", then it has no error tips.

For the second issue, in my Chinese system Windows XP, ctrl + space will do a input method switch, so, I cant reproduce it. Continuing enter "retu" will give the right tip "return".

Edit
After change the short cut to "ALT ＋／", I can't reproduce the second issue. It can give the correct tip list with the prefix I entered.




[attachment deleted by admin]

[blueshake]

This  not only happened in the new CC Branch of Code::Blocks,but also in normal version.

[blueshake]

I found all the keywords had this problem. and the keywords will show twice .for example .
when you type cla and the codecompletion tip will show two class.

[ollydbg]

I found all the keywords had this problem. and the keywords will show twice .for example .
when you type cla and the codecompletion tip will show two class.

Really? But I have only one "class" in the tip list.



[attachment deleted by admin]

[blueshake]

That is right.I download the svn 5716 Nightly builds.and it still exist ,it is strange .
see the attachment .

[attachment deleted by admin]

[ollydbg]

That's strange. My local copy was using "trunk + cc_branch + my own patch" , so...

[blueshake]

Currently i found some strange behaviors using the new CC Branch of Code::Blocks:
1) if i enter
Code:

while ()

and then press "." it will be autocompleted as
Code:

while ().while

this problem can be solved by this patch.

Code Select Expand

Index: nativeparser.cpp
===================================================================
--- nativeparser.cpp (revision 5744)
+++ nativeparser.cpp (working copy)
@@ -1214,8 +1214,8 @@
                         case '(': --nest; break;
                     }
                 }
-                if ((x > 0) && (wxIsalnum(line.GetChar(x - 1)) || line.GetChar(x - 1) == '_'))
-                    --x;
+                //if ((x > 0) && (wxIsalnum(line.GetChar(x - 1)) || line.GetChar(x - 1) == '_'))
+                //    --x;
             }
         }
     }


[ollydbg]

Can you explain a little about this modify on the function:

Code Select Expand


unsigned int NativeParser::FindCCTokenStart(const wxString& line)
{
    ......
    return x;
}


Thanks!

[ollydbg]

The function is list below:

Code Select Expand


unsigned int NativeParser::FindCCTokenStart(const wxString& line)
{
   int x = line.Length() - 1;
   int nest = 0;

   bool repeat = true;
   while (repeat)
   {
       repeat = false;
       while ((x >= 0) && (wxIsalnum(line.GetChar(x)) || line.GetChar(x) == '_'))
           --x;

       if ( (x > 0) &&
            ( (line.GetChar(x) == '>' && line.GetChar(x - 1) == '-') ||
              (line.GetChar(x) == ':' && line.GetChar(x - 1) == ':') ) )
       {
           x -= 2;
           repeat = true;
       }
       else if ((x >= 0) && (line.GetChar(x) == '.'))
       {
           --x;
           repeat = true;
       }

       if (repeat)
       {
           // now we 're just before the "." or "->" or "::"
           // skip any whitespace
           while ((x >= 0) && (line.GetChar(x) == ' ' || line.GetChar(x) == '\t'))
               --x;

           // check for function/array/cast ()
           if ((x >= 0) && (line.GetChar(x) == ')' || line.GetChar(x) == ']'))
           {
               ++nest;
               while (--x >= 0 && nest != 0)
               {
                   #if wxCHECK_VERSION(2, 9, 0)
                   switch (line.GetChar(x).GetValue())
                   #else
                   switch (line.GetChar(x))
                   #endif
                   {
                       case ']':
                       case ')': ++nest; break;

                       case '[':
                       case '(': --nest; break;
                   }
               }
               if ((x > 0) && (wxIsalnum(line.GetChar(x - 1)) || line.GetChar(x - 1) == '_'))
                   --x;
           }
       }
   }
   ++x;

   if (x < 0)
       x = 0;

   while (line.GetChar(x) == ' ' || line.GetChar(x) == '\t')
       ++x;

   //Manager::Get()->GetLogManager()->DebugLog("Starting at %d \"%s\"", x, line.Mid(x).c_str());
   return x;
}


I think we need to "skip any whitespace" after we find the "[" in the following line

Code Select Expand


while     ().


That is what I think is right :

Code Select Expand


               while (--x >= 0 && nest != 0)
               {
                   #if wxCHECK_VERSION(2, 9, 0)
                   switch (line.GetChar(x).GetValue())
                   #else
                   switch (line.GetChar(x))
                   #endif
                   {
                       case ']':
                       case ')': ++nest; break;

                       case '[':
                       case '(': --nest; break;
                   }
               }
               //***************ADD code here:
               // skip any whitespace
               while ((x >= 0) && (line.GetChar(x) == ' ' || line.GetChar(x) == '\t'))
                    --x;
                //**************
               ......




[blueshake]

so should we  add the codes

Code Select Expand

               while ((x >= 0) && (line.GetChar(x) == ' ' || line.GetChar(x) == '\t'))
                    --x;
to the function or others?

[MortenMacFly]

Both of you got me irritated by now... So I am simply looking forward to a new patch... ;-)

[ollydbg]

Step into the CC plugin when I enter the "." after

Code Select Expand


while   ()


I find that the "BreakUpComponents" function will give the components as "while".

See the screenshot


So, the code-completion related code will firstly search the global token trees, and the Keywords list, finally, it find one match "while".

So, the wrong call tip displayed.

I think the expect result that BreakUpcomponents return should be *empty* instead of a "while".

[blueshake]

i think the problem can be solved by this patch too.
which one is better, still need to further discussed.

Code Select Expand

Index: nativeparser.cpp
===================================================================
--- nativeparser.cpp (revision 5744)
+++ nativeparser.cpp (working copy)
@@ -1248,21 +1247,20 @@
             ++startAt;
         }
     }
-
-//    Manager::Get()->GetLogManager()->DebugLog(_T("at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str());
-    while ((startAt < line.Length()) && (wxIsalnum(line.GetChar(startAt)) || line.GetChar(startAt) == '_'))
+    //Manager::Get()->GetLogManager()->DebugLog(F(_T("at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str()));
+    while ((startAt < line.Length()) && (wxIsalnum(line.GetChar(startAt)) || line.GetChar(startAt) == '_' || line.GetChar(startAt) == ' ' || line.GetChar(startAt) == '\t'))
     {
         res << line.GetChar(startAt);
         ++startAt;
     }
-
+    res.Trim();
     while ((nest > 0) && (startAt < line.Length()))
     {
         if (line.GetChar(startAt) == ')')
             --nest;
         ++startAt;
     }
-    //Manager::Get()->GetLogManager()->DebugLog("Done nest: at %d (%c): res=%s", startAt, line.GetChar(startAt), res.c_str());
+    //Manager::Get()->GetLogManager()->DebugLog(F(_T("Done nest: at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str()));

     if ((startAt < line.Length()) && (line.GetChar(startAt) == '(' || line.GetChar(startAt) == '['))
     {


[MortenMacFly]

Code Select Expand

Index: nativeparser.cpp
===================================================================
--- nativeparser.cpp (revision 5744)
+++ nativeparser.cpp (working copy)
@@ -1248,21 +1247,20 @@
             ++startAt;
         }
     }
-
-//    Manager::Get()->GetLogManager()->DebugLog(_T("at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str());
-    while ((startAt < line.Length()) && (wxIsalnum(line.GetChar(startAt)) || line.GetChar(startAt) == '_'))
+    //Manager::Get()->GetLogManager()->DebugLog(F(_T("at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str()));
+    while ((startAt < line.Length()) && (wxIsalnum(line.GetChar(startAt)) || line.GetChar(startAt) == '_' || line.GetChar(startAt) == ' ' || line.GetChar(startAt) == '\t'))
     {
         res << line.GetChar(startAt);
         ++startAt;
     }
-
+    res.Trim();
     while ((nest > 0) && (startAt < line.Length()))
     {
         if (line.GetChar(startAt) == ')')
             --nest;
         ++startAt;
     }
-    //Manager::Get()->GetLogManager()->DebugLog("Done nest: at %d (%c): res=%s", startAt, line.GetChar(startAt), res.c_str());
+    //Manager::Get()->GetLogManager()->DebugLog(F(_T("Done nest: at %d (%c): res=%s"), startAt, line.GetChar(startAt), res.c_str()));

     if ((startAt < line.Length()) && (line.GetChar(startAt) == '(' || line.GetChar(startAt) == '['))
     {


...so is this the only thing to apply and the other two modifications are obsolete?