From 87768a047cc4ea16cd031e47ae9c9272f870adc5 Mon Sep 17 00:00:00 2001 From: Satnam Singh Date: Mon, 22 Sep 2014 15:20:14 -0700 Subject: [PATCH] Minor Go-style suggeston: move decode and error assignment into if clause check. --- api/handler.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/api/handler.go b/api/handler.go index 56ab8a7d..b45e74e5 100644 --- a/api/handler.go +++ b/api/handler.go @@ -172,8 +172,7 @@ func getContainerInfoRequest(body io.ReadCloser) (*info.ContainerInfoRequest, er query.NumStats = 64 decoder := json.NewDecoder(body) - err := decoder.Decode(&query) - if err != nil && err != io.EOF { + if err := decoder.Decode(&query); err != nil && err != io.EOF { return nil, fmt.Errorf("unable to decode the json value: %s", err) }