Remove redundant "begin" block

Author: ono-max

lib/debug/server_dap.rb

@@ -119,29 +119,27 @@ class RetryBecauseCantRead < Exception
     end
 
     def recv_request
-      begin
-        r = IO.select([@sock])
-
-        @session.process_group.sync do
-          raise RetryBecauseCantRead unless IO.select([@sock], nil, nil, 0)
-
-          case header = @sock.gets
-          when /Content-Length: (\d+)/
-            b = @sock.read(2)
-            raise b.inspect unless b == "\r\n"
-
-            l = @sock.read(s = $1.to_i)
-            show_protocol :>, l
-            JSON.load(l)
-          when nil
-            nil
-          else
-            raise "unrecognized line: #{l} (#{l.size} bytes)"
-          end
+      r = IO.select([@sock])
+
+      @session.process_group.sync do
+        raise RetryBecauseCantRead unless IO.select([@sock], nil, nil, 0)
+
+        case header = @sock.gets
+        when /Content-Length: (\d+)/
+          b = @sock.read(2)
+          raise b.inspect unless b == "\r\n"
+
+          l = @sock.read(s = $1.to_i)
+          show_protocol :>, l
+          JSON.load(l)
+        when nil
+          nil
+        else
+          raise "unrecognized line: #{l} (#{l.size} bytes)"
         end
-      rescue RetryBecauseCantRead
-        retry
       end
+    rescue RetryBecauseCantRead
+      retry
     end
 
     def process

lib/debug/source_repository.rb

@@ -40,12 +40,10 @@ def all_iseq iseq, rs = []
     end
 
     private def add_path path
-      begin
-        src = File.read(path)
-        src = src.gsub("\r\n", "\n") # CRLF -> LF
-        @files[path] = SrcInfo.new(src.lines)
-      rescue SystemCallError
-      end
+      src = File.read(path)
+      src = src.gsub("\r\n", "\n") # CRLF -> LF
+      @files[path] = SrcInfo.new(src.lines)
+    rescue SystemCallError
     end
 
     private def get_si iseq

lib/debug/thread_client.rb

@@ -341,32 +341,30 @@ def instance_eval_for_cmethod frame_self, src
     end
 
     def frame_eval src, re_raise: false
-      begin
-        @success_last_eval = false
+      @success_last_eval = false
 
-        b = current_frame&.eval_binding || TOPLEVEL_BINDING
+      b = current_frame&.eval_binding || TOPLEVEL_BINDING
 
-        result = if b
-                   f, _l = b.source_location
-                   b.eval(src, "(rdbg)/#{f}")
-                 else
-                   frame_self = current_frame.self
-                   instance_eval_for_cmethod(frame_self, src)
-                 end
-        @success_last_eval = true
-        result
+      result = if b
+                  f, _l = b.source_location
+                  b.eval(src, "(rdbg)/#{f}")
+                else
+                  frame_self = current_frame.self
+                  instance_eval_for_cmethod(frame_self, src)
+                end
+      @success_last_eval = true
+      result
 
-      rescue Exception => e
-        return yield(e) if block_given?
+    rescue Exception => e
+      return yield(e) if block_given?
 
-        puts "eval error: #{e}"
+      puts "eval error: #{e}"
 
-        e.backtrace_locations&.each do |loc|
-          break if loc.path == __FILE__
-          puts "  #{loc}"
-        end
-        raise if re_raise
+      e.backtrace_locations&.each do |loc|
+        break if loc.path == __FILE__
+        puts "  #{loc}"
       end
+      raise if re_raise
     end
 
     def show_src(frame_index: @current_frame_index,