ehh, actually in all list structs capacity goes before len. added explicit tests for lists where len != capacity

Author: iliabylich

bytes.h

@@ -10,8 +10,8 @@
 typedef struct LIB_RUBY_PARSER_ByteList
 {
     char *ptr;
-    size_t len;
     size_t capacity;
+    size_t len;
 } LIB_RUBY_PARSER_ByteList;
 /*
     ByteList destructor.

comment.h

@@ -36,8 +36,8 @@ typedef struct LIB_RUBY_PARSER_Comment
 typedef struct LIB_RUBY_PARSER_CommentList
 {
     LIB_RUBY_PARSER_Comment *ptr;
-    size_t len;
     size_t capacity;
+    size_t len;
 } LIB_RUBY_PARSER_CommentList;
 
 /*

diagnostic.h

@@ -58,8 +58,8 @@ void LIB_RUBY_PARSER_drop_diagnostic(LIB_RUBY_PARSER_Diagnostic *diagnostic);
 typedef struct LIB_RUBY_PARSER_DiagnosticList
 {
     LIB_RUBY_PARSER_Diagnostic *ptr;
-    size_t len;
     size_t capacity;
+    size_t len;
 } LIB_RUBY_PARSER_DiagnosticList;
 /*
     DiagnosticList destructor.

magic_comment.h

@@ -31,8 +31,8 @@ typedef struct LIB_RUBY_PARSER_MagicComment
 typedef struct LIB_RUBY_PARSER_MagicCommentList
 {
     LIB_RUBY_PARSER_MagicComment *ptr;
-    size_t len;
     size_t capacity;
+    size_t len;
 } LIB_RUBY_PARSER_MagicCommentList;
 
 /*

ruby-parser-c/src/bytes.rs

@@ -7,7 +7,9 @@ blob_type!(BytesBlob, Bytes);
 #[cfg(feature = "tests")]
 #[no_mangle]
 pub extern "C" fn lib_ruby_parser__test__make_byte_list(i1: u8, i2: u8, i3: u8) -> ByteListBlob {
-    ByteListBlob::from(vec![i1, i2, i3])
+    let mut v = vec![i1, i2, i3];
+    v.reserve(10);
+    ByteListBlob::from(v)
 }
 
 #[no_mangle]

ruby-parser-c/src/comment.rs

@@ -34,7 +34,9 @@ pub extern "C" fn lib_ruby_parser__test__make_comment(location: Loc, kind: Comme
 #[cfg(feature = "tests")]
 #[no_mangle]
 pub extern "C" fn lib_ruby_parser__test__make_comment_list(comment: Comment) -> CommentListBlob {
-    CommentListBlob::from(vec![comment])
+    let mut v = vec![comment];
+    v.reserve(10);
+    CommentListBlob::from(v)
 }
 
 #[no_mangle]

ruby-parser-c/src/diagnostic.rs

@@ -32,7 +32,9 @@ pub extern "C" fn LIB_RUBY_PARSER_drop_diagnostic(diagnostic: *mut Diagnostic) {
 pub extern "C" fn lib_ruby_parser__test__make_diagnostic_list(
     diagnostic: DiagnosticBlob,
 ) -> DiagnosticListBlob {
-    DiagnosticListBlob::from(vec![Diagnostic::from(diagnostic)])
+    let mut v = vec![Diagnostic::from(diagnostic)];
+    v.reserve(10);
+    DiagnosticListBlob::from(v)
 }
 
 #[no_mangle]

ruby-parser-c/src/magic_comment.rs

@@ -52,7 +52,9 @@ pub extern "C" fn lib_ruby_parser__test__make_magic_comment(
 pub extern "C" fn lib_ruby_parser__test__make_magic_comment_list(
     magic_comment: MagicComment,
 ) -> MagicCommentListBlob {
-    MagicCommentListBlob::from(vec![magic_comment])
+    let mut v = vec![magic_comment];
+    v.reserve(10);
+    MagicCommentListBlob::from(v)
 }
 
 #[no_mangle]

ruby-parser-c/src/source_line.rs

@@ -22,7 +22,9 @@ pub extern "C" fn lib_ruby_parser__test__make_source_line(
 pub extern "C" fn lib_ruby_parser__test__make_source_line_list(
     source_line: SourceLine,
 ) -> SourceLineListBlob {
-    SourceLineListBlob::from(vec![source_line])
+    let mut v = vec![source_line];
+    v.reserve(10);
+    SourceLineListBlob::from(v)
 }
 
 #[no_mangle]

ruby-parser-c/src/string.rs

@@ -18,7 +18,9 @@ pub extern "C" fn LIB_RUBY_PARSER_new_string_from_cstr(ptr: *const i8) -> String
 #[cfg(feature = "tests")]
 #[no_mangle]
 pub extern "C" fn lib_ruby_parser__test__make_string_foo() -> StringBlob {
-    StringBlob::from(String::from("foo"))
+    let mut s = String::from("foo");
+    s.reserve(20);
+    StringBlob::from(s)
 }
 
 #[no_mangle]