Stylistic fixes

[govpn.git] / src / cypherpunks.ru / govpn / logger.go

diff --git a/src/cypherpunks.ru/govpn/logger.go b/src/cypherpunks.ru/govpn/logger.go
index 07b8c1a9869d23e781777b04ca4665a1096b1ce8..02dc2ae911f541976e648904f84d02e4971d210c 100644 (file)
--- a/src/cypherpunks.ru/govpn/logger.go
+++ b/src/cypherpunks.ru/govpn/logger.go
@@ -39,22 +39,16 @@ type syslogFormatter struct{}
 
 // Format converts a log entry into a list of bytes
 func (sf *syslogFormatter) Format(entry *logrus.Entry) ([]byte, error) {
-       buf := bytes.NewBuffer(nil)
-       var (
-               err   error
-               index int
-       )
-       keys := make([]string, len(entry.Data))
+       var buf bytes.Buffer
+       var err error
+       keys := make([]string, 0, len(entry.Data))
        for k := range entry.Data {
-               keys[index] = k
-               index++
+               keys = append(keys, k)
        }
        sort.Strings(keys)
 
-       for index = range keys {
-               k := keys[index]
-               v := entry.Data[k]
-               if _, err = buf.WriteString(fmt.Sprintf("[%s]%+v ", k, v)); err != nil {
+       for _, k := range keys {
+               if _, err = buf.WriteString(fmt.Sprintf("[%s]%+v ", k, entry.Data[k])); err != nil {
                        return nil, errors.Wrapf(err, "buf.WriteString %s", k)
                }
        }
@@ -64,7 +58,8 @@ func (sf *syslogFormatter) Format(entry *logrus.Entry) ([]byte, error) {
        return buf.Bytes(), nil
 }
 
-// NewLogger returns a logger for specified level. Syslog or Windows Events can be turned on.
+// NewLogger returns a logger for specified level. Syslog or Windows
+// Events can be turned on.
 func NewLogger(level string, syslog bool) (*logrus.Logger, error) {
        var logger *logrus.Logger
        logLevel, err := logrus.ParseLevel(level)