[cfe-dev] [-Wdocumentation][RFC] \param with multiple identifiers

[Dmitri Gribenko via cfe-dev]

Hi Mark,

Thank you for working on this bug!

On Sun, Aug 25, 2019 at 4:30 PM Mark de Wever <koraq at xs4all.nl> wrote:
> 1.
> I can change ParamIndex to a SmallVector and change the AST output to
>   [in] explicitly Param="x,y,z" ParamIndex=1 ParamIndex=2 ParamIndex=3
> I wonder whether that would make the users of the AST output happy.
> Suddenly there can be multiple ParamIndex 'fields'.

I would strongly prefer this option. With regards to APIs, AST or
otherwise, Clang's first priority is to keep code understandable and
maintainable in the long run, backwards compatibility is not a
priority. Therefore, we should choose the API that makes the most
sense, and then adjust the rest of the code as needed.


Dmitri

--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/