Address issues found by golint.

* No functional changes
* Did not address the ALL_CAPS issues
* Did not modify public APIs and field names.

logger/log.go

@@ -1,4 +1,6 @@
 // Copyright 2012-2015 Apcera Inc. All rights reserved.
+
+//Package logger provides logging facilities for the NATS server
 package logger
 
 import (
@@ -7,6 +9,7 @@ import (
 	"os"
 )
 
+// Logger is the server logger
 type Logger struct {
 	logger     *log.Logger
 	debug      bool
@@ -18,6 +21,7 @@ type Logger struct {
 	traceLabel string
 }
 
+// NewStdLogger creates a logger with output directed to Stderr
 func NewStdLogger(time, debug, trace, colors, pid bool) *Logger {
 	flags := 0
 	if time {
@@ -44,6 +48,7 @@ func NewStdLogger(time, debug, trace, colors, pid bool) *Logger {
 	return l
 }
 
+// NewFileLogger creates a logger with output directed to a file
 func NewFileLogger(filename string, time, debug, trace, pid bool) *Logger {
 	fileflags := os.O_WRONLY | os.O_APPEND | os.O_CREATE
 	f, err := os.OpenFile(filename, fileflags, 0660)
@@ -93,24 +98,29 @@ func setColoredLabelFormats(l *Logger) {
 	l.traceLabel = fmt.Sprintf(colorFormat, 33, "TRC")
 }
 
+// Noticef logs a notice statement
 func (l *Logger) Noticef(format string, v ...interface{}) {
 	l.logger.Printf(l.infoLabel+format, v...)
 }
 
+// Errorf logs an error statement
 func (l *Logger) Errorf(format string, v ...interface{}) {
 	l.logger.Printf(l.errorLabel+format, v...)
 }
 
+// Fatalf logs a fatal error
 func (l *Logger) Fatalf(format string, v ...interface{}) {
 	l.logger.Fatalf(l.fatalLabel+format, v...)
 }
 
+// Debugf logs a debug statement
 func (l *Logger) Debugf(format string, v ...interface{}) {
 	if l.debug == true {
 		l.logger.Printf(l.debugLabel+format, v...)
 	}
 }
 
+// Tracef logs a trace statement
 func (l *Logger) Tracef(format string, v ...interface{}) {
 	if l.trace == true {
 		l.logger.Printf(l.traceLabel+format, v...)

logger/syslog.go

@@ -11,12 +11,14 @@ import (
 	"net/url"
 )
 
+// SysLogger provides a system logger facility
 type SysLogger struct {
 	writer *syslog.Writer
 	debug  bool
 	trace  bool
 }
 
+// NewSysLogger creates a new system logger
 func NewSysLogger(debug, trace bool) *SysLogger {
 	w, err := syslog.New(syslog.LOG_DAEMON|syslog.LOG_NOTICE, "gnatsd")
 	if err != nil {
@@ -30,6 +32,7 @@ func NewSysLogger(debug, trace bool) *SysLogger {
 	}
 }
 
+// NewRemoteSysLogger creates a new remote system logger
 func NewRemoteSysLogger(fqn string, debug, trace bool) *SysLogger {
 	network, addr := getNetworkAndAddr(fqn)
 	w, err := syslog.Dial(network, addr, syslog.LOG_DEBUG, "gnatsd")
@@ -62,24 +65,29 @@ func getNetworkAndAddr(fqn string) (network, addr string) {
 	return
 }
 
+// Noticef logs a notice statement
 func (l *SysLogger) Noticef(format string, v ...interface{}) {
 	l.writer.Notice(fmt.Sprintf(format, v...))
 }
 
+// Fatalf logs a fatal error
 func (l *SysLogger) Fatalf(format string, v ...interface{}) {
 	l.writer.Crit(fmt.Sprintf(format, v...))
 }
 
+// Errorf logs an error statement
 func (l *SysLogger) Errorf(format string, v ...interface{}) {
 	l.writer.Err(fmt.Sprintf(format, v...))
 }
 
+// Debugf logs a debug statement
 func (l *SysLogger) Debugf(format string, v ...interface{}) {
 	if l.debug {
 		l.writer.Debug(fmt.Sprintf(format, v...))
 	}
 }
 
+// Tracef logs a trace statement
 func (l *SysLogger) Tracef(format string, v ...interface{}) {
 	if l.trace {
 		l.writer.Notice(fmt.Sprintf(format, v...))

logger/syslog_test.go

@@ -150,7 +150,7 @@ func expectSyslogOutput(t *testing.T, line string, expected string) {
 
 func runSyslog(c net.PacketConn, done chan<- string) {
 	var buf [4096]byte
-	var rcvd string = ""
+	var rcvd string
 	for {
 		n, _, err := c.ReadFrom(buf[:])
 		if err != nil || n == 0 {