Patch #3392 2012-12-07 17:54

p2rkw

CC, classbrowser:use Token::m_FullType instead of m_BaseType
Download
3392-CC_classbrowse.patch (950 bytes)
Category
Plugin::Refinement
Status
Accepted
Close date
2012-12-16 14:04
Assigned to
mortenmacfly
Index: src/plugins/codecompletion/classbrowserbuilderthread.cpp
===================================================================
--- src/plugins/codecompletion/classbrowserbuilderthread.cpp    (wersja 8640)
+++ src/plugins/codecompletion/classbrowserbuilderthread.cpp    (kopia robocza)
@@ -1039,8 +1039,8 @@
             // modification suggested by ollydbg in http://forums.codeblocks.org/index.php/topic,10242.msg70865.html#msg70865:
 //            if (!token->m_ActualType.IsEmpty())
 //                 str = str + _T(" : ") + token->m_ActualType;
-            if (!token->m_BaseType.IsEmpty())
-                str = str + _T(" : ") + token->m_BaseType + token->m_TemplateArgument;
+            if (!token->m_FullType.IsEmpty())
+                str = str + _T(" : ") + token->m_FullType + token->m_TemplateArgument;
 
             wxTreeItemId child = tree->AppendItem(parent, str, img, img, new CCTreeCtrlData(sfToken, token, tokenKindMask));
 
mortenmacfly 2012-12-13 14:07

...why?

ID_100 2012-12-14 17:38

Because, if you have function that returns pointer or reference:

void* getObject() { /*...*/ );

then m_BaseType is "void", so class browser will display this function as:

getObject(): void

And this suggests that this function returns nothing. Same thing with variables.

It's may be bit confusing.

mortenmacfly 2012-12-14 17:56

Ah - OK, that seems valid. I'll take care and do some testing...