Skip to content

Commit

Permalink
remove unnecessary self-assignment
Browse files Browse the repository at this point in the history
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
3a3aee0

Signed-off-by: Šimon Lukašík <[email protected]>
  • Loading branch information
isimluk committed Sep 12, 2023
1 parent 7da3f79 commit c99c5e5
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions lib/fluent/fluent_log_event_router.rb
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit c99c5e5

Please sign in to comment.