issue guard#510: switch to Listen::MonotonicTime.new except for :mtime

lib/listen/adapter/base.rb

@@ -93,9 +93,9 @@ def _stop
       end
 
       def _timed(title)
-        start = Time.now.to_f
+        start = MonotonicTime.now
         yield
-        diff = Time.now.to_f - start
+        diff = MonotonicTime.now - start
         Listen.logger.info format('%s: %.05f seconds', title, diff)
       rescue
         Listen.logger.warn "#{title} crashed: #{$ERROR_INFO.inspect}"

lib/listen/adapter/polling.rb

@@ -21,12 +21,13 @@ def _configure(_, &callback)
 
       def _run
         loop do
-          start = Time.now.to_f
+          start = MonotonicTime.now
           @polling_callbacks.each do |callback|
             callback.call(nil)
-            nap_time = options.latency - (Time.now.to_f - start)
-            # TODO: warn if nap_time is negative (polling too slow)
-            sleep(nap_time) if nap_time > 0
+            if (nap_time = options.latency - (MonotonicTime.now - start)) > 0
+              # TODO: warn if nap_time is negative (polling too slow)
+              sleep(nap_time)
+            end
           end
         end
       end

lib/listen/event/config.rb

@@ -29,10 +29,6 @@ def call(*args)
         @block.call(*args) if @block
       end
 
-      def timestamp
-        Time.now.to_f
-      end
-
       def callable?
         @block
       end

lib/listen/event/processor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'listen/monotonic_time'
+
 module Listen
   module Event
     class Processor
@@ -33,7 +35,7 @@ class Stopped < RuntimeError
 
       def _wait_until_events_calm_down
         loop do
-          now = _timestamp
+          now = MonotonicTime.now
 
           # Assure there's at least latency between callbacks to allow
           # for accumulating changes
@@ -70,7 +72,7 @@ def _sleep(seconds)
       end
 
       def _remember_time_of_first_unprocessed_event
-        @first_unprocessed_event_time ||= _timestamp
+        @first_unprocessed_event_time ||= MonotonicTime.now
       end
 
       def _reset_no_unprocessed_events
@@ -85,7 +87,7 @@ def _deadline
       # returns the event or `nil` when the event_queue is closed
       def _wait_until_events
         config.event_queue.pop.tap do |_event|
-          @first_unprocessed_event_time ||= _timestamp
+          @first_unprocessed_event_time ||= MonotonicTime.now
         end
       end
 
@@ -96,10 +98,6 @@ def _flush_wakeup_reasons
         end
       end
 
-      def _timestamp
-        config.timestamp
-      end
-
       # for easier testing without sleep loop
       def _process_changes(event)
         _reset_no_unprocessed_events
@@ -113,9 +111,9 @@ def _process_changes(event)
         result = [hash[:modified], hash[:added], hash[:removed]]
         return if result.all?(&:empty?)
 
-        block_start = _timestamp
+        block_start = MonotonicTime.now
         config.call(*result)
-        Listen.logger.debug "Callback took #{_timestamp - block_start} sec"
+        Listen.logger.debug "Callback took #{MonotonicTime.now - block_start} sec"
       end
 
       attr_reader :config

spec/lib/listen/event/processor_spec.rb

@@ -18,7 +18,7 @@
   end
 
   let(:state) do
-    { time: 0 }
+    { time: 0.0 }
   end
 
   def status_for_time(time)
@@ -39,7 +39,7 @@ def status_for_time(time)
       status_for_time(state[:time]) == :paused
     end
 
-    allow(config).to receive(:timestamp) do
+    allow(Time).to receive(:now) do
       state[:time]
     end
   end
@@ -67,9 +67,9 @@ def status_for_time(time)
         it 'does not sleep' do
           expect(config).to_not receive(:sleep)
           expect(event_queue).to receive(:pop).and_return(event)
-          t = Time.now.to_f
+          t = Listen::MonotonicTime.now
           subject.loop_for(1)
-          diff = Time.now.to_f - t
+          diff = Listen::MonotonicTime.now - t
           expect(diff).to be < 0.01
         end
       end