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

Commonmarker: Member dfree not found #2732

Closed
Th3-M4jor opened this issue Sep 21, 2022 · 2 comments · Fixed by #2841
Closed

Commonmarker: Member dfree not found #2732

Th3-M4jor opened this issue Sep 21, 2022 · 2 comments · Fixed by #2841

Comments

@Th3-M4jor
Copy link

Th3-M4jor commented Sep 21, 2022

This Error does not occur when using MRI 3.0.3

home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/ext/commonmarker/commonmarker.c:535:in `rb_node_insert_before': Member 'dfree' not found. (Polyglot::ForeignException)
        from /home/spartan364/.rvm/rubies/truffleruby-22.2.0/lib/truffle/truffle/cext_ruby.rb:41:in `<unknown>'
        from /home/spartan364/.rvm/rubies/truffleruby-22.2.0/lib/truffle/truffle/cext_ruby.rb:41:in `CommonMarker::Node#insert_before'
        from ./markdown_test.rb:12:in `block in <main>'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:16:in `CommonMarker::Node#walk'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:18:in `block in CommonMarker::Node#walk'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:72:in `CommonMarker::Node#each'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:17:in `CommonMarker::Node#walk'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:18:in `block in CommonMarker::Node#walk'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:72:in `CommonMarker::Node#each'
        from /home/spartan364/.rvm/gems/truffleruby-22.2.0/gems/commonmarker-0.23.5/lib/commonmarker/node.rb:17:in `CommonMarker::Node#walk'
        from ./markdown_test.rb:7:in `<main>'

Minimal reproduction:

require "commonmarker"

doc = CommonMarker.render_doc("# this is a test string")

doc.walk do |node|
  if node.type == :text
    node.insert_before(node)
  end
end

Note: This is a duplicate of #2165, however it is a 2 year old issue. As such, I am opening a separate issue rather than asking for an update there.

Update: Tested with truffleruby 22.3.0-dev-76cfbe8d, same result.

@eregon eregon added the cexts label Sep 30, 2022
@eregon
Copy link
Member

eregon commented Sep 30, 2022

Thanks for the report, we'll take a look.
I don't want duplicate issues, so in this case let's close the older one since it has less details.

@eregon
Copy link
Member

eregon commented Jan 26, 2023

Let's track this in #2830

@eregon eregon closed this as completed Jan 26, 2023
graalvmbot pushed a commit that referenced this issue Jan 26, 2023
@eregon eregon closed this as not planned Won't fix, can't repro, duplicate, stale Jan 26, 2023
@eregon eregon self-assigned this Jan 26, 2023
@eregon eregon added this to the 23.0.0 Release milestone Jan 26, 2023
nirvdrum pushed a commit to Shopify/truffleruby that referenced this issue Jan 30, 2023
sophie-kaleba pushed a commit to sophie-kaleba/truffleruby that referenced this issue Feb 16, 2023
sophie-kaleba pushed a commit to sophie-kaleba/truffleruby that referenced this issue Feb 16, 2023
john-heinnickel pushed a commit to thermofisher-jch/truffleruby that referenced this issue Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants