Merge pull request #2707 from calmh/notok

The "OK" log level is silly and should not exist
This commit is contained in:
Audrius Butkevicius 2016-01-16 22:21:06 +00:00
commit 693e1c93f1
4 changed files with 8 additions and 36 deletions

View File

@ -338,7 +338,7 @@ func main() {
if err != nil { if err != nil {
l.Fatalln("Upgrade:", err) // exits 1 l.Fatalln("Upgrade:", err) // exits 1
} }
l.Okln("Upgraded from", options.upgradeTo) l.Infoln("Upgraded from", options.upgradeTo)
return return
} }
@ -462,14 +462,14 @@ func performUpgrade(release upgrade.Release) {
if err != nil { if err != nil {
l.Fatalln("Upgrade:", err) l.Fatalln("Upgrade:", err)
} }
l.Okf("Upgraded to %q", release.Tag) l.Infof("Upgraded to %q", release.Tag)
} else { } else {
l.Infoln("Attempting upgrade through running Syncthing...") l.Infoln("Attempting upgrade through running Syncthing...")
err = upgradeViaRest() err = upgradeViaRest()
if err != nil { if err != nil {
l.Fatalln("Upgrade:", err) l.Fatalln("Upgrade:", err)
} }
l.Okln("Syncthing upgrading") l.Infoln("Syncthing upgrading")
os.Exit(exitUpgrading) os.Exit(exitUpgrading)
} }
} }
@ -850,7 +850,7 @@ func syncthingMain(runtimeOptions RuntimeOptions) {
mainService.Stop() mainService.Stop()
l.Okln("Exiting") l.Infoln("Exiting")
if runtimeOptions.cpuProfile { if runtimeOptions.cpuProfile {
pprof.StopCPUProfile() pprof.StopCPUProfile()

View File

@ -23,7 +23,6 @@ const (
LevelDebug LogLevel = iota LevelDebug LogLevel = iota
LevelVerbose LevelVerbose
LevelInfo LevelInfo
LevelOK
LevelWarn LevelWarn
LevelFatal LevelFatal
NumLevels NumLevels
@ -42,8 +41,6 @@ type Logger interface {
Verbosef(format string, vals ...interface{}) Verbosef(format string, vals ...interface{})
Infoln(vals ...interface{}) Infoln(vals ...interface{})
Infof(format string, vals ...interface{}) Infof(format string, vals ...interface{})
Okln(vals ...interface{})
Okf(format string, vals ...interface{})
Warnln(vals ...interface{}) Warnln(vals ...interface{})
Warnf(format string, vals ...interface{}) Warnf(format string, vals ...interface{})
Fatalln(vals ...interface{}) Fatalln(vals ...interface{})
@ -165,24 +162,6 @@ func (l *logger) Infof(format string, vals ...interface{}) {
l.callHandlers(LevelInfo, s) l.callHandlers(LevelInfo, s)
} }
// Okln logs a line with an OK prefix.
func (l *logger) Okln(vals ...interface{}) {
l.mut.Lock()
defer l.mut.Unlock()
s := fmt.Sprintln(vals...)
l.logger.Output(2, "OK: "+s)
l.callHandlers(LevelOK, s)
}
// Okf logs a formatted line with an OK prefix.
func (l *logger) Okf(format string, vals ...interface{}) {
l.mut.Lock()
defer l.mut.Unlock()
s := fmt.Sprintf(format, vals...)
l.logger.Output(2, "OK: "+s)
l.callHandlers(LevelOK, s)
}
// Warnln logs a formatted line with a WARNING prefix. // Warnln logs a formatted line with a WARNING prefix.
func (l *logger) Warnln(vals ...interface{}) { func (l *logger) Warnln(vals ...interface{}) {
l.mut.Lock() l.mut.Lock()

View File

@ -19,8 +19,6 @@ func TestAPI(t *testing.T) {
l.AddHandler(LevelDebug, checkFunc(t, LevelDebug, &debug)) l.AddHandler(LevelDebug, checkFunc(t, LevelDebug, &debug))
info := 0 info := 0
l.AddHandler(LevelInfo, checkFunc(t, LevelInfo, &info)) l.AddHandler(LevelInfo, checkFunc(t, LevelInfo, &info))
ok := 0
l.AddHandler(LevelOK, checkFunc(t, LevelOK, &ok))
warn := 0 warn := 0
l.AddHandler(LevelWarn, checkFunc(t, LevelWarn, &warn)) l.AddHandler(LevelWarn, checkFunc(t, LevelWarn, &warn))
@ -28,20 +26,15 @@ func TestAPI(t *testing.T) {
l.Debugln("test", 0) l.Debugln("test", 0)
l.Infof("test %d", 1) l.Infof("test %d", 1)
l.Infoln("test", 1) l.Infoln("test", 1)
l.Okf("test %d", 2)
l.Okln("test", 2)
l.Warnf("test %d", 3) l.Warnf("test %d", 3)
l.Warnln("test", 3) l.Warnln("test", 3)
if debug != 8 { if debug != 6 {
t.Errorf("Debug handler called %d != 8 times", debug) t.Errorf("Debug handler called %d != 8 times", debug)
} }
if info != 6 { if info != 4 {
t.Errorf("Info handler called %d != 6 times", info) t.Errorf("Info handler called %d != 6 times", info)
} }
if ok != 4 {
t.Errorf("Ok handler called %d != 4 times", ok)
}
if warn != 2 { if warn != 2 {
t.Errorf("Warn handler called %d != 2 times", warn) t.Errorf("Warn handler called %d != 2 times", warn)
} }

View File

@ -189,7 +189,7 @@ func (m *Model) StartFolderRW(folder string) {
m.folderRunnerTokens[folder] = append(m.folderRunnerTokens[folder], token) m.folderRunnerTokens[folder] = append(m.folderRunnerTokens[folder], token)
m.fmut.Unlock() m.fmut.Unlock()
l.Okln("Ready to synchronize", folder, "(read-write)") l.Infoln("Ready to synchronize", folder, "(read-write)")
} }
func (m *Model) warnAboutOverwritingProtectedFiles(folder string) { func (m *Model) warnAboutOverwritingProtectedFiles(folder string) {
@ -241,7 +241,7 @@ func (m *Model) StartFolderRO(folder string) {
m.folderRunnerTokens[folder] = append(m.folderRunnerTokens[folder], token) m.folderRunnerTokens[folder] = append(m.folderRunnerTokens[folder], token)
m.fmut.Unlock() m.fmut.Unlock()
l.Okln("Ready to synchronize", folder, "(read only; no external updates accepted)") l.Infoln("Ready to synchronize", folder, "(read only; no external updates accepted)")
} }
func (m *Model) RemoveFolder(folder string) { func (m *Model) RemoveFolder(folder string) {