WIP: Add Redcarpet::Markdown#initialize_copy #198

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@yaotti
yaotti commented Feb 9, 2013

I tried to implement Redcarpet::markdown#initialize to make
markdown.dup and markdown.clone work, but I couldn't.
(markdown = Redcarpet::Markdown.new(Redcarpet::Render::HTML))

Somehow Data_Get_Struct(self, struct sd_markdown, sdm_self);
at ext/redcarpet/rc_markdown.c:143 assign null into sdm_self
when markdown.dup is called.
markdown.send(:initialize_copy, markdown2) works though.

Could you give advices to me?

@mattr-
Collaborator
mattr- commented Apr 4, 2013

Could you explain why you need multiple markdown rendering objects?

@yaotti
yaotti commented Sep 19, 2013

I thought I needed multiple md renderer objects when I created the PR but it was my mistake, so I'll close this PR :octocat:

@yaotti yaotti closed this Sep 19, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment