fix: code review notes

This commit is contained in:
Nikita Semenov 2022-12-29 21:45:18 +03:00
parent 57d40f18ef
commit cce06d57af
5 changed files with 6 additions and 12 deletions

View File

@ -25,7 +25,7 @@ import os
public struct DefaultLoggerHandler: LogHandler {
public var logInfo: OSLog
public var logger: LogOutputRepresentater?
public var logger: LogOutputStream?
public init(logInfo: OSLog) {
self.logInfo = logInfo
@ -33,7 +33,7 @@ public struct DefaultLoggerHandler: LogHandler {
if #available(iOS 14, *) {
self.logger = Logger(logInfo)
} else {
self.logger = DefaultOutput()
self.logger = DefaultOutputStream()
}
}

View File

@ -25,9 +25,3 @@ import os
public protocol LogHandler {
func log(type: OSLogType, log: OSLog?, _ message: String)
}
public extension LogHandler {
func log(type: OSLogType, log: OSLog?, _ message: String) {
// empty implementation
}
}

View File

@ -23,7 +23,7 @@
import os
import _SwiftOSOverlayShims
public struct DefaultOutput: LogOutputRepresentater {
public struct DefaultOutputStream: LogOutputStream {
public func log(type: OSLogType, log: OSLog?, _ message: String) {
guard let logInfo = log, logInfo.isEnabled(type: type) else {
@ -34,7 +34,7 @@ public struct DefaultOutput: LogOutputRepresentater {
var mutableMessage = message
mutableMessage.withUTF8 { (buf: UnsafeBufferPointer<UInt8>) in
buf.baseAddress!.withMemoryRebound(to: CChar.self, capacity: buf.count) { str in
buf.baseAddress?.withMemoryRebound(to: CChar.self, capacity: buf.count) { str in
withVaList([]) { valist in
_swift_os_log(#dsohandle, ra, logInfo, type, str, valist)
}

View File

@ -22,6 +22,6 @@
import os
public protocol LogOutputRepresentater {
public protocol LogOutputStream {
func log(type: OSLogType, log: OSLog?, _ message: String)
}

View File

@ -23,7 +23,7 @@
import os
@available(iOS 14.0, *)
extension Logger: LogOutputRepresentater {
extension Logger: LogOutputStream {
public func log(type: OSLogType, log: OSLog?, _ message: String) {
self.log(level: type, "\(message)")
}