pointsinside: defer check for Rvcg until committed to use it

* closes #454
natverse · Jan 22, 2021 · efb5bf4 · efb5bf4
1 parent 42ca544
commit efb5bf4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/R/hxsurf.R b/R/hxsurf.R
@@ -709,10 +709,7 @@ pointsinside<-function(x, surf, ...) UseMethod('pointsinside')
 #' @rdname pointsinside
 pointsinside.default<-function(x, surf, ..., rval=c('logical','distance', 
                                                     'mesh3d', 'consistent_logical')) {
-  if(!requireNamespace('Rvcg', quietly = TRUE))
-    stop("Please install suggested library Rvcg to use pointsinside")
   rval=match.arg(rval)
-
   if(rval=='logical') {
     # use optimised contains_points approach
     return(contains_points(surf, x, ...))
@@ -723,6 +720,9 @@ pointsinside.default<-function(x, surf, ..., rval=c('logical','distance',
            "with boundingbox objects!")
   }
 
+  if(!requireNamespace('Rvcg', quietly = TRUE))
+    stop("Please install suggested library Rvcg to use pointsinside")
+
   if(!inherits(surf,'mesh3d')) {
     surf=as.mesh3d(surf, ...)
   }