uint -> unsigned int

All checks OK

Author: jlaria

cran-comments.md

@@ -1,43 +1,25 @@
 ## Test environments
 * local Debian GNU/Linux bullseye install, R 3.6.1
-* rhub (`"fedora-clang-devel"`)
+* rhub's `check_with_sanitizers`
+* win-builder all
 
 ## R CMD check results
 
 ### (local)
 
-checking installed package size ... NOTE
-    installed size is  5.1Mb
-    sub-directories of 1Mb or more:
-      libs   4.7Mb
+    checking installed package size ... NOTE
+        installed size is  5.1Mb
+        sub-directories of 1Mb or more:
+          libs   4.7Mb
 
 0 errors | 0 warnings | 1 note 
 
-This note only appears when compiling with g++ in Linux. I believe it is related to Rcpp modules and g++, because `Rcpp::Rcpp.package.skeleton(module=T)` produces the same note.
+### Debian Linux, R-devel, GCC ASAN/UBSAN
 
-### r-devel-linux-x86_64-fedora-clang
+0 errors | 0 warnings | 0 note 
 
-(BEFORE)
-ERROR
+### win-builder
 
+0 errors | 0 warnings | 1 note 
 
-    In file included from nExtJT.cpp:4:
-    In file included from /data/gannet/ripley/R/test-clang/RcppArmadillo/include/RcppArmadillo.h:31:
-    In file included from /data/gannet/ripley/R/test-clang/RcppArmadillo/include/RcppArmadilloForward.h:26:
-    In file included from /data/gannet/ripley/R/test-clang/Rcpp/include/RcppCommon.h:168:
-    In file included from /data/gannet/ripley/R/test-clang/Rcpp/include/Rcpp/as.h:25:
-    /data/gannet/ripley/R/test-clang/Rcpp/include/Rcpp/internal/Exporter.h:31:30: error: reference member 't' binds to a temporary object whose lifetime would be shorter than the lifetime of the constructed object
-                    Exporter( SEXP x ) : t(x){}
-                    
-
-This is fixed in the new version. Solution found [here](https://stackoverflow.com/questions/56004251/rcpp-module-error-with-a-constructor-taking-numericvector-as-parameter)
-
-Result on rhub **Fedora Linux, R-devel, clang, gfortran**
-
-* checking CRAN incoming feasibility ... NOTE
-Maintainer: ‘Juan C. Laria <juank.laria@gmail.com>’
-
-Days since last update: 1
-
-Possibly mis-spelled words in DESCRIPTION:
-  Culp (15:600)
+Days since last update: 2

src/losses.h

@@ -55,7 +55,7 @@ double R_logit(const arma::vec & beta,
 
     // There is a more efficient and stable computation
     double rlogit = 0;
-    for (uint i = 0; i < eta.n_elem; i++)
+    for (unsigned int i = 0; i < eta.n_elem; i++)
     {
         rlogit += eta[i] = log1pexp(eta[i]) - y[i] * eta[i];
     }
@@ -67,7 +67,7 @@ double R_logit2(const arma::vec & beta,
     arma::vec eta = X*beta;
     // There is a more efficient and stable computation
     double rlogit = 0;
-    for (uint i = 0; i < eta.n_elem; i++)
+    for (unsigned int i = 0; i < eta.n_elem; i++)
     {
         rlogit += eta[i] = log1pexp(eta[i]) - y * eta[i];
     }
@@ -90,7 +90,7 @@ arma::vec grad_R_logit(const arma::vec & beta,
 
     // More stable
     arma::vec tmp = X*beta;
-    for (uint i = 0; i < tmp.n_elem; i++)
+    for (unsigned int i = 0; i < tmp.n_elem; i++)
     {
         tmp[i] = inv1pexpm(tmp[i]) - y[i];
     }
@@ -101,7 +101,7 @@ arma::vec grad_R_logit2(const arma::vec & beta,
                        double y){
     // More stable
     arma::vec tmp = X*beta;
-    for (uint i = 0; i < tmp.n_elem; i++)
+    for (unsigned int i = 0; i < tmp.n_elem; i++)
     {
         tmp[i] = inv1pexpm(tmp[i]) - y;
     }

src/nExtJT.cpp

@@ -33,7 +33,7 @@
 // Soft-threshold operator
 arma::vec soft_thresh(const arma::vec & z, double l){
   arma::vec S = arma::vec(z.n_elem);
-  for(uint i=0; i<z.n_elem; i++){
+  for(unsigned int i=0; i<z.n_elem; i++){
     if(std::abs(z[i]) <= l){
       S(i) = 0;
     }else{