nodoc internals

We do not expose any sort of API beyond what's in the config file
manpage to our users.  Do not mislead them into thinking we
currently have a stable API (though I'm considering one).

This avoids wasting disk space and installation time for users who
do not have a: "gem: --no-ri --no-rdoc" line in their ~/.gemrc

5 files changed, 20 insertions, 15 deletions

diff --git a/lib/yahns.rb b/lib/yahns.rb
index 641959c..bbf86f2 100644
--- a/lib/yahns.rb
+++ b/lib/yahns.rb
@@ -14,9 +14,13 @@ require 'sleepy_penguin'
     Unicorn.__send__(:remove_const, sym) if Unicorn.const_defined?(sym)
 end
 
-# yahns exposes no user-visible API outside of the config file
+# yahns exposes no user-visible API outside of the config file.
+# See http://yahns.yhbt.net/yahns_config.txt for the config documentation
+# and http://yahns.yhbt.net/ for the homepage.
 # Internals are subject to change.
-module Yahns # :nodoc:
+
+module Yahns
+  # :stopdoc:
   # We populate this at startup so we can figure out how to reexecute
   # and upgrade the currently running instance of yahns
   # Unlike unicorn, this Hash is NOT a stable/public interface.
@@ -57,15 +61,16 @@ module Yahns # :nodoc:
   # try to use the monotonic clock in Ruby >= 2.1, it is immune to clock
   # offset adjustments and generates less garbage (Float vs Time object)
   begin
-    Process.clock_gettime(Process::CLOCK_MONOTONIC)
-    def self.now
-      Process.clock_gettime(Process::CLOCK_MONOTONIC)
+    def self.now # :nodoc:
+      Process.clock_gettime(Process::CLOCK_MONOTONIC) # :nodoc:
     end
   rescue NameError, NoMethodError
-    def self.now # Ruby <= 2.0
-      Time.now.to_f
+    def self.now # :nodoc:
+      Time.now.to_f # Ruby <= 2.0
     end
   end
+
+  # :startdoc:
 end
 
 # FIXME: require lazily
diff --git a/lib/yahns/config.rb b/lib/yahns/config.rb
index e39115c..2b71fe5 100644
--- a/lib/yahns/config.rb
+++ b/lib/yahns/config.rb
@@ -18,6 +18,10 @@ class Yahns::Config # :nodoc:
     @config_file = config_file
     @block = nil
     config_reload!
+
+    # FIXME: we shouldn't have this at all when we go Unicorn 5-only
+    Unicorn::HttpParser.respond_to?(:keepalive_requests=) and
+      Unicorn::HttpParser.keepalive_requests = 0xffffffff
   end
 
   def _check_in_block(ctx, var)
diff --git a/lib/yahns/http_client.rb b/lib/yahns/http_client.rb
index db53587..8a341f9 100644
--- a/lib/yahns/http_client.rb
+++ b/lib/yahns/http_client.rb
@@ -2,11 +2,7 @@
 # Copyright (C) 2013-2015 all contributors <yahns-public@yhbt.net>
 # License: GPLv3 or later (https://www.gnu.org/licenses/gpl-3.0.txt)
 class Yahns::HttpClient < Kgio::Socket # :nodoc:
-  NULL_IO = StringIO.new("")
-
-  # FIXME: we shouldn't have this at all when we go Unicorn 5-only
-  Unicorn::HttpParser.respond_to?(:keepalive_requests=) and
-    Unicorn::HttpParser.keepalive_requests = 0xffffffff
+  NULL_IO = StringIO.new('') # :nodoc:
 
   include Yahns::HttpResponse
   QEV_FLAGS = Yahns::Queue::QEV_RD # used by acceptor
diff --git a/lib/yahns/proxy_pass.rb b/lib/yahns/proxy_pass.rb
index 9512318..728ab3e 100644
--- a/lib/yahns/proxy_pass.rb
+++ b/lib/yahns/proxy_pass.rb
@@ -10,7 +10,7 @@ require 'timeout'
 require_relative 'proxy_http_response'
 
 class Yahns::ProxyPass # :nodoc:
-  class ReqRes < Kgio::Socket
+  class ReqRes < Kgio::Socket # :nodoc:
     attr_writer :resbuf
     attr_accessor :proxy_trailers
 
diff --git a/lib/yahns/sendfile_compat.rb b/lib/yahns/sendfile_compat.rb
index da2a9ec..bd79c0c 100644
--- a/lib/yahns/sendfile_compat.rb
+++ b/lib/yahns/sendfile_compat.rb
@@ -2,7 +2,7 @@
 # Copyright (C) 2013-2015 all contributors <yahns-public@yhbt.net>
 # License: GPLv3 or later (https://www.gnu.org/licenses/gpl-3.0.txt)
 
-module Yahns::SendfileCompat
+module Yahns::SendfileCompat # :nodoc:
   def trysendfile(io, offset, count)
     return 0 if count == 0
     count = 0x4000 if count > 0x4000
@@ -22,6 +22,6 @@ module Yahns::SendfileCompat
   end
 end
 
-class IO
+class IO # :nodoc:
   include Yahns::SendfileCompat
 end