Avoid readability-avoid-const-params-in-decls? about rcpp

[MichaelChirico]

:)

filed on the off chance this fits easily into something that routine's already doing. it's definitely not worth investing much new effort into.

[eddelbuettel]

I am a little lost now. What do you (realistically) expect us to do there? Reimplement a llvm parser? Methinks the tool you want to use exists, the repo you want apply it too exists (and you reference it three times). But you file a ticket here?

[MichaelChirico]

Rcpp is responsible for creating RcppExports.cpp, right? And in doing so it's copying the definition over to a declaration. That's where Rcpp comes in.

[eddelbuettel]

Sure. Mechanically true. Rcpp also does not write the initial declaration.

Anyway, your itch to scratch so I look forward to any PRs from you 😉. File is https://github.com/RcppCore/Rcpp/blob/master/src/attributes.cpp and a mere 3700 lines.

[eddelbuettel]

"It's hard." The src/attributes.cpp code is good and clever and not too ambitious. C++ is a complicated languages. Sometimes doing less is better.