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

RuntimeError with compiled multi-byte variable #4660

Closed
jakago opened this issue Jun 9, 2017 · 3 comments
Closed

RuntimeError with compiled multi-byte variable #4660

jakago opened this issue Jun 9, 2017 · 3 comments

Comments

@jakago
Copy link

@jakago jakago commented Jun 9, 2017

multibytevar.rb

# coding: utf-8

# 'α'.bytes => [206, 177]
p α = 1

run with jruby 9.1.9.0

$ jruby multibytevar.rb
1

compile and run with jruby 9.1.9.0

$ jrubyc multibytevar.rb
$ jruby multibytevar.class
RuntimeError: BUG: interpreter fell through to end unexpectedly
  <main> at multibytevar.class:1
@headius
Copy link
Member

@headius headius commented Jun 9, 2017

Another possible persistence issue, maybe? @enebo

@enebo
Copy link
Member

@enebo enebo commented Jun 12, 2017

The fix for #4662 I thought would affect this but what it reads back is not being seen as a valid variable and this ends up violating some logic when the interpreter first interacts with it.

I marked this against 9.2 since we have 2 side branches changing our internals to be based on either (ByteList or RubySymbol - an ongoing discussion). What we have to date is pushing a ByteList through the parser. In the case of StaticScope we have a ByteList now so it will trivially fix this (although we may change this to a RubySymbol before all is said and done).

@headius
Copy link
Member

@headius headius commented May 16, 2018

Fixed in 9.2 by @enebo symbol work.

@headius headius closed this May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.