Merge lp:~sergiusens/ciborium/minor_check_loop_fix into lp:ciborium

Proposed by Sergio Schvezov
Status: Merged
Approved by: Ricardo Salveti
Approved revision: 66
Merged at revision: 59
Proposed branch: lp:~sergiusens/ciborium/minor_check_loop_fix
Merge into: lp:ciborium
Prerequisite: lp:~sergiusens/ciborium/1372775
Diff against target: 12 lines (+1/-1)
1 file modified
cmd/ciborium/main.go (+1/-1)
To merge this branch: bzr merge lp:~sergiusens/ciborium/minor_check_loop_fix
Reviewer Review Type Date Requested Status
Ricardo Salveti (community) Approve
Review via email: mp+235882@code.launchpad.net

Commit message

Make mountpoint slice capacity mapLen but len 0.

To post a comment you must log in.
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmd/ciborium/main.go'
2--- cmd/ciborium/main.go 2014-09-17 18:41:46 +0000
3+++ cmd/ciborium/main.go 2014-09-25 00:14:51 +0000
4@@ -60,7 +60,7 @@
5 defer m.lock.Unlock()
6
7 mapLen := len(m.mountpoints)
8- mountpoints := make([]mountpoint, mapLen, mapLen)
9+ mountpoints := make([]mountpoint, 0, mapLen)
10 for p := range m.mountpoints {
11 mountpoints = append(mountpoints, p)
12 }

Subscribers

People subscribed via source and target branches