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 1 commit into from Jan 15, 2018

Remove not used methods #4973

merged 1 commit into from Jan 15, 2018


Copy link

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

No description provided.

@kares kares added this to the JRuby milestone Jan 15, 2018
@kares kares merged commit d42f53e into jruby:master Jan 15, 2018
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
@yui-knk yui-knk deleted the remove_setMethods branch Jan 15, 2018
kares added a commit to kares/jruby that referenced this issue 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
  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
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants