Minor corrections

Simplify logic in ui.go. In that diff, `st` would already have the
Reversed flag set; there's no need to set it again.

Also, minor changes to remove all 3 statickcheck errors VS code's
Go extension complains about.

nav.go

@@ -1353,14 +1353,17 @@ func (nav *nav) sync() error {
  }
 
  oldmarks := nav.marks
- err = nav.readMarks()
+ errMarks := nav.readMarks()
  for _, ch := range gOpts.tempmarks {
  tmp := string(ch)
  if v, e := oldmarks[tmp]; e {
  nav.marks[tmp] = v
  }
  }
  err = nav.readTags()
+ if errMarks != nil {
+ return errMarks
+ }
  return err
 }
 

server.go

@@ -61,7 +61,7 @@ func listen(l net.Listener) {
 
 func echoerr(c net.Conn, msg string) {
  fmt.Fprintln(c, msg)
- log.Printf(msg)
+ log.Print(msg)
 }
 
 func echoerrf(c net.Conn, format string, a ...interface{}) {

ui.go

@@ -453,7 +453,7 @@ func (win *win) printDir(screen tcell.Screen, dir *dir, selections map[string]in
  tag, ok := tags[path]
  if ok {
  if i == dir.pos {
- win.print(screen, lnwidth+1, i, st.Reverse(true), tag)
+ win.print(screen, lnwidth+1, i, st, tag)
  } else {
  win.print(screen, lnwidth+1, i, tcell.StyleDefault, fmt.Sprintf(gOpts.tagfmt, tag))
  }
@@ -611,10 +611,6 @@ func (ui *ui) echomsg(msg string) {
  log.Print(msg)
 }
 
-func (ui *ui) echomsgf(format string, a ...interface{}) {
- ui.echomsg(fmt.Sprintf(format, a...))
-}
-
 func (ui *ui) echoerr(msg string) {
  ui.msg = fmt.Sprintf(gOpts.errorfmt, msg)
  log.Printf("error: %s", msg)