Skip to content

Commit 1a433cd

Browse files
committed
cli/command/node: inline vars and use struct literals
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
1 parent 3d2bd97 commit 1a433cd

1 file changed

Lines changed: 22 additions & 19 deletions

File tree

cli/command/node/formatter.go

Lines changed: 22 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -114,27 +114,31 @@ func FormatWrite(fmtCtx formatter.Context, nodes []swarm.Node, info system.Info)
114114

115115
// formatWrite writes the context.
116116
func formatWrite(fmtCtx formatter.Context, nodes []swarm.Node, info system.Info) error {
117-
render := func(format func(subContext formatter.SubContext) error) error {
117+
nodeCtx := &nodeContext{
118+
HeaderContext: formatter.HeaderContext{
119+
Header: formatter.SubHeaderContext{
120+
"ID": nodeIDHeader,
121+
"Self": selfHeader,
122+
"Hostname": hostnameHeader,
123+
"Status": formatter.StatusHeader,
124+
"Availability": availabilityHeader,
125+
"ManagerStatus": managerStatusHeader,
126+
"EngineVersion": engineVersionHeader,
127+
"TLSStatus": tlsStatusHeader,
128+
},
129+
},
130+
}
131+
return fmtCtx.Write(nodeCtx, func(format func(subContext formatter.SubContext) error) error {
118132
for _, node := range nodes {
119-
nodeCtx := &nodeContext{n: node, info: info}
120-
if err := format(nodeCtx); err != nil {
133+
if err := format(&nodeContext{
134+
n: node,
135+
info: info,
136+
}); err != nil {
121137
return err
122138
}
123139
}
124140
return nil
125-
}
126-
nodeCtx := nodeContext{}
127-
nodeCtx.Header = formatter.SubHeaderContext{
128-
"ID": nodeIDHeader,
129-
"Self": selfHeader,
130-
"Hostname": hostnameHeader,
131-
"Status": formatter.StatusHeader,
132-
"Availability": availabilityHeader,
133-
"ManagerStatus": managerStatusHeader,
134-
"EngineVersion": engineVersionHeader,
135-
"TLSStatus": tlsStatusHeader,
136-
}
137-
return fmtCtx.Write(&nodeCtx, render)
141+
})
138142
}
139143

140144
type nodeContext struct {
@@ -205,7 +209,7 @@ func inspectFormatWrite(fmtCtx formatter.Context, refs []string, getRef inspect.
205209
if fmtCtx.Format != nodeInspectPrettyTemplate {
206210
return inspect.Inspect(fmtCtx.Output, refs, string(fmtCtx.Format), getRef)
207211
}
208-
render := func(format func(subContext formatter.SubContext) error) error {
212+
return fmtCtx.Write(&nodeInspectContext{}, func(format func(subContext formatter.SubContext) error) error {
209213
for _, ref := range refs {
210214
nodeI, _, err := getRef(ref)
211215
if err != nil {
@@ -220,8 +224,7 @@ func inspectFormatWrite(fmtCtx formatter.Context, refs []string, getRef inspect.
220224
}
221225
}
222226
return nil
223-
}
224-
return fmtCtx.Write(&nodeInspectContext{}, render)
227+
})
225228
}
226229

227230
type nodeInspectContext struct {

0 commit comments

Comments
 (0)