merliseclyde / BAS

BAS R package https://merliseclyde.github.io/BAS/
https://merliseclyde.github.io/BAS/
GNU General Public License v3.0
41 stars 16 forks source link

drop use of SETLENGTH #82

Open merliseclyde opened 4 days ago

merliseclyde commented 4 days ago

As SETLENGTH is not part of the public API, all calls of SETLENGTH need to be replaced in advance of strict enforcement

merliseclyde commented 2 days ago

removed all occurrences of SETLENGTH and updated code appropriately to truncate over-allocated vectors per https://github.com/r-lib/fs/issues/450

merliseclyde commented 2 days ago

Reopening due to imbalance problems flagged when running tests outside of R CMD check. Code has reverted back to using SETLENGTH to avoid the stack imbalance