_do_subset_xts() is used everywhere except for src/na.c. Test that using it in src/na.c doesn't break anything internally (i.e. na.omit.xts() and na.exclude.xts()).
We also need to ensure it doesn't change the output of na_omit_xts(), which is callable externally via R_RegisterCCallable() and xtsAPI.h. That said, neither of the *do_subset_xts() functions are directly callable externally (i.e. via R_RegisterCCallable() or via xtsAPI.h), so we can swap the functionality of one for the other.
_do_subset_xts()is used everywhere except forsrc/na.c. Test that using it insrc/na.cdoesn't break anything internally (i.e.na.omit.xts()andna.exclude.xts()).We also need to ensure it doesn't change the output of
na_omit_xts(), which is callable externally viaR_RegisterCCallable()andxtsAPI.h. That said, neither of the*do_subset_xts()functions are directly callable externally (i.e. viaR_RegisterCCallable()or viaxtsAPI.h), so we can swap the functionality of one for the other.