Skip to content

Deprecate RubyHash constructors for 10.1 abstraction#9474

Merged
headius merged 1 commit into
jruby:jruby-10.0from
headius:deprecated_hash_constructors
Jun 4, 2026
Merged

Deprecate RubyHash constructors for 10.1 abstraction#9474
headius merged 1 commit into
jruby:jruby-10.0from
headius:deprecated_hash_constructors

Conversation

@headius

@headius headius commented Jun 4, 2026

Copy link
Copy Markdown
Member

No description provided.

@headius headius added this to the JRuby 10.0.6.0 milestone Jun 4, 2026
@headius
headius changed the base branch from master to jruby-10.0 June 4, 2026 12:33
@headius
headius force-pushed the deprecated_hash_constructors branch 3 times, most recently from 90ee9be to 624fddb Compare June 4, 2026 18:47
@headius
headius force-pushed the deprecated_hash_constructors branch from 624fddb to 93927e3 Compare June 4, 2026 20:09
@headius
headius merged commit 8d62308 into jruby:jruby-10.0 Jun 4, 2026
125 of 126 checks passed
@headius
headius deleted the deprecated_hash_constructors branch June 4, 2026 20:19
headius added a commit that referenced this pull request Jun 11, 2026
Similar to #9489 and #9474, deprecate all
public RubyArray constructors to support making it abstract in
10.1.

See #9457
headius added a commit to headius/jruby that referenced this pull request Jun 13, 2026
This constructor was restored by deprecation patches from 10.0 in
jruby#9474 but the merge of that to 10.1 left it as a normal
non-delegated constructor. This patch makes the following changes:

* Make this constructor delegate, like the other deprecated
  RubyHash constructors.
* Remove the protected constructor from RubyHashLinkedBuckets, so
  that it lives only in RubyHash and we can be sure all callers
  need delegation.

The constructor should move back into RubyHashLinkedBuckets as a
protected internal constructor once we can safely remove the
constructors from RubyHash (10.2 or later).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant