Skip to content

Make internal BigMath method a private method#432

Merged
tompng merged 1 commit intoruby:masterfrom
tompng:bigmath_private_method
Sep 15, 2025
Merged

Make internal BigMath method a private method#432
tompng merged 1 commit intoruby:masterfrom
tompng:bigmath_private_method

Conversation

@tompng
Copy link
Copy Markdown
Member

@tompng tompng commented Sep 15, 2025

private def has no effect in BigMath module. Change it to private_class_method

module BigMath
  module_function

  # Private BigMath#foo and public BigMath.foo
  def foo; end 

  # Same as above. private has no effect
  private def foo; end

  # Private BigMath#foo and private BigMath.foo
  private_class_method def foo; end
end

@tompng tompng force-pushed the bigmath_private_method branch from 724a3fa to c49f0de Compare September 15, 2025 20:05
@tompng tompng merged commit d93b542 into ruby:master Sep 15, 2025
81 checks passed
@tompng tompng deleted the bigmath_private_method branch September 15, 2025 20:12
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