.rbind.xts created unnecessary env variable?

joshuaulrich / xts

Extensible time series class that provides uniform handling of many R time series classes by extending zoo.

http://joshuaulrich.github.io/xts/

GNU General Public License v2.0

220 stars 71 forks source link

Closed harvey131 closed 4 years ago

harvey131 commented 4 years ago

I do not understand why .rbind.xts needs to create a new environment on line 99 of bind.R

"env <- new.env()"

Is this a piece of legacy code that can be removed?

joshuaulrich commented 4 years ago

Hi Harvey, thanks for the report! Your question is correct. That line appears in the very first commit of .rbind.xts(), but env is never used.