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

[Mark de Wever via cfe-dev]

Hi Dmitri,

On Sun, Aug 25, 2019 at 06:58:58PM +0200, Dmitri Gribenko wrote:
> > 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.

Then I'll look at this approach. Thanks for the feedback.

Cheers,
Mark