markly 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93a192bb4cf0e216365ffa17fc619ecc726cb7cbd9a3661ae3357ce5a8dbb1c4
-  data.tar.gz: ec39190db7ad173cdbbcddd73850fc07faa578d37892217cedd329efad060610
+  metadata.gz: 27b98de3f04fbba3681df9356148129c3adb335740846ba638a4fb2790029e9e
+  data.tar.gz: c6bdd14cff454a313050128e4b4f32e1b7db2306581d8fde7475ccd9021fac36
 SHA512:
-  metadata.gz: 5668b9796316e769f73a6d97c495116e3361b9cc0a12fa92c8ec35461954a63d7568e8975af4720d060605ccb2ecdcc96bf123da2074271216a2e55e3afa378d
-  data.tar.gz: 132e25acaad81e6953c4c25a206dc6e37ba0f2cff3176887fb81882817edd81fa6c2a8eb24cf274046635b30f1179832f2daeb00ec78e18f4afb537166e4aab7
+  metadata.gz: 69a8a8366b7b3eb19d08c1cedd839c5a0a3ec15d3c0e3be1f80e9adcd4a7849704a733249aebb86771a822a3e4b0dbb4a5643b208597be9316c7282f5b6d471a
+  data.tar.gz: 1540db6af546c59fff663ab5f056519fdf48523e56c35a28281e3d5c34f75b4c2d0e0176666af4453b87b16faefeb19880da511067468a34d648d97c15d1a2ff

data/ext/markly/markly.c

@@ -41,11 +41,13 @@ static VALUE sym_right;
 static VALUE sym_center;
 
 static void rb_Markly_Node_free(void *data) {
-	// It's important that the `free` function does not inspect the node data, as it may be part of a tree that was already freed.
-	cmark_node *node = (cmark_node*)data;
-	
-	if (cmark_node_parent(node) == NULL) {
-		cmark_node_free(node);
+	// If a parent of this node is already freed, `rb_Markly_Node_freed` will ensure all the nodes are nullified.
+	if (data) {
+		cmark_node *node = (cmark_node*)data;
+		
+		if (cmark_node_parent(node) == NULL) {
+			cmark_node_free(node);
+		}
 	}
 }
 
@@ -79,6 +81,12 @@ static const rb_data_type_t rb_Markly_Node_Type = {
 	.flags = RUBY_TYPED_FREE_IMMEDIATELY,
 };
 
+static void rb_Markly_Node_freed(cmark_mem *mem, void *user_data) {
+	VALUE self = (VALUE)user_data;
+	
+	RTYPEDDATA_DATA(self) = NULL;
+}
+
 static VALUE rb_Markly_Node_wrap(cmark_node *node) {
 	if (node == NULL)
 		return Qnil;
@@ -91,6 +99,7 @@ static VALUE rb_Markly_Node_wrap(cmark_node *node) {
 	
 	VALUE self = TypedData_Wrap_Struct(rb_Markly_Node, &rb_Markly_Node_Type, node);
 	cmark_node_set_user_data(node, (void *)self);
+	cmark_node_set_user_data_free_func(node, rb_Markly_Node_freed);
 	
 	return self;
 }
@@ -121,16 +130,6 @@ static const rb_data_type_t rb_Markly_Parser_Type = {
 	.flags = RUBY_TYPED_FREE_IMMEDIATELY,
 };
 
-static VALUE rb_Markly_Parser_root(VALUE self) {
-	cmark_parser *parser = NULL;
-	
-	TypedData_Get_Struct(self, cmark_parser, &rb_Markly_Parser_Type, parser);
-	
-	assert(parser != NULL);
-	
-	return rb_Markly_Node_wrap(parser->root);
-}
-
 static VALUE rb_Markly_Parser_alloc(VALUE self) {
 	return TypedData_Wrap_Struct(self, &rb_Markly_Parser_Type, NULL);
 }
@@ -1193,7 +1192,6 @@ __attribute__((visibility("default"))) void Init_markly() {
 	rb_define_method(rb_Markly_Parser, "initialize", rb_Markly_Parser_initialize, 1);
 	rb_define_method(rb_Markly_Parser, "enable", rb_Markly_Parser_enable, 1);
 	rb_define_method(rb_Markly_Parser, "parse", rb_Markly_Parser_parse, 1);
-	rb_define_method(rb_Markly_Parser, "root", rb_Markly_Parser_root, 0);
 	
   rb_Markly_Node = rb_define_class_under(rb_Markly, "Node", rb_cObject);
   rb_define_singleton_method(rb_Markly_Node, "new", rb_node_new, 1);

data/lib/markly/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Markly
-	VERSION = '0.2.0'
+	VERSION = '0.2.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: markly
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Garen Torikian
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-06-13 00:00:00.000000000 Z
+date: 2020-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bake