Wire up parse_comments

Author: headius

java/wasm-full/src/main/java/org/ruby_lang/prism/wasm/full/Prism.java

@@ -88,6 +88,10 @@ public boolean parseSuccess(Source source, Options options) {
         return exports.pmSerializeParseSuccessP(source.pointer, source.length, options.pointer) != 0;
     }
 
+    public void parseComments(Buffer buffer, Source source, Options options) {
+        exports.pmSerializeParseComments(buffer.pointer, source.pointer, source.length, options.pointer);
+    }
+
     public class Buffer implements AutoCloseable {
         final int pointer;
 

lib/prism/ffi.rb

@@ -105,7 +105,12 @@ def parse_comments(code, **options)
     # to use mmap when it is available.
     def parse_file_comments(filepath, **options)
       options[:filepath] = filepath
-      FFICommon.with_file(filepath) { |string| FFICommon.parse_comments(string, string.read, options) }
+      FFICommon.with_file(filepath) do |file|
+        code = file.read
+        FFICommon.with_string(code) do |string|
+          FFICommon.parse_comments(string, code, options)
+        end
+      end
     end
 
     # Mirror the Prism.parse_lex API by using the serialization API.

lib/prism/ffi/common.rb

@@ -35,6 +35,13 @@ def parse_lex(string, code, options) # :nodoc:
       end
     end
 
+    def parse_comments(string, code, options) # :nodoc:
+      with_buffer do |buffer|
+        parse_comments_only(buffer, string, options)
+        Serialize.load_parse_comments(code, buffer.read, options.fetch(:freeze, false))
+      end
+    end
+
     # Return the value that should be dumped for the command_line option.
     def dump_options_command_line(options)
       command_line = options.fetch(:command_line, "")
@@ -217,7 +224,7 @@ def parse_stream(buffer, callback, eof_callback, options, source) # :nodoc:
       raise NotImplementedError
     end
 
-    def parse_comments(string, code, options) # :nodoc:
+    def parse_comments_only(string, code, options) # :nodoc:
       raise NotImplementedError
     end
 

lib/prism/ffi/native_ffi.rb

@@ -287,11 +287,8 @@ def parse_stream(buffer, callback, eof_callback, options, source) # :nodoc:
       end
     end
 
-    def parse_comments(string, code, options) # :nodoc:
-      with_buffer do |buffer|
-        LibRubyParser.pm_serialize_parse_comments(buffer.pointer, string.pointer, string.length, dump_options(options))
-        Serialize.load_parse_comments(code, buffer.read, options.fetch(:freeze, false))
-      end
+    def parse_comments_only(buffer, string, options) # :nodoc:
+      LibRubyParser.pm_serialize_parse_comments(buffer.pointer, string.pointer, string.length, dump_options(options))
     end
 
     def parse_lex_only(buffer, string, options) # :nodoc:

lib/prism/ffi/wasm_ffi.rb

@@ -75,8 +75,8 @@ def parse_stream(buffer, callback, eof_callback, options, source) # :nodoc:
       raise NotImplementedError
     end
 
-    def parse_comments(string, code, options) # :nodoc:
-      raise NotImplementedError
+    def parse_comments_only(buffer, string, options) # :nodoc:
+      PRISM.parse_comments(buffer, string, PRISM.new_options(dump_options(options).to_java_bytes))
     end
 
     def parse_lex_only(buffer, string, options) # :nodoc: