In codecompletion.cpp there are four regex similar to this:
wxRegEx ppIf(wxT("^[ \t]*#[ \t]*if"));
IMHO the TAB character is injected in a hackish way, that may fail in the future (in fact it is impossible to debug these expressions in the regex test bed). The proper way to match a TAB is using the \t
regex escape sequence, which needs two backslashes in a C string:
wxRegEx ppIf(wxT("^[ \\t]*#[ \\t]*if"));
but this sequence only works in wxRE_ADVANCED mode. Given the regex is trying to match white space, the best and fully compatible way is:
wxRegEx ppIf(wxT("^[[:blank:]]*#[[:blank:]]*if"));
Tha attached patch fixes the four strings
This new version removes
_()
andwxT()
also.Hi, Miguel Gimenez, since you are C::B developer now, so I think you can commit this patch.
Thanks.
I see this patch is OK from my side.
Thank you
Fixed in r12545