Feature #21287
closed
Remove SortedSet autoload and set/sorted_set
Description
When implementing core Set, I moved the SortedSet autoload from lib/set.rb to prelude.rb. However, since this autoload only works if the sorted_set gem is installed, I don't think it's a good idea to have in core. I would like to remove it. I've submitted a pull request that removes it: https://github.com/ruby/ruby/pull/13188 .
I'm not sure whether people would prefer a deprecation warning before removal, to alert users who have the sorted_set gem installed and are relying on the autoload. Previously, the SortedSet autoload was only setup if you had already referenced Set to force loading lib/set.rb. So Set; SortedSet worked, but SortedSet; Set did not. If people would prefer a deprecation warning, I think it's fine to deprecate in 3.5 and remove in 3.6, and I can submit a separate pull request for that approach.
Updated by mame (Yusuke Endoh) 21 days ago
Discussed at the dev meeting, and @matz (Yukihiro Matsumoto) said "go ahead".
Updated by jeremyevans (Jeremy Evans) 10 days ago
- Status changed from Open to Closed
Applied in changeset git|c52f4eea564058a8a9865ccc8b2aa6de0c04d156.
Remove SortedSet autoload and set/sorted_set
Implements [Feature #21287]