DoxyIt icon indicating copy to clipboard operation
DoxyIt copied to clipboard

Can't recognize function if the "(" written in the following lines.

Open lifishake opened this issue 9 years ago • 9 comments

Reports "Unknown exception" when function is following format:

void fun
(
int param
)
{
}

But when parenthesis and function in the same line, everything is OK.

void fun (
int param
)
{
}

Our strange coding standard. Please! Thanks in advance.

lifishake avatar Dec 25 '15 01:12 lifishake

That is quit a strange format. :) Should be an easy fix when I get time next week to check it out.

dail8859 avatar Dec 25 '15 01:12 dail8859

So it looks like there is a bug in the regular expression engine DoxyIt is using. I don't plan on digging into the regular expression parser either. So this might take a while to fix...

dail8859 avatar Dec 29 '15 21:12 dail8859

It's ok. Not a serious problem, there is no pressing needs.

lifishake avatar Jan 04 '16 01:01 lifishake

Maybe changing from trex to c++11 std::regex might solve the issue, if it is really in the regex engine itself, but that would require a major rework and check if current features are supported by std::regex.

chcg avatar Feb 07 '17 17:02 chcg

I had looked at this at one point but due to the static linking, the DLL size went up quite a bit from what I remember. This specific issues is a bit of a odd case. If the trex code had more issues I'd probably switch to something else but I'm not too worried about it currently.

dail8859 avatar Feb 07 '17 18:02 dail8859

Likely that the std::regex is much bigger than trex.

Maybe it is worth mentioning in the readme.md that a modified version 1.3 is used of trex as the version info is not part of the trex.h or .cpp.

chcg avatar Feb 07 '17 18:02 chcg

Likely that the std::regex is much bigger than trex.

Oh I'm sure it is. I went with trex because it was simple and small.

Maybe it is worth mentioning in the readme.md that a modified version 1.3 is used of trex as the version info is not part of the trex.h or .cpp.

I'm not sure what it would benefit really. And if I recall it's not like I had to make major changes to it.

dail8859 avatar Feb 07 '17 18:02 dail8859

Nice to know if somebody like me having a look at the open\fixed bugs of trex, if the one mentioned here is a known or new one.

chcg avatar Feb 07 '17 18:02 chcg

If the trex library was active and being updated I do see how that could help. AFAICT it has been abandoned for years.

dail8859 avatar Feb 07 '17 18:02 dail8859