Skip to content

Commit 5f81447

Browse files
committed
make FileLogger a pure sink
1 parent 4f0e7a9 commit 5f81447

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

src/filelogger.jl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,16 @@ struct FileLogger <: AbstractLogger
77
always_flush::Bool
88
end
99

10-
function FileLogger(path::AbstractString; min_level=Info, append=false, always_flush=true)
10+
function FileLogger(path; append=false, always_flush=true)
1111
filehandle = open(path, append ? "a" : "w")
12-
FileLogger(SimpleLogger(filehandle, min_level), always_flush)
12+
FileLogger(SimpleLogger(filehandle, BelowMinLevel), always_flush)
1313
end
1414

1515

16-
function handle_message(filelogger::FileLogger, level, message, _module, group, id, file, line; kwargs...)
17-
handle_message(filelogger.logger, level, message, _module, group, id, file, line; kwargs...)
16+
function handle_message(filelogger::FileLogger, args...; kwargs...)
17+
handle_message(filelogger.logger, args...; kwargs...)
1818
filelogger.always_flush && flush(filelogger.logger.stream)
1919
end
20-
shouldlog(filelogger::FileLogger, level, _module, group, id) = shouldlog(filelogger.logger, level, _module, group, id)
21-
min_enabled_level(filelogger::FileLogger) = min_enabled_level(filelogger.logger)
20+
shouldlog(filelogger::FileLogger, arg...) = true
21+
min_enabled_level(filelogger::FileLogger) = BelowMinLevel
2222
catch_exceptions(filelogger::FileLogger) = catch_exceptions(filelogger.logger)

test/runtests.jl

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@ using Base.CoreLogging: Debug, Info, Warn
66
using Test: collect_test_logs, TestLogger
77

88

9-
10-
119
@testset "Demux" begin
1210
testlogger_info = TestLogger(min_level=Info)
1311
testlogger_warn = TestLogger(min_level=Warn)
@@ -22,8 +20,6 @@ using Test: collect_test_logs, TestLogger
2220

2321
end
2422

25-
26-
2723
@testset "Filter" begin
2824
testlogger = TestLogger()
2925
yodawg_filter(lvl, msg, args...; kwargs...) = startswith(msg, "Yo Dawg!")

0 commit comments

Comments
 (0)