<html><body><div style="color:#000; background-color:#fff; font-family:Courier New, courier, monaco, monospace, sans-serif;font-size:10pt"><div><span>Thanks. Just to note the it's traits::</span><span style="font-family: courier, 'courier new', monospace; font-size: 14px; line-height: 19px; white-space: pre-wrap;">is_finite<STRSXP>, rather than is_na(), that needs to (always) return false</span></div><div><br></div> <div style="font-family: 'Courier New', courier, monaco, monospace, sans-serif; font-size: 10pt;"> <div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"> <div dir="ltr"> <hr size="1"> <font size="2" face="Arial"> <b><span style="font-weight:bold;">From:</span></b> Dirk Eddelbuettel <edd@debian.org><br> <b><span style="font-weight: bold;">To:</span></b> Dirk Eddelbuettel <edd@debian.org> <br><b><span style="font-weight: bold;">Cc:</span></b> Thomas Tse <tommy_228_228@yahoo.com.hk>;
"rcpp-devel@lists.r-forge.r-project.org" <rcpp-devel@lists.r-forge.r-project.org> <br> <b><span style="font-weight: bold;">Sent:</span></b> Thursday, October 3, 2013 7:46 AM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: [Rcpp-devel] inconsistent is_na() vs. is.na()<br> </font> </div> <div class="y_msg_container"><br><br>Thomas,<br><br>Thanks again -- I committed both suggested changes to is_na() [ to follow R's<br>behaviour in case of NA and NaN, and to return false for STRSXPs ].<br><br>Keep the fixes coming in :)<br><br>Dirk<br><br>PS Diff output would be preferred, either from the diff program (usuanlly<br>with flag -u) or from 'svn diff'. <br><br>-- <br>Dirk Eddelbuettel | <a ymailto="mailto:edd@debian.org" href="mailto:edd@debian.org">edd@debian.org</a> | <a href="http://dirk.eddelbuettel.com/" target="_blank">http://dirk.eddelbuettel.com</a><br><br><br></div> </div> </div> </div></body></html>