remove Rcpp::export tags on lightweight function defs

[johnlaing]

Addresses #408

[eddelbuettel]

Had not thought of that approach either. If that works locally thumbs up!

[eddelbuettel]

Looks good.

I would vote for squash merge and rebase into #404 and can help if needed.

[johnlaing]

It works:

jlaing@penguin:~/dev/Rblpapi$ Rscript -e 'Rcpp::compileAttributes()'
jlaing@penguin:~/dev/Rblpapi$ git status
On branch bugfix/duplicate_exports
Your branch is up to date with 'origin/bugfix/duplicate_exports'.

nothing to commit, working tree clean

Doing the same on master would produce different exports files.

Happy to let you execute squash/merge/rebase/whatever

[eddelbuettel]

Yes my 'Looks good' was based on quick checkout of the branch, installing, re-compiling attributes and re-installing.

I'll add a quick ChangeLog for you and then proceed. Ok?

[eddelbuettel]

Ok -- #404 is now on top of the updated master and the same two commits ahead with the actual change by @klin333 as well as my conditional build (including your removal of the export tag in the one altered file / half).

Should be good. And I had forgotten that I had described what I may try next already in #407