Skip to content

Refactor out base TextLogger from StreamLogger #2406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
timcassell merged 4 commits into from
Aug 18, 2023
Merged

Refactor out base TextLogger from StreamLogger #2406

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1653079
Refactor out base TextWriterLogger from StreamLogger
nietras Aug 17, 2023
8869a21
rename to TextLogger
nietras Aug 17, 2023
e412712
fix Id
nietras Aug 18, 2023
03f7d49
rearrange
nietras Aug 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
rearrange
  • Loading branch information
@nietras
nietras committed Aug 18, 2023
commit 03f7d49bb765c5bfb77716f535ea51a5a1779790
4 changes: 2 additions & 2 deletions src/BenchmarkDotNet/Loggers/TextLogger.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ public class TextLogger : ILogger, IDisposable

public TextLogger(TextWriter writer) => this.writer = writer;

public void Dispose() => writer.Dispose();

public virtual string Id => nameof(TextLogger);
public int Priority => 0;

Expand All @@ -21,5 +19,7 @@ public class TextLogger : ILogger, IDisposable
public void WriteLine(LogKind logKind, string text) => writer.WriteLine(text);

public void Flush() => writer.Flush();

public void Dispose() => writer.Dispose();
}
}