[ARVADOS] updated: 5bf9312a5174f97f00db383836eb7666dc500293

Git user git at public.curoverse.com
Thu Feb 9 13:26:13 EST 2017


Summary of changes:
 sdk/go/manifest/manifest.go | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

       via  5bf9312a5174f97f00db383836eb7666dc500293 (commit)
      from  2b7834020290b28d797333f90fcb87e5da67d616 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5bf9312a5174f97f00db383836eb7666dc500293
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Thu Feb 9 13:25:40 2017 -0500

    9397: Use idomatic loop form.

diff --git a/sdk/go/manifest/manifest.go b/sdk/go/manifest/manifest.go
index 208cfdd..e8be7a2 100644
--- a/sdk/go/manifest/manifest.go
+++ b/sdk/go/manifest/manifest.go
@@ -200,7 +200,7 @@ func (s *ManifestStream) sendFileSegmentIterByName(filepath string, ch chan<- *F
 			// Shouldn't happen, file segments are checked in parseManifestStream
 			panic(fmt.Sprintf("File segment %v extends past end of stream", fTok))
 		}
-		for i < len(s.Blocks) {
+		for ; i < len(s.Blocks); i++ {
 			blockPos := s.BlockOffsets[i]
 			blockEnd := s.BlockOffsets[i+1]
 			if blockEnd <= wantPos {
@@ -227,7 +227,6 @@ func (s *ManifestStream) sendFileSegmentIterByName(filepath string, ch chan<- *F
 				fseg.Len = int(wantPos+wantLen-blockPos) - fseg.Offset
 			}
 			ch <- &fseg
-			i += 1
 		}
 	}
 }

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list