Please help me fix it, I find the reasons!

[Loaden]

I found that VC compiler auto-complete function can not be used for it.
Because the VC header files is special, example:

Code Select Expand

class ATL_NO_VTABLE CHyperLinkImpl : public ATL::CWindowImpl< T, TBase, TWinTraits >
{
public:
LPTSTR m_lpstrLabel;
LPTSTR m_lpstrHyperLink;

HCURSOR m_hCursor;
HFONT m_hFont;
HFONT m_hFontNormal;
if remove ATL_NO_VTABLE, it's became:

Code Select Expand

class CHyperLinkImpl : public ATL::CWindowImpl< T, TBase, TWinTraits >
{
public:


it's work fine!
Have any way to fix it?
Thanks!!

[MortenMacFly]

Have any way to fix it?

You can try the CC replacements -> replace ATL_NO_VTABLE with "" (nothing).
Settings -> Editor -> CC & Symbols browser -> Tab "C++ parser" -> Add a "replacement" token.

[Loaden]

Have any way to fix it?

You can try the CC replacements -> replace ATL_NO_VTABLE with "" (nothing).
Settings -> Editor -> CC & Symbols browser -> Tab "C++ parser" -> Add a "replacement" token.

Thanks!
But i can't add replacement ATL_NO_VTABLE with nothing.


[attachment deleted by admin]

[ollydbg]

In the ccoptiondlg.cpp.

Code Select Expand

bool CCOptionsDlg::ValidateReplacementToken(wxString& from, wxString& to)
{
    wxRegEx re(_T("[A-Za-z_]+[0-9]*[A-Za-z_]*"));
    from.Trim(true).Trim(false);
    to.Trim(true).Trim(false);
    if (!re.Matches(from) || !re.Matches(to))
    {
        cbMessageBox(_("Replacement tokens can only contain alphanumeric characters and underscores..."),
                    _("Error"), wxICON_ERROR);
        return false;
    }
    return true;
}

That's the cause of your problem.

[Loaden]

In the ccoptiondlg.cpp.

Code Select Expand

bool CCOptionsDlg::ValidateReplacementToken(wxString& from, wxString& to)
{
    wxRegEx re(_T("[A-Za-z_]+[0-9]*[A-Za-z_]*"));
    from.Trim(true).Trim(false);
    to.Trim(true).Trim(false);
    if (!re.Matches(from) || !re.Matches(to))
    {
        cbMessageBox(_("Replacement tokens can only contain alphanumeric characters and underscores..."),
                    _("Error"), wxICON_ERROR);
        return false;
    }
    return true;
}

That's the cause of your problem.

Thanks!
I comment this code, and do it ok.
Now, class ATL_NO_VTABLE CHyperLinkImpl : public ATL::CWindowImpl< T, TBase, TWinTraits > is OK!

But, other headers, egg. string winuser.h, i don't know how to make it work.

Code Select Expand

WINUSERAPI
int
WINAPI
MessageBoxA(
    __in_opt HWND hWnd,
    __in_opt LPCSTR lpText,
    __in_opt LPCSTR lpCaption,
    __in UINT uType);
WINUSERAPI
int
WINAPI
MessageBoxW(
    __in_opt HWND hWnd,
    __in_opt LPCWSTR lpText,
    __in_opt LPCWSTR lpCaption,
    __in UINT uType);
#ifdef UNICODE
#define MessageBox  MessageBoxW
#else
#define MessageBox  MessageBoxA
#endif // !UNICODE

[Loaden]

I find: if the header file size more than 132KB, CC will not work!!
egg. Shlwapi.h，and CC work fine.

and if cut winuser.h from line 1 to line 8446, so, CC work fine.
Why?
need int to long long?

[ollydbg]

What do you mean by "CC will not work!!"?

It seems the Parser can't recognize the function argument like this:

Code Select Expand

MessageBoxA(
    __in_opt HWND hWnd,
    __in_opt LPCSTR lpText,
    __in_opt LPCSTR lpCaption,
    __in UINT uType);

The related implementation was in

void ParserThread::HandleFunction(const wxString& name, bool isOperator)

So, you can exam these code carefully to see the reason.

[MortenMacFly]

Code Select Expand

MessageBoxA(
    __in_opt HWND hWnd,
    __in_opt LPCSTR lpText,
    __in_opt LPCSTR lpCaption,
    __in UINT uType);

Try replacing this "__in_opt" and alike with "empty", too... ;-) :lol: :lol: :lol:

[Loaden]

Code Select Expand

MessageBoxA(
    __in_opt HWND hWnd,
    __in_opt LPCSTR lpText,
    __in_opt LPCSTR lpCaption,
    __in UINT uType);

Try replacing this "__in_opt" and alike with "empty", too... ;-) :lol: :lol: :lol:

It does not parameters question, it is the analysis of the header file is too large, it can not be fully analyzed with CC.

[ollydbg]

Hi, all.

Today, I'm thinking how to refactor the replacement function in

In Tokenizer.cpp GetToken() function

Code Select Expand


   inline const wxString& ThisOrReplacement(const wxString& str) const
   {
       ConfigManagerContainer::StringToStringMap::const_iterator it = s_Replacements.find(str);
       if (it != s_Replacements.end())
           return it->second;
       return str;
   }


I think this function is some kind too simple. At least, it can do a search, then a replacement.

(I have exam the code in ctags and its manual, especially in it's code to do Macro replacement)
Here
http://ctags.sourceforge.net/ctags.html#OPERATIONAL%20DETAILS

−I identifier−list

Specifies a list of identifiers which are to be specially handled while parsing C and C++ source files. This option is specifically provided to handle special cases arising through the use of preprocessor macros.

When the identifiers listed are simple identifiers, these identifiers will be ignored during parsing of the source files.

If an identifier is suffixed with a '+' character, ctags will also ignore any parenthesis-enclosed argument list which may immediately follow the identifier in the source files.

If two identifiers are separated with the '=' character, the first identifiers is replaced by the second identifiers for parsing purposes.

So, at least, we can do the same thing like in Ctags.

In the option.c of Ctags source code, there are the related codes to doing identifier replacement.

Code Select Expand


/*  Determines whether or not "name" should be ignored, per the ignore list.
*/
extern boolean isIgnoreToken (
const char *const name, boolean *const pIgnoreParens,
const char **const replacement)
{
boolean result = FALSE;

if (Option.ignore != NULL)
{
const size_t nameLen = strlen (name);
unsigned int i;

if (pIgnoreParens != NULL)
*pIgnoreParens = FALSE;

for (i = 0  ;  i < stringListCount (Option.ignore)  ;  ++i)
{
vString *token = stringListItem (Option.ignore, i);

if (strncmp (vStringValue (token), name, nameLen) == 0)
{
const size_t tokenLen = vStringLength (token);

if (nameLen == tokenLen)
{
result = TRUE;
break;
}
else if (tokenLen == nameLen + 1  &&
vStringChar (token, tokenLen - 1) == '+')
{
result = TRUE;
if (pIgnoreParens != NULL)
*pIgnoreParens = TRUE;
break;
}
else if (vStringChar (token, nameLen) == '=')
{
if (replacement != NULL)
*replacement = vStringValue (token) + nameLen + 1;
break;
}
}
}
}
return result;
}


Here is what we can do in our Code completion code:

Code Select Expand


   inline const wxString& ThisOrReplacement(const wxString& str) const
   {
       ConfigManagerContainer::StringToStringMap::const_iterator it = s_Replacements.find(str);
       if (it != s_Replacements.end()){

           //Here we should example the "it->second"
           // if it is a "=new_token" string, the current_token should be replaced by "new_token"
           // if it is a "-" , the current string should be ignored, then we should return next token
           // if it is a "+",  the next "parenthesis-enclosed argument list" should be ignored
           // if it match any other pre-defined string grama, we will do other thing , eg, expand the current   token to two tokens...
           //......
           //return it->second;
       }
       return str;
   }



Any Comments?

[ollydbg]

@loaden:

I copy a large header from "Visual studio C++ header folder" which named "WINUSER.h" (232KB)

And I paste to the root of my test project folder, and renamed to "testWINUSER.h"

Then create a simple project

Code Select Expand


#include "testWINUSER.H"
int main()
{
    UserHandleGrantA  //code completion works fine!
    return 0;
}


And every thing works fine..  At least the last function declaration in that header file is

Code Select Expand


BOOL UserHandleGrantAccess(
    HANDLE hUserHandle,
    HANDLE hJob);


And code completion works fine! This means the totally 232K files were parsed, not break in the middle.

[blueshake]

I do a patch for this ,class CHyperLinkImpl can be parsed correctly  now.

[ollydbg]

I do a patch for this ,class CHyperLinkImpl can be parsed correctly  now.

where can I find this patch??

[Jenna]

I do a patch for this ,class CHyperLinkImpl can be parsed correctly  now.

where can I find this patch??

http://forums.next.codeblocks.org/index.php/topic,10896.msg74448/topicseen.html#msg74448

[Loaden]

Thanks to ollydbg!
[removed non-english content]

It's seems have other reason, but not the file size.
It's maybe: max allowed matches. by default, it's be 16340, now i change the value to : 100000