-
-
Notifications
You must be signed in to change notification settings - Fork 329
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
Implement String#getbyte #2141
Implement String#getbyte #2141
Conversation
19b73eb
to
2a52b3c
Compare
a143f65
to
623c869
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! One last thing: can you please also remove the definition of getbyte
on the base encoding class?
opal/opal/corelib/string/encoding.rb
Lines 59 to 61 in bdbd1bf
def getbyte(*) | |
raise NotImplementedError | |
end |
@elia Done! I've also added a changelog 😉 |
59ed8e5
to
2c5f2d4
Compare
Also make the implementation of Stirng#bytes rely on #getbyte instead of forwarding to the encoding class which never implemented it.
2c5f2d4
to
a61eac1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this @Mogztter!
(I squashed all the back and forth commits into a single one and fixed the placement of the new changelog entry)
Based on #2140