Developer forums (C::B DEVELOPMENT STRICTLY!) > CodeCompletion redesign
New code completion remarks/issues
blueshake:
--- Quote ---I'm not satisfied with this, maybe, I need to implement a macro replacement rule for the "CVAPI" macro. Wink
--- End quote ---
we can make a compare here.
if the position of token (void) and token cvFunction are in the same line.keep the CVAPI(void) in m_Str and continue to parse.
MortenMacFly:
--- Quote from: blueshake on December 02, 2009, 01:59:36 am ---we can make a compare here.
--- End quote ---
I'd rather go with the token replacement as suggested by OllyDgb. This is configurable and we don't need to implement particular CV SDK hocus-pocus in the plugin. Thus the plugin should not know antyhing about CV. OR make it more generic so that it applies to other similar constructs as well. But this might break something else.
Monolith:
I have notice a couple issues.
(1) There is a problem with typedefs with multiple definitions. For instance, typedef WNDCLASSEXW WNDCLASSEX,*LPWNDCLASSEX,*PWNDCLASSEX; wont get recognized by code complete, however, typedef WNDCLASSEXW WNDCLASSEX; will work.
(2) The parser should recognize definitions, the difference between UNICODE and not UNICODE. For instance, I have a ANSI app, but I had to edit the UNICODE typedef of WNDCLASSEX to get the Code Complete to work.
(3) #define should be recognized as its counterparts. (#define MessageBox MessageBoxW will not be recognized as MessageBoxW.)
You guys have made great strides lately, keep up the good work.
oBFusCATed:
2 and 3 are known problems -> there is no code written to parse the #defines, as far as I know
blueshake:
--- Quote ---(1) There is a problem with typedefs with multiple definitions. For instance, typedef WNDCLASSEXW WNDCLASSEX,*LPWNDCLASSEX,*PWNDCLASSEX; wont get recognized by code complete, however, typedef WNDCLASSEXW WNDCLASSEX; will work.
--- End quote ---
um,confirm it.That is because the "," is not handled.thanks for your report.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version