[Rcpp-devel] Why not export CxxFlags and LdFlags?

[Romain François]

On 01/05/2010 11:14 AM, Henning Redestig wrote:
> 2010/1/5 Romain François<francoisromain at free.fr>:
>> On 01/05/2010 06:20 AM, Henning Redestig wrote:
>> I guess the main reason is that these functions don't need the package to be
>> loaded to run.
>>
>> If you submit a patch (including documentation) I'll review it, unless Dirk
>> has stronger reasons not to export them.
>
> Okidoki, here it comes.

Please mind the "including documentation" part of my email.

Your patch will produce some warnings at R CMD check because there will 
be undocumented functions. You need to write an Rd file, or append to 
one and make sure R CMD check does not complain.

Also; it would be more useful if you generate the patch using diff.

svn diff > patch.diff

Romain

-- 
Romain Francois
Professional R Enthusiast
+33(0) 6 28 91 30 30
http://romainfrancois.blog.free.fr
|- http://tr.im/IW9B : C++ exceptions at the R level
|- http://tr.im/IlMh : CPP package: exposing C++ objects
`- http://tr.im/HlX9 : new package : bibtex