From 29b300146dcd4499e07f8e99f3ce6d1e6c88796a Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 17 Mar 2017 00:07:38 +0000 Subject: epoll: add newline to Kernel#warn messages for atomicity Relying on the record separator ($\ or "\n") is not atomic on multi-process or multi-threaded systems writing to the same log file or pipe. --- lib/sleepy_penguin/epoll.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/sleepy_penguin/epoll.rb b/lib/sleepy_penguin/epoll.rb index 637db8d..da1502e 100644 --- a/lib/sleepy_penguin/epoll.rb +++ b/lib/sleepy_penguin/epoll.rb @@ -166,7 +166,7 @@ class SleepyPenguin::Epoll begin @io.epoll_ctl(CTL_MOD, io, events) rescue Errno::ENOENT - warn "epoll event cache failed (mod -> add)" + warn "epoll event cache failed (mod -> add)\n" @io.epoll_ctl(CTL_ADD, io, events) @marks[fd] = io end @@ -174,7 +174,7 @@ class SleepyPenguin::Epoll begin @io.epoll_ctl(CTL_ADD, io, events) rescue Errno::EEXIST - warn "epoll event cache failed (add -> mod)" + warn "epoll event cache failed (add -> mod)\n" @io.epoll_ctl(CTL_MOD, io, events) end @marks[fd] = io -- cgit v1.2.3-24-ge0c7