Print timing for page requests and re-parsing in server mode (#1648)

## Summary

- Silence the stats progress bar in server mode — it uses `\r` on a tty
which collides with the server's own logging
- Print response time for page requests (not assets or status polls):
`200 /RDoc.html (12.3ms)`
- Print re-parse duration with relative paths: `Re-parsed lib/rdoc.rb
(17.5ms)`

```
Serving documentation at: http://localhost:4000
Press Ctrl+C to stop.

200 /RDoc.html (14.9ms)
Re-parsed lib/rdoc.rb (15.7ms)
200 /RDoc.html (8.5ms)
```

Author: st0012

lib/rdoc/server.rb

@@ -67,6 +67,8 @@ def initialize(rdoc, port)
     @store = rdoc.store
     @port = port
 
+    # Silence stats output — the server prints its own timing.
+    @rdoc.stats.verbosity = 0
     @generator = create_generator
     @template_dir = File.expand_path(@generator.template_dir)
     @page_cache = {}
@@ -102,6 +104,12 @@ def start
 
   private
 
+  def measure
+    start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+    yield
+    ((Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time) * 1000).round(1)
+  end
+
   def create_generator
     gen = RDoc::Generator::Aliki.new(@store, @options)
     gen.file_output = false
@@ -138,7 +146,14 @@ def handle_client(client)
       return write_response(client, 405, 'text/plain', 'Method Not Allowed')
     end
 
-    status, content_type, body = route(path)
+    if path.start_with?('/__') || %r{\A/(?:css|js)/}.match?(path)
+      status, content_type, body = route(path)
+    else
+      duration_ms = measure do
+        status, content_type, body = route(path)
+      end
+      $stderr.puts "#{status} #{path} (#{duration_ms}ms)"
+    end
     write_response(client, status, content_type, body)
   rescue => e
     write_response(client, 500, 'text/html', <<~HTML)
@@ -185,6 +200,8 @@ def write_response(client, status, content_type, body)
     client.write(header)
     client.write(body_bytes)
     client.flush
+  rescue Errno::EPIPE
+    # Client disconnected before we finished writing — harmless.
   end
 
   ##
@@ -347,19 +364,23 @@ def reparse_and_refresh(changed_files, removed_files)
       end
 
       unless changed_files.empty?
-        $stderr.puts "Re-parsing: #{changed_files.join(', ')}"
-        changed_files.each do |f|
-          begin
+        changed_file_names = []
+        duration_ms = measure do
+          changed_files.each do |f|
             relative = @rdoc.relative_path_for(f)
-            @store.clear_file_contributions(relative, keep_position: true)
-            @rdoc.parse_file(f)
-            @file_mtimes[f] = File.mtime(f) rescue nil
-          rescue => e
-            $stderr.puts "Error parsing #{f}: #{e.message}"
+            changed_file_names << relative
+            begin
+              @store.clear_file_contributions(relative, keep_position: true)
+              @rdoc.parse_file(f)
+              @file_mtimes[f] = File.mtime(f) rescue nil
+            rescue => e
+              $stderr.puts "Error parsing #{f}: #{e.message}"
+            end
           end
-        end
 
-        @store.cleanup_stale_contributions
+          @store.cleanup_stale_contributions
+        end
+        $stderr.puts "Re-parsed #{changed_file_names.join(', ')} (#{duration_ms}ms)"
       end
 
       @store.complete(@options.visibility)

lib/rdoc/stats.rb

@@ -39,10 +39,17 @@ def initialize(store, num_files, verbosity = 1)
     @start            = Time.now
     @undoc_params     = 0
 
+    self.verbosity = verbosity
+  end
+
+  ##
+  # Sets the verbosity level, rebuilding the display outputter.
+
+  def verbosity=(verbosity)
     @display = case verbosity
-               when 0 then Quiet.new   num_files
-               when 1 then Normal.new  num_files
-               else        Verbose.new num_files
+               when 0 then Quiet.new   @num_files
+               when 1 then Normal.new  @num_files
+               else        Verbose.new @num_files
                end
   end