From 652e2d8f5d0e0210a4ef3cb541504433dbae57b3 Mon Sep 17 00:00:00 2001 From: Hiroshi SHIBATA Date: Thu, 1 Dec 2022 19:45:09 +0900 Subject: [PATCH] [ruby/syntax_suggest] No longer need PathnameFromMessage with SyntaxError#path. https://bugs.ruby-lang.org/issues/19138 Co-authored-by: Nobuyoshi Nakada https://github.com/ruby/syntax_suggest/commit/8e1e7b3298 --- lib/syntax_suggest/core_ext.rb | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/lib/syntax_suggest/core_ext.rb b/lib/syntax_suggest/core_ext.rb index 40f5fe1375..616a6ed983 100644 --- a/lib/syntax_suggest/core_ext.rb +++ b/lib/syntax_suggest/core_ext.rb @@ -25,15 +25,12 @@ if SyntaxError.method_defined?(:detailed_message) require "syntax_suggest/api" unless defined?(SyntaxSuggest::DEFAULT_VALUE) message = super - file = if highlight - SyntaxSuggest::PathnameFromMessage.new(super(highlight: false, **kwargs)).call.name - else - SyntaxSuggest::PathnameFromMessage.new(message).call.name - end - - io = SyntaxSuggest::MiniStringIO.new + file = path if file + file = Pathname.new(file) + io = SyntaxSuggest::MiniStringIO.new + SyntaxSuggest.call( io: io, source: file.read,