Superfluous parentheses in declarations are handled incorrectly about cppinsights HOT 4 CLOSED

Hello @DivineWinds,

thanks for spotting this! A fix is on its way.

Andreas

from cppinsights.

Thanks for the fix @andreasfertig. I'm sorry for being a nuisance but unfortunately I discovered a problem with it. void (**p); and deeper multilevel pointers still exhibit the old behaviour. Also, using cv-qualifiers breaks even the one level pointer case, that is, void (*volatile p); becomes void (*pvolatile); (I'm only using volatile to skip the initalizer. const does the same thing). Combining the two results in the old behaviour as well: void (**volatile p); becomes void (**volatile) p;.

EDIT: I don't know if this is useful in any way but turning any of the above examples into array declarations seems to fix them, e.g. void (**volatile p[2]); remains unchanged. There are however pathological array declarations like void (*(*p)[3]); -> void (*p(*)[3]);. The last thing I noticed is that adding typedef to any of the examples, including the last one makes them behave correctly.

from cppinsights.

Hello @DivineWinds,

thanks for the testing. What is the use-case for all that? Is this production code?

Andreas

from cppinsights.

from cppinsights.