Skip to content

fix mac transcoder #57

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.

Sign up for GitHub

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

Jump to bottom
Merged
enebo merged 1 commit into from
Mar 22, 2022
Merged

fix mac transcoder #57

enebo merged 1 commit into from
Mar 22, 2022

Conversation

@ahorek
Copy link
Contributor

@ahorek ahorek commented Mar 16, 2022 •
edited
Loading

the value was incorrectly hardcoded in the generate.rb script, this restores the right value

fixes

 TestCSVEncodings::DifferentOFS#test_regular_expression_escaping:
Java::JavaLang::AssertionError: UTF8-MAC state not large enough
    org.jcodings.transcode.TranscodeFunctions.bufClear(TranscodeFunctions.java:1035)
    org.jcodings.transcode.TranscodeFunctions.fromUtf8MacInit(TranscodeFunctions.java:1017)
    org.jcodings.transcode.specific.From_UTF8_MAC_Transcoder.stateInit(From_UTF8_MAC_Transcoder.java:35)
    org.jcodings.transcode.Transcoding.<init>(Transcoding.java:38)

refs jruby/jruby#7142

@enebo enebo merged commit df5d675 into jruby:master Mar 22, 2022
@headius headius added this to the 1.0.56 milestone Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.0.56

Development

Successfully merging this pull request may close these issues.

3 participants

@ahorek @headius @enebo