Permalink
Browse files

fix ineffassign: use processes array

uses processes variable instead of calling the Processes() func again

Signed-off-by: Daniel Dao <dqminh89@gmail.com>
  • Loading branch information...
1 parent 23a644d commit 726b88f9b73f4dfeb5c4a344af592699b58b6af8 @dqminh dqminh committed Jan 11, 2017
Showing with 1 addition and 1 deletion.
  1. +1 −1 execution/service.go
@@ -35,7 +35,7 @@ func New(ctx context.Context, executor Executor) (*Service, error) {
// generate exit event for all processes, (generate event for init last)
processes := c.Processes()
processes = append(processes[1:], processes[0])
- for _, p := range c.Processes() {
+ for _, p := range processes {
if p.Status() != Stopped {
p.Signal(os.Kill)
}

0 comments on commit 726b88f

Please sign in to comment.