From 5b5f80be7d6f631d3b282cc883609b6f306f7871 Mon Sep 17 00:00:00 2001 From: Marc Di Luzio Date: Sat, 27 Jun 2020 02:02:18 +0100 Subject: [PATCH] Clean up logging a little --- cmd/rove-accountant/main.go | 3 +-- pkg/game/world.go | 4 +--- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/cmd/rove-accountant/main.go b/cmd/rove-accountant/main.go index 31f2914..e774797 100644 --- a/cmd/rove-accountant/main.go +++ b/cmd/rove-accountant/main.go @@ -68,8 +68,7 @@ func (a *accountantServer) GetValue(_ context.Context, in *accounts.DataKey) (*a a.sync.RLock() defer a.sync.RUnlock() - // Try and fetch the rover - log.Printf("Getting value for account %s: %s\n", in.Account, in.Key) + // Try and fetch the value data, err := a.accountant.GetValue(in.Account, in.Key) if err != nil { log.Printf("Error: %s\n", err) diff --git a/pkg/game/world.go b/pkg/game/world.go index 8491a7d..9048900 100644 --- a/pkg/game/world.go +++ b/pkg/game/world.go @@ -430,16 +430,14 @@ func (w *World) ExecuteCommandQueues() { // ExecuteCommand will execute a single command func (w *World) ExecuteCommand(c *Command, rover string) (err error) { - log.Printf("Executing command: %+v\n", *c) + log.Printf("Executing command: %+v for %s\n", *c, rover) switch c.Command { case CommandMove: if dir, err := bearing.FromString(c.Bearing); err != nil { return err - } else if _, err := w.MoveRover(rover, dir); err != nil { return err - } case CommandStash: