cargo +nightly check --tests

extendr-api/src/ownership.rs

@@ -280,7 +280,6 @@ impl Ownership {
 mod test {
     use super::*;
     use crate::*;
-    use libR_sys::{Rf_ScalarInteger, Rf_protect, Rf_unprotect};
 
     #[test]
     fn basic_test() {

extendr-api/src/robj/into_robj.rs

@@ -1,6 +1,5 @@
 use super::*;
 use crate::scalar::Scalar;
-use crate::single_threaded;
 
 mod repeat_into_robj;
 

extendr-api/src/robj/mod.rs

@@ -17,7 +17,6 @@ use crate::*;
 
 use crate::scalar::{Rbool, Rfloat, Rint};
 use std::collections::HashMap;
-use std::iter::IntoIterator;
 use std::ops::{Range, RangeInclusive};
 
 // deprecated

extendr-api/src/scalar/rbool.rs

@@ -1,7 +1,6 @@
 use crate::scalar::macros::*;
 use crate::scalar::Scalar;
 use crate::*;
-use std::convert::TryFrom;
 
 /// `Rbool` is a wrapper for `i32` in the context of an R's logical vector.
 ///

extendr-api/src/scalar/rfloat.rs

@@ -2,7 +2,6 @@ use crate::prelude::{Rint, Scalar};
 use crate::scalar::macros::*;
 use crate::*;
 use std::cmp::Ordering::*;
-use std::convert::TryFrom;
 use std::ops::{Add, Div, Mul, Neg, Sub};
 use std::ops::{AddAssign, DivAssign, MulAssign, SubAssign};
 

extendr-api/src/scalar/rint.rs

@@ -2,7 +2,6 @@ use crate::scalar::macros::*;
 use crate::scalar::Scalar;
 use crate::*;
 use std::cmp::Ordering::*;
-use std::convert::TryFrom;
 use std::ops::{Add, Div, Mul, Neg, Not, Sub};
 use std::ops::{AddAssign, DivAssign, MulAssign, SubAssign};
 

extendr-api/src/wrapper/altrep.rs

@@ -1,4 +1,4 @@
-use prelude::{Rbool, Rcplx, Rfloat, Rint, Scalar};
+use prelude::{Rcplx, Rfloat, Rint, Scalar};
 
 use super::*;
 

extendr-api/src/wrapper/complexes.rs

@@ -1,6 +1,5 @@
 use super::scalar::{c64, Rcplx};
 use super::*;
-use std::iter::FromIterator;
 
 /// An obscure `NA`-aware wrapper for R's complex vectors.
 /// Can be used to iterate over vectors obtained from R

extendr-api/src/wrapper/doubles.rs

@@ -1,6 +1,5 @@
 use super::scalar::{Rfloat, Scalar};
 use super::*;
-use std::iter::FromIterator;
 
 /// An obscure `NA`-aware wrapper for R's double vectors.
 /// Can be used to iterate over vectors obtained from R

extendr-api/src/wrapper/integers.rs

@@ -1,6 +1,5 @@
 use super::scalar::{Rint, Scalar};
 use super::*;
-use std::iter::FromIterator;
 
 /// An obscure `NA`-aware wrapper for R's integer vectors.
 /// Can be used to iterate over vectors obtained from R

extendr-api/src/wrapper/list.rs

@@ -1,6 +1,3 @@
-use crate::robj::Attributes;
-use std::iter::FromIterator;
-
 use super::*;
 
 #[derive(PartialEq, Clone)]

extendr-api/src/wrapper/logicals.rs

@@ -1,6 +1,5 @@
-use super::scalar::{Rbool, Scalar};
+use super::scalar::Scalar;
 use super::*;
-use std::iter::FromIterator;
 
 /// An obscure `NA`-aware wrapper for R's logical vectors.
 /// Can be used to iterate over vectors obtained from R

extendr-api/src/wrapper/matrix.rs

@@ -1,7 +1,6 @@
 //! Wrappers for matrices with deferred arithmetic.
 
 use super::*;
-use crate::robj::GetSexp;
 use crate::scalar::Scalar;
 use std::ops::{Index, IndexMut};
 

extendr-api/src/wrapper/mod.rs

@@ -1,7 +1,7 @@
 //! Wrappers are lightweight proxies for references to R datatypes.
 //! They do not contain an Robj (see array.rs for an example of this).
 
-use crate::robj::{GetSexp, Rinternals};
+use crate::robj::Rinternals;
 use crate::*;
 use libR_sys::*;
 

extendr-api/src/wrapper/strings.rs

@@ -1,6 +1,3 @@
-use std::convert::From;
-use std::iter::FromIterator;
-
 use super::*;
 
 #[derive(PartialEq, Clone)]

extendr-macros/src/wrappers.rs

@@ -160,7 +160,7 @@ pub fn make_function_wrappers(
             Ok(_self_robj)
         )
     };
-
+    // TODO: the unsafe in here is unnecessary
     wrappers.push(parse_quote!(
         #[no_mangle]
         #[allow(non_snake_case, clippy::not_unsafe_ptr_arg_deref)]