Skip to content

Commit

Permalink
Another performance improvement on the Formatter by using Kernel#spri…
Browse files Browse the repository at this point in the history
…ntf over String#% (#75)

* Prefer Kernel#sprintf over String#% for formatting Strings
  • Loading branch information
amatsuda authored Dec 8, 2022
1 parent b41d7c6 commit 4116759
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions lib/logger/formatter.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
class Logger
# Default formatter for log messages.
class Formatter
Format = "%s, [%s #%d] %5s -- %s: %s\n"
Format = "%.1s, [%s #%d] %5s -- %s: %s\n"
DatetimeFormat = "%Y-%m-%dT%H:%M:%S.%6N"

attr_accessor :datetime_format
Expand All @@ -13,8 +13,7 @@ def initialize
end

def call(severity, time, progname, msg)
Format % [severity[0, 1], format_datetime(time), Process.pid, severity, progname,
msg2str(msg)]
sprintf(Format, severity, format_datetime(time), Process.pid, severity, progname, msg2str(msg))
end

private
Expand Down

0 comments on commit 4116759

Please sign in to comment.