commit c2ef61f450233d3c629ade8eb2083e43fa75c97a
author Rhys Hiltner <rhys@justin.tv> Fri Sep 23 14:57:19 2016 -0700
committer Brad Fitzpatrick <bradfitz@golang.org> Sat Sep 24 00:48:36 2016 +0000
tree 15fad02a07f58b9a3c86fed344b97c9a3793509c
parent 3f4088edb48e8a4e3c66a5f8e7b2a78615fcb83f

go/ast/astutil: fix loop logic in AddNamedImport

When merging import declarations into a single block, AddNamedImport
modifies the list of declarations in the provided file while
iterating over the list. Take care to adjust the index into the list
so as to not skip entries or fall off the end.

Fixes golang/go#17213

Change-Id: I807246f762c965ea1fc51eb57759f6088336db86
Reviewed-on: https://go-review.googlesource.com/29681
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>