Remove old vec_is_vector() callable (#2104)

* Remove old `vec_is_vector()` callable

It has been a few years since we deprecated it in favor of `obj_is_vector()`. Only dplyr uses the `obj_is_vector()` callable.

No one using `vec_is_vector()` now:
https://github.com/search?q=org%3Acran+vec_is_vector+language%3Ac&type=code
https://github.com/search?q=org%3Acran+vec_is_vector+language%3AC%2B%2B+&type=code

* NEWS

Author: DavisVaughan

NEWS.md

@@ -1,5 +1,7 @@
 # vctrs (development version)
 
+* The deprecated C callable for `vec_is_vector()` has been removed.
+
 * Fixed the C level signature for the `exp_short_init_compact_seq()` callable.
 
 * Experimental "partial" type support has been removed. This idea never panned out and was not widely used. The following functions have been removed (#2101):

inst/include/vctrs.c

@@ -5,13 +5,8 @@ bool (*obj_is_vector)(SEXP) = NULL;
 R_len_t (*short_vec_size)(SEXP) = NULL;
 SEXP (*short_vec_recycle)(SEXP, R_len_t) = NULL;
 
-// Deprecated
-bool (*vec_is_vector)(SEXP) = NULL;
-
 void vctrs_init_api(void) {
   obj_is_vector = (bool (*)(SEXP)) R_GetCCallable("vctrs", "obj_is_vector");
   short_vec_size = (R_len_t (*)(SEXP)) R_GetCCallable("vctrs", "short_vec_size");
   short_vec_recycle = (SEXP (*)(SEXP, R_len_t)) R_GetCCallable("vctrs", "short_vec_recycle");
-
-  vec_is_vector = (bool (*)(SEXP)) R_GetCCallable("vctrs", "vec_is_vector");
 }

inst/include/vctrs.h

@@ -10,11 +10,6 @@ extern bool (*obj_is_vector)(SEXP);
 extern R_len_t (*short_vec_size)(SEXP);
 extern SEXP (*short_vec_recycle)(SEXP, R_len_t);
 
-// Deprecated in favor of `obj_is_vector()`
-// version: 0.5.3
-// date: 2023-02-15
-extern bool (*vec_is_vector)(SEXP);
-
 void vctrs_init_api(void);
 
 #endif

src/init.c

@@ -413,11 +413,6 @@ export void R_init_vctrs(DllInfo *dll)
     R_RegisterCCallable("vctrs", "short_vec_size",     (DL_FUNC) &short_vec_size);
     R_RegisterCCallable("vctrs", "short_vec_recycle",  (DL_FUNC) &short_vec_recycle);
 
-    // Deprecated
-    // In the public header
-    // See `inst/include/vctrs.h` for details
-    R_RegisterCCallable("vctrs", "vec_is_vector", (DL_FUNC) &maturing_obj_is_vector);
-
     // Experimental
     // Exported but not available in the public header
     R_RegisterCCallable("vctrs", "exp_vec_cast",                (DL_FUNC) &exp_vec_cast);