From 5111742a5a100e44fccf191b12a13c452b025944 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Thu, 20 Jan 2022 18:34:17 +0000 Subject: get rid of DTAS.dedupe_str wrapper Ruby 2.3+ supports String#@-, though it did not deduplicate strings. But 2.5 is already old at this point and most users can be expected to have it. This gives some memory regressions for Ruby <= 2.4 users, but cuts down on the code we maintain and reduces bytecode overhead for 2.5+ users. --- lib/dtas/watchable/fiddle_ino.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/dtas/watchable/fiddle_ino.rb') diff --git a/lib/dtas/watchable/fiddle_ino.rb b/lib/dtas/watchable/fiddle_ino.rb index e85fea1..b24bbf6 100644 --- a/lib/dtas/watchable/fiddle_ino.rb +++ b/lib/dtas/watchable/fiddle_ino.rb @@ -1,4 +1,4 @@ -# Copyright (C) 2013-2020 all contributors +# Copyright (C) all contributors # License: GPL-3.0+ # frozen_string_literal: true require 'fiddle' @@ -53,7 +53,7 @@ class DTAS::Watchable::InotifyReadableIter # :nodoc: name.size == len or raise "short name #{name.inspect} != #{len}" name.sub!(/\0+\z/, '') or raise "missing: `\\0', inotify_event.name=#{name.inspect}" - name = DTAS.dedupe_str(name) + name = -name end ie = InotifyEvent.new(wd, mask, cookie, len, name) if event -- cgit v1.2.3-24-ge0c7