From c99c5e53cbad6139f6e7ab988aaf5a1f1c00a025 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0imon=20Luka=C5=A1=C3=ADk?= Date: Tue, 12 Sep 2023 14:16:06 +0200 Subject: [PATCH] remove unnecessary self-assignment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Previously this code was assigning instance variable like @log_event_router = log_event_router later, during the refactoring of Fluent Engine it was changed to log_event_router = log_event_router which can be now removed. For reference please review changeset at 3a3aee0227b621b61d1d7f258147f7ff661316e0 Signed-off-by: Šimon Lukašík --- lib/fluent/fluent_log_event_router.rb | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/fluent/fluent_log_event_router.rb b/lib/fluent/fluent_log_event_router.rb index b31ac16ce0..3ab7c6c393 100644 --- a/lib/fluent/fluent_log_event_router.rb +++ b/lib/fluent/fluent_log_event_router.rb @@ -47,8 +47,6 @@ def self.build(root_agent) # it's not suppressed in default event router for non-log-event events log_event_router.suppress_missing_match! - log_event_router = log_event_router - unmatched_tags = Fluent::Log.event_tags.select { |t| !log_event_router.match?(t) } unless unmatched_tags.empty? $log.warn "match for some tags of log events are not defined in @FLUENT_LOG label (to be ignored)", tags: unmatched_tags