benjjneb / dada2

Accurate sample inference from amplicon data with single nucleotide resolution
http://benjjneb.github.io/dada2/
GNU Lesser General Public License v3.0
469 stars 142 forks source link

Warning when installing version #101

Closed spholmes closed 8 years ago

spholmes commented 8 years ago

Installation from github of dada2 version 1.1.4 on Mac with R version 3.3.1 Here are the two messages:

In file included from RcppExports.cpp:4: In file included from ./../inst/include/dada2.h:7: ./../inst/include/dada2_RcppExports.h:14:14: warning: unused function 'validateSignature' [-Wunused-function] void validateSignature(const char* sig) { ^ 1 warning generated.

nwalign_vectorized.cpp:79:43: warning: unused variable 'ptr_index' [-Wunused-variable] int16_t *ptr_left, *ptr_diag, *ptr_up, *ptr_index, *ptr_d, *ptr_p; ^ 1 warning generated.

benjjneb commented 8 years ago

The devel branch is "active" so it will have a few of these compiler warnings for stuff such as a variable name we're no longer using. I clean these up for releases, but don't sweat them too much the rest of the time.

As long as it compiles, these sorts of compiler warnings can generally be ignored.

spholmes commented 8 years ago

Thanks Ben sorry for bothering you, just checking I have everything I need for WH, Susan

On Wed, Aug 3, 2016 at 10:44 AM, benjjneb notifications@github.com wrote:

The devel branch is "active" so it will have a few of these compiler warnings for stuff such as a variable name we're no longer using. I clean these up for releases, but don't sweat them too much the rest of the time.

As long as it compiles, these sorts of compiler warnings can generally be ignored.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/benjjneb/dada2/issues/101#issuecomment-237304397, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJcvSw83b_f9g6J4U9FHoIcizHEhjogks5qcNN3gaJpZM4JbxOp .

Susan Holmes Professor, Statistics and BioX John Henry Samter Fellow in Undergraduate Education Sequoia Hall, 390 Serra Mall Stanford, CA 94305 http://www-stat.stanford.edu/~susan/