User forums > Nightly builds

The 01 September 2015 build (10474) is out.

<< < (4/4)

ollydbg:

--- Quote from: ollydbg on September 05, 2015, 04:47:50 am ---The second issue is that it looks like the "*“ is lost in the Token.
...

--- End quote ---
Tested again, I see that "*" is only exists in "m_FullType", not in "m_BaseType", then with the following patch, you issue can be solved.

--- Code: --- src/plugins/codecompletion/codecompletion.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/plugins/codecompletion/codecompletion.cpp b/src/plugins/codecompletion/codecompletion.cpp
index 4814840..8434566 100644
--- a/src/plugins/codecompletion/codecompletion.cpp
+++ b/src/plugins/codecompletion/codecompletion.cpp
@@ -2742,7 +2742,8 @@ int CodeCompletion::DoAllMethodsImpl()
                 str << ed->GetLineIndentString(line - 1);
             if (addDoxgenComment)
                 str << _T("/** @brief ") << token->m_Name << _T("\n  *\n  * @todo: document this function\n  */\n");
-            wxString type = token->m_BaseType;
+            wxString type = token->m_FullType;
+            // "int *" or "int &" ->  "int*" or "int&"
             if ((type.Last() == _T('&') || type.Last() == _T('*')) && type[type.Len() - 2] == _T(' '))
             {
                 type[type.Len() - 2] = type.Last();

--- End code ---
I change the base type to full type.

Xaviou:

--- Quote from: ollydbg on September 05, 2015, 04:47:50 am ---@Xaviou
The first issue is fixed in trunk now.
--- End quote ---
Thank you for your reactivity.


--- Quote from: ollydbg on September 05, 2015, 04:47:50 am ---The second issue is that it looks like the "*“ is lost in the Token.
--- End quote ---
I don't know if it can help, but I've just saw that the dialog box witch allows to check/uncheck the missing methods we want to add has the correct definition.
Only the inserted code is wrong.

Regards
Xav'

ollydbg:

--- Quote from: Xaviou on September 05, 2015, 09:33:13 am ---...
Only the inserted code is wrong.
...

--- End quote ---
I fix this issue in rev10484.

Navigation

[0] Message Index

[*] Previous page