From 451a1022e16ec6307328125a41244a837c6edcdf Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 21 Mar 2009 22:46:34 -0700 Subject: Don't bother unlinking UNIX sockets Since we always unlink existing sockets when binding, there's no point in having code to unlink the sockets when we exit. Additionally, the old code path was racy. --- lib/unicorn.rb | 1 - lib/unicorn/socket.rb | 7 ------- 2 files changed, 8 deletions(-) diff --git a/lib/unicorn.rb b/lib/unicorn.rb index 2f86de2..eefbfc1 100644 --- a/lib/unicorn.rb +++ b/lib/unicorn.rb @@ -106,7 +106,6 @@ module Unicorn @listeners.delete_if do |io| if dead_names.include?(sock_name(io)) @io_purgatory.delete_if { |pio| pio.fileno == io.fileno } - destroy_safely(io) true else false diff --git a/lib/unicorn/socket.rb b/lib/unicorn/socket.rb index 4913261..9519448 100644 --- a/lib/unicorn/socket.rb +++ b/lib/unicorn/socket.rb @@ -62,13 +62,6 @@ module Unicorn end end - def destroy_safely(io) - if io.respond_to?(:path) && File.stat(io.path).ino == io.stat.ino - File.unlink(io.path) rescue nil - end - io.close rescue nil - end - # creates a new server, socket. address may be a HOST:PORT or # an absolute path to a UNIX socket. address can even be a Socket # object in which case it is immediately returned -- cgit v1.2.3-24-ge0c7