Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove not used methods #4973

Merged
merged 1 commit into from
Jan 15, 2018
Merged

Remove not used methods #4973

merged 1 commit into from
Jan 15, 2018

Conversation

yui-knk
Copy link
Contributor

@yui-knk yui-knk commented Jan 14, 2018

No description provided.

@kares kares added this to the JRuby 9.2.0.0 milestone Jan 15, 2018
@kares kares merged commit d42f53e into jruby:master Jan 15, 2018
@yui-knk yui-knk deleted the remove_setMethods branch January 15, 2018 06:06
kares added a commit to kares/jruby that referenced this pull request Jan 19, 2018
* master: (270 commits)
  Mild refactoring of invokers code (came from bytelist_love work).  As I need to make changes to generated code I am trying to comment/document some portions and reduce some duplication when possible.
  resolve regression with Set impl - SortedSet did not clone/dup properly
  be explicit about non-happening exception + avoid dynamic ** call
  cleanup Rational - use RubyInteger#idiv so we're no hitting call-sites
  introduce long overloads for common (integer) ops + start using them
  raise ArgumentError
  use encoding none for n/N option
  fix IOOBE in test_regexp quote
  Fix documents
  Revise COPYRIGHT YEAR
  use num2int in StringIO#ungetc
  Fix a typo
  Remove not used methods (jruby#4973)
  Inline the regexp validation inside ParserSupport
  Simplify the regexp validation logic
  Add support for validating Regexp in Ripper
  Make sure to clear $! when rescuing RaiseException
  Align validation code with MRI
  Move Regexp checking from ParserSupport to Lexer
  Extract parseRegexpFlags to Lexer
  ...
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.

None yet

2 participants