linting: Consider pre-allocating sliceVar (prealloc)

thaJeztah · thaJeztah · commit d0dee3cebe71 · 2023-11-20T16:02:16.000+01:00
While updating, also addressed some redundant fmt.Sprintf()

    opts/throttledevice.go:86:2: Consider pre-allocating `out` (prealloc)
        var out []string
        ^
    opts/ulimit.go:37:2: Consider pre-allocating `out` (prealloc)
        var out []string
        ^
    opts/ulimit.go:47:2: Consider pre-allocating `ulimits` (prealloc)
        var ulimits []*units.Ulimit
        ^
    opts/weightdevice.go:68:2: Consider pre-allocating `out` (prealloc)
        var out []string
        ^
    cli/context/store/metadatastore.go:96:2: Consider pre-allocating `res` (prealloc)
        var res []Metadata
        ^
    cli/context/store/store.go:127:2: Consider pre-allocating `names` (prealloc)
        var names []string
        ^
    cli/compose/loader/loader.go:223:2: Consider pre-allocating `keys` (prealloc)
        var keys []string
        ^
    cli/compose/loader/loader.go:397:2: Consider pre-allocating `services` (prealloc)
        var services []types.ServiceConfig
        ^
    cli/command/stack/loader/loader.go:63:2: Consider pre-allocating `msgs` (prealloc)
        var msgs []string
        ^
    cli/command/stack/loader/loader.go:118:2: Consider pre-allocating `configFiles` (prealloc)
        var configFiles []composetypes.ConfigFile
        ^
    cli/command/formatter/container.go:245:2: Consider pre-allocating `joinLabels` (prealloc)
        var joinLabels []string
        ^
    cli/command/formatter/container.go:265:2: Consider pre-allocating `mounts` (prealloc)
        var mounts []string
        ^
    cli/command/formatter/container.go:316:2: Consider pre-allocating `result` (prealloc)
        var result []string
        ^
    cli/command/formatter/displayutils.go:43:2: Consider pre-allocating `display` (prealloc)
        var (
        ^
    cli/command/formatter/volume.go:103:2: Consider pre-allocating `joinLabels` (prealloc)
        var joinLabels []string
        ^
    cli-plugins/manager/manager_test.go:49:2: Consider pre-allocating `dirs` (prealloc)
        var dirs []string
        ^
    cli/command/swarm/init.go:69:2: Consider pre-allocating `defaultAddrPool` (prealloc)
        var defaultAddrPool []string
        ^
    cli/command/manifest/push.go:195:2: Consider pre-allocating `blobReqs` (prealloc)
        var blobReqs []manifestBlob
        ^
    cli/command/secret/formatter.go:111:2: Consider pre-allocating `joinLabels` (prealloc)
        var joinLabels []string
        ^
    cli/command/network/formatter.go:104:2: Consider pre-allocating `joinLabels` (prealloc)
        var joinLabels []string
        ^
    cli/command/context/list.go:52:2: Consider pre-allocating `contexts` (prealloc)
        var contexts []*formatter.ClientContext
        ^
    cli/command/config/formatter.go:104:2: Consider pre-allocating `joinLabels` (prealloc)
        var joinLabels []string
        ^
    cli/command/trust/common_test.go:23:2: Consider pre-allocating `targetNames` (prealloc)
        var targetNames []string
        ^
    cli/command/service/generic_resource_opts.go:55:2: Consider pre-allocating `generic` (prealloc)
        var generic []swarm.GenericResource
        ^
    cli/command/service/generic_resource_opts.go:98:2: Consider pre-allocating `l` (prealloc)
        var l []swarm.GenericResource
        ^
    cli/command/service/opts.go:378:2: Consider pre-allocating `netAttach` (prealloc)
        var netAttach []swarm.NetworkAttachmentConfig
        ^
    cli/command/service/update.go:731:2: Consider pre-allocating `limits` (prealloc)
        var limits []*units.Ulimit
        ^
    cli/command/service/update.go:1315:2: Consider pre-allocating `newNetworks` (prealloc)
        var newNetworks []swarm.NetworkAttachmentConfig
        ^
    cli/command/service/update.go:1514:2: Consider pre-allocating `out` (prealloc)
        var out []string
        ^
    cli/compose/convert/service.go:713:2: Consider pre-allocating `ulimits` (prealloc)
        var ulimits []*units.Ulimit
        ^
    cli/compose/convert/volume.go:13:2: Consider pre-allocating `mounts` (prealloc)
        var mounts []mount.Mount
        ^
    cli/command/stack/swarm/list.go:39:2: Consider pre-allocating `stacks` (prealloc)
        var stacks []*formatter.Stack
        ^

Signed-off-by: Sebastiaan van Stijn &lt;github@gone.nl&gt;
diff --git a/cli-plugins/manager/manager_test.go b/cli-plugins/manager/manager_test.go
@@ -46,7 +46,7 @@ func TestListPluginCandidates(t *testing.T) {
 	)
 	defer dir.Remove()
 
-	var dirs []string
+	dirs := make([]string, 0, 6)
 	for _, d := range []string{"plugins1", "nonexistent", "plugins2", "plugins3", "plugins4", "plugins5"} {
 		dirs = append(dirs, dir.Join(d))
 	}
diff --git a/cli/command/config/formatter.go b/cli/command/config/formatter.go
@@ -101,7 +101,7 @@ func (c *configContext) Labels() string {
 	if mapLabels == nil {
 		return ""
 	}
-	var joinLabels []string
+	joinLabels := make([]string, 0, len(mapLabels))
 	for k, v := range mapLabels {
 		joinLabels = append(joinLabels, k+"="+v)
 	}
diff --git a/cli/command/context/list.go b/cli/command/context/list.go
@@ -51,7 +51,7 @@ func runList(dockerCli command.Cli, opts *listOptions) error {
 	var (
 		curContext = dockerCli.CurrentContext()
 		curFound   bool
-		contexts   []*formatter.ClientContext
+		contexts   = make([]*formatter.ClientContext, 0, len(contextMap))
 	)
 	for _, rawMeta := range contextMap {
 		isCurrent := rawMeta.Name == curContext
diff --git a/cli/command/formatter/container.go b/cli/command/formatter/container.go
@@ -245,7 +245,7 @@ func (c *ContainerContext) Labels() string {
 		return ""
 	}
 
-	var joinLabels []string
+	joinLabels := make([]string, 0, len(c.c.Labels))
 	for k, v := range c.c.Labels {
 		joinLabels = append(joinLabels, k+"="+v)
 	}
@@ -265,7 +265,7 @@ func (c *ContainerContext) Label(name string) string {
 // If the trunc option is set, names can be truncated (ellipsized).
 func (c *ContainerContext) Mounts() string {
 	var name string
-	var mounts []string
+	mounts := make([]string, 0, len(c.c.Mounts))
 	for _, m := range c.c.Mounts {
 		if m.Name == "" {
 			name = m.Source
@@ -289,7 +289,7 @@ func (c *ContainerContext) LocalVolumes() string {
 		}
 	}
 
-	return fmt.Sprintf("%d", count)
+	return strconv.Itoa(count)
 }
 
 // Networks returns a comma-separated string of networks that the container is
@@ -299,7 +299,7 @@ func (c *ContainerContext) Networks() string {
 		return ""
 	}
 
-	networks := []string{}
+	networks := make([]string, 0, len(c.c.NetworkSettings.Networks))
 	for k := range c.c.NetworkSettings.Networks {
 		networks = append(networks, k)
 	}
@@ -316,7 +316,7 @@ func DisplayablePorts(ports []types.Port) string {
 		last  uint16
 	}
 	groupMap := make(map[string]*portGroup)
-	var result []string
+	var result []string //nolint:prealloc
 	var hostMappings []string
 	var groupMapKeys []string
 	sort.Slice(ports, func(i, j int) bool {
@@ -331,7 +331,7 @@ func DisplayablePorts(ports []types.Port) string {
 				hostMappings = append(hostMappings, fmt.Sprintf("%s:%d->%d/%s", port.IP, port.PublicPort, port.PrivatePort, port.Type))
 				continue
 			}
-			portKey = fmt.Sprintf("%s/%s", port.IP, port.Type)
+			portKey = port.IP + "/" + port.Type
 		}
 		group := groupMap[portKey]
 
@@ -372,7 +372,7 @@ func formGroup(key string, start, last uint16) string {
 	if ip != "" {
 		group = fmt.Sprintf("%s:%s->%s", ip, group, group)
 	}
-	return fmt.Sprintf("%s/%s", group, groupType)
+	return group + "/" + groupType
 }
 
 func comparePorts(i, j types.Port) bool {
diff --git a/cli/command/formatter/displayutils.go b/cli/command/formatter/displayutils.go
@@ -41,7 +41,7 @@ func Ellipsis(s string, maxDisplayWidth int) string {
 	}
 
 	var (
-		display      []int
+		display      = make([]int, 0, len(rs))
 		displayWidth int
 	)
 	for _, r := range rs {
diff --git a/cli/command/formatter/volume.go b/cli/command/formatter/volume.go
@@ -100,7 +100,7 @@ func (c *volumeContext) Labels() string {
 		return ""
 	}
 
-	var joinLabels []string
+	joinLabels := make([]string, 0, len(c.v.Labels))
 	for k, v := range c.v.Labels {
 		joinLabels = append(joinLabels, k+"="+v)
 	}
diff --git a/cli/command/manifest/push.go b/cli/command/manifest/push.go
@@ -192,9 +192,9 @@ func buildManifestDescriptor(targetRepo *registry.RepositoryInfo, imageManifest
 }
 
 func buildBlobRequestList(imageManifest types.ImageManifest, repoName reference.Named) ([]manifestBlob, error) {
-	var blobReqs []manifestBlob
-
-	for _, blobDigest := range imageManifest.Blobs() {
+	blobs := imageManifest.Blobs()
+	blobReqs := make([]manifestBlob, 0, len(blobs))
+	for _, blobDigest := range blobs {
 		canonical, err := reference.WithDigest(repoName, blobDigest)
 		if err != nil {
 			return nil, err
diff --git a/cli/command/network/formatter.go b/cli/command/network/formatter.go
@@ -101,7 +101,7 @@ func (c *networkContext) Labels() string {
 		return ""
 	}
 
-	var joinLabels []string
+	joinLabels := make([]string, 0, len(c.n.Labels))
 	for k, v := range c.n.Labels {
 		joinLabels = append(joinLabels, k+"="+v)
 	}
diff --git a/cli/command/secret/formatter.go b/cli/command/secret/formatter.go
@@ -108,7 +108,7 @@ func (c *secretContext) Labels() string {
 	if mapLabels == nil {
 		return ""
 	}
-	var joinLabels []string
+	joinLabels := make([]string, 0, len(mapLabels))
 	for k, v := range mapLabels {
 		joinLabels = append(joinLabels, k+"="+v)
 	}
diff --git a/cli/command/service/generic_resource_opts.go b/cli/command/service/generic_resource_opts.go
@@ -52,7 +52,7 @@ func ParseGenericResources(value []string) ([]swarm.GenericResource, error) {
 
 // genericResourcesFromGRPC converts a GRPC GenericResource to a GenericResource
 func genericResourcesFromGRPC(genericRes []*swarmapi.GenericResource) []swarm.GenericResource {
-	var generic []swarm.GenericResource
+	generic := make([]swarm.GenericResource, 0, len(genericRes))
 	for _, res := range genericRes {
 		var current swarm.GenericResource
 
@@ -95,7 +95,7 @@ func buildGenericResourceMap(genericRes []swarm.GenericResource) (map[string]swa
 }
 
 func buildGenericResourceList(genericRes map[string]swarm.GenericResource) []swarm.GenericResource {
-	var l []swarm.GenericResource
+	l := make([]swarm.GenericResource, 0, len(genericRes))
 
 	for _, res := range genericRes {
 		l = append(l, res)
diff --git a/cli/command/service/opts.go b/cli/command/service/opts.go
@@ -378,8 +378,9 @@ func resolveNetworkID(ctx context.Context, apiClient client.NetworkAPIClient, ne
 }
 
 func convertNetworks(networks opts.NetworkOpt) []swarm.NetworkAttachmentConfig {
-	var netAttach []swarm.NetworkAttachmentConfig
-	for _, net := range networks.Value() {
+	nws := networks.Value()
+	netAttach := make([]swarm.NetworkAttachmentConfig, 0, len(nws))
+	for _, net := range nws {
 		netAttach = append(netAttach, swarm.NetworkAttachmentConfig{
 			Target:     net.Target,
 			Aliases:    net.Aliases,
diff --git a/cli/command/service/update.go b/cli/command/service/update.go
@@ -727,8 +727,10 @@ func updateUlimits(flags *pflag.FlagSet, ulimits []*units.Ulimit) []*units.Ulimi
 			newUlimits[ulimit.Name] = ulimit
 		}
 	}
-
-	var limits []*units.Ulimit
+	if len(newUlimits) == 0 {
+		return nil
+	}
+	limits := make([]*units.Ulimit, 0, len(newUlimits))
 	for _, ulimit := range newUlimits {
 		limits = append(limits, ulimit)
 	}
@@ -1307,7 +1309,7 @@ func updateNetworks(ctx context.Context, apiClient client.NetworkAPIClient, flag
 	}
 
 	existingNetworks := make(map[string]struct{})
-	var newNetworks []swarm.NetworkAttachmentConfig
+	var newNetworks []swarm.NetworkAttachmentConfig //nolint:prealloc
 	for _, network := range specNetworks {
 		if _, exists := idsToRemove[network.Target]; exists {
 			continue
@@ -1503,10 +1505,13 @@ func updateCapabilities(flags *pflag.FlagSet, containerSpec *swarm.ContainerSpec
 }
 
 func capsList(caps map[string]bool) []string {
+	if len(caps) == 0 {
+		return nil
+	}
 	if caps[opts.AllCapabilities] {
 		return []string{opts.AllCapabilities}
 	}
-	var out []string
+	out := make([]string, 0, len(caps))
 	for c := range caps {
 		out = append(out, c)
 	}
diff --git a/cli/command/stack/loader/loader.go b/cli/command/stack/loader/loader.go
@@ -61,7 +61,7 @@ func getDictsFrom(configFiles []composetypes.ConfigFile) []map[string]interface{
 }
 
 func propertyWarnings(properties map[string]string) string {
-	var msgs []string
+	msgs := make([]string, 0, len(properties))
 	for name, description := range properties {
 		msgs = append(msgs, fmt.Sprintf("%s: %s", name, description))
 	}
@@ -129,7 +129,7 @@ func buildEnvironment(env []string) (map[string]string, error) {
 }
 
 func loadConfigFiles(filenames []string, stdin io.Reader) ([]composetypes.ConfigFile, error) {
-	var configFiles []composetypes.ConfigFile
+	configFiles := make([]composetypes.ConfigFile, 0, len(filenames))
 
 	for _, filename := range filenames {
 		configFile, err := loadConfigFile(filename, stdin)
diff --git a/cli/command/stack/swarm/list.go b/cli/command/stack/swarm/list.go
@@ -36,7 +36,7 @@ func GetStacks(dockerCli command.Cli) ([]*formatter.Stack, error) {
 			ztack.Services++
 		}
 	}
-	var stacks []*formatter.Stack
+	stacks := make([]*formatter.Stack, 0, len(m))
 	for _, stack := range m {
 		stacks = append(stacks, stack)
 	}
diff --git a/cli/command/swarm/init.go b/cli/command/swarm/init.go
@@ -66,11 +66,10 @@ func newInitCommand(dockerCli command.Cli) *cobra.Command {
 }
 
 func runInit(dockerCli command.Cli, flags *pflag.FlagSet, opts initOptions) error {
-	var defaultAddrPool []string
-
 	client := dockerCli.Client()
 	ctx := context.Background()
 
+	defaultAddrPool := make([]string, 0, len(opts.defaultAddrPools))
 	for _, p := range opts.defaultAddrPools {
 		defaultAddrPool = append(defaultAddrPool, p.String())
 	}
diff --git a/cli/command/system/events_test.go b/cli/command/system/events_test.go
@@ -16,7 +16,7 @@ import (
 )
 
 func TestEventsFormat(t *testing.T) {
-	var evts []events.Message
+	var evts []events.Message //nolint:prealloc
 	for i, action := range []events.Action{events.ActionCreate, events.ActionStart, events.ActionAttach, events.ActionDie} {
 		evts = append(evts, events.Message{
 			Status: string(action),
diff --git a/cli/command/trust/common_test.go b/cli/command/trust/common_test.go
@@ -20,7 +20,7 @@ func TestMatchReleasedSignaturesSortOrder(t *testing.T) {
 
 	rows := matchReleasedSignatures(targets)
 
-	var targetNames []string
+	targetNames := make([]string, 0, len(rows))
 	for _, r := range rows {
 		targetNames = append(targetNames, r.SignedTag)
 	}
diff --git a/cli/compose/convert/service.go b/cli/compose/convert/service.go
@@ -710,7 +710,7 @@ func convertUlimits(origUlimits map[string]*composetypes.UlimitsConfig) []*units
 			}
 		}
 	}
-	var ulimits []*units.Ulimit
+	ulimits := make([]*units.Ulimit, 0, len(newUlimits))
 	for _, ulimit := range newUlimits {
 		ulimits = append(ulimits, ulimit)
 	}
diff --git a/cli/compose/convert/volume.go b/cli/compose/convert/volume.go
@@ -12,14 +12,13 @@ type volumes map[string]composetypes.VolumeConfig
 
 // Volumes from compose-file types to engine api types
 func Volumes(serviceVolumes []composetypes.ServiceVolumeConfig, stackVolumes volumes, namespace Namespace) ([]mount.Mount, error) {
-	var mounts []mount.Mount
-
+	mounts := make([]mount.Mount, 0, len(serviceVolumes))
 	for _, volumeConfig := range serviceVolumes {
-		mount, err := convertVolumeToMount(volumeConfig, stackVolumes, namespace)
+		mnt, err := convertVolumeToMount(volumeConfig, stackVolumes, namespace)
 		if err != nil {
 			return nil, err
 		}
-		mounts = append(mounts, mount)
+		mounts = append(mounts, mnt)
 	}
 	return mounts, nil
 }
diff --git a/cli/compose/loader/loader.go b/cli/compose/loader/loader.go
@@ -220,7 +220,7 @@ func GetUnsupportedProperties(configDicts ...map[string]interface{}) []string {
 }
 
 func sortedKeys(set map[string]bool) []string {
-	var keys []string
+	keys := make([]string, 0, len(set))
 	for key := range set {
 		keys = append(keys, key)
 	}
@@ -394,7 +394,7 @@ func formatInvalidKeyError(keyPrefix string, key interface{}) error {
 // LoadServices produces a ServiceConfig map from a compose file Dict
 // the servicesDict is not validated if directly used. Use Load() to enable validation
 func LoadServices(servicesDict map[string]interface{}, workingDir string, lookupEnv template.Mapping) ([]types.ServiceConfig, error) {
-	var services []types.ServiceConfig
+	services := make([]types.ServiceConfig, 0, len(servicesDict))
 
 	for name, serviceDef := range servicesDict {
 		serviceConfig, err := LoadService(name, serviceDef.(map[string]interface{}), workingDir, lookupEnv)
diff --git a/cli/context/store/metadatastore.go b/cli/context/store/metadatastore.go
@@ -109,7 +109,7 @@ func (s *metadataStore) list() ([]Metadata, error) {
 		}
 		return nil, err
 	}
-	var res []Metadata
+	res := make([]Metadata, 0, len(ctxDirs))
 	for _, dir := range ctxDirs {
 		c, err := s.getByID(contextdir(dir))
 		if err != nil {
diff --git a/cli/context/store/store.go b/cli/context/store/store.go
@@ -125,7 +125,7 @@ func Names(s Lister) ([]string, error) {
 	if err != nil {
 		return nil, err
 	}
-	var names []string
+	names := make([]string, 0, len(list))
 	for _, item := range list {
 		names = append(names, item.Name)
 	}
diff --git a/cli/manifest/types/types.go b/cli/manifest/types/types.go
@@ -55,14 +55,18 @@ func PlatformSpecFromOCI(p *ocispec.Platform) *manifestlist.PlatformSpec {
 
 // Blobs returns the digests for all the blobs referenced by this manifest
 func (i ImageManifest) Blobs() []digest.Digest {
-	digests := []digest.Digest{}
+	var digests []digest.Digest
 	switch {
 	case i.SchemaV2Manifest != nil:
-		for _, descriptor := range i.SchemaV2Manifest.References() {
+		refs := i.SchemaV2Manifest.References()
+		digests = make([]digest.Digest, 0, len(refs))
+		for _, descriptor := range refs {
 			digests = append(digests, descriptor.Digest)
 		}
 	case i.OCIManifest != nil:
-		for _, descriptor := range i.OCIManifest.References() {
+		refs := i.OCIManifest.References()
+		digests = make([]digest.Digest, 0, len(refs))
+		for _, descriptor := range refs {
 			digests = append(digests, descriptor.Digest)
 		}
 	}
diff --git a/opts/throttledevice.go b/opts/throttledevice.go
@@ -62,9 +62,8 @@ type ThrottledeviceOpt struct {
 
 // NewThrottledeviceOpt creates a new ThrottledeviceOpt
 func NewThrottledeviceOpt(validator ValidatorThrottleFctType) ThrottledeviceOpt {
-	values := []*blkiodev.ThrottleDevice{}
 	return ThrottledeviceOpt{
-		values:    values,
+		values:    []*blkiodev.ThrottleDevice{},
 		validator: validator,
 	}
 }
@@ -85,7 +84,7 @@ func (opt *ThrottledeviceOpt) Set(val string) error {
 
 // String returns ThrottledeviceOpt values as a string.
 func (opt *ThrottledeviceOpt) String() string {
-	var out []string
+	out := make([]string, 0, len(opt.values))
 	for _, v := range opt.values {
 		out = append(out, v.String())
 	}
@@ -95,7 +94,9 @@ func (opt *ThrottledeviceOpt) String() string {
 
 // GetList returns a slice of pointers to ThrottleDevices.
 func (opt *ThrottledeviceOpt) GetList() []*blkiodev.ThrottleDevice {
-	return append([]*blkiodev.ThrottleDevice{}, opt.values...)
+	out := make([]*blkiodev.ThrottleDevice, 0, len(opt.values))
+	copy(out, opt.values)
+	return out
 }
 
 // Type returns the option type
diff --git a/opts/ulimit.go b/opts/ulimit.go
@@ -34,7 +34,7 @@ func (o *UlimitOpt) Set(val string) error {
 
 // String returns Ulimit values as a string. Values are sorted by name.
 func (o *UlimitOpt) String() string {
-	var out []string
+	out := make([]string, 0, len(*o.values))
 	for _, v := range *o.values {
 		out = append(out, v.String())
 	}
@@ -44,7 +44,7 @@ func (o *UlimitOpt) String() string {
 
 // GetList returns a slice of pointers to Ulimits. Values are sorted by name.
 func (o *UlimitOpt) GetList() []*units.Ulimit {
-	var ulimits []*units.Ulimit
+	ulimits := make([]*units.Ulimit, 0, len(*o.values))
 	for _, v := range *o.values {
 		ulimits = append(ulimits, v)
 	}
diff --git a/opts/weightdevice.go b/opts/weightdevice.go

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ func TestListPluginCandidates(t *testing.T) {`
`46`	`46`	`)`
`47`	`47`	`defer dir.Remove()`
`48`	`48`
`49`		`- var dirs []string`
	`49`	`+ dirs := make([]string, 0, 6)`
`50`	`50`	`for _, d := range []string{"plugins1", "nonexistent", "plugins2", "plugins3", "plugins4", "plugins5"} {`
`51`	`51`	`dirs = append(dirs, dir.Join(d))`
`52`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ func (c *configContext) Labels() string {`
`101`	`101`	`if mapLabels == nil {`
`102`	`102`	`return ""`
`103`	`103`	`}`
`104`		`- var joinLabels []string`
	`104`	`+ joinLabels := make([]string, 0, len(mapLabels))`
`105`	`105`	`for k, v := range mapLabels {`
`106`	`106`	`joinLabels = append(joinLabels, k+"="+v)`
`107`	`107`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ func runList(dockerCli command.Cli, opts *listOptions) error {`
`51`	`51`	`var (`
`52`	`52`	`curContext = dockerCli.CurrentContext()`
`53`	`53`	`curFound bool`
`54`		`- contexts []*formatter.ClientContext`
	`54`	`+ contexts = make([]*formatter.ClientContext, 0, len(contextMap))`
`55`	`55`	`)`
`56`	`56`	`for _, rawMeta := range contextMap {`
`57`	`57`	`isCurrent := rawMeta.Name == curContext`
Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,7 @@ func (c *ContainerContext) Labels() string {`
`245`	`245`	`return ""`
`246`	`246`	`}`
`247`	`247`
`248`		`- var joinLabels []string`
	`248`	`+ joinLabels := make([]string, 0, len(c.c.Labels))`
`249`	`249`	`for k, v := range c.c.Labels {`
`250`	`250`	`joinLabels = append(joinLabels, k+"="+v)`
`251`	`251`	`}`
`@@ -265,7 +265,7 @@ func (c *ContainerContext) Label(name string) string {`
`265`	`265`	`// If the trunc option is set, names can be truncated (ellipsized).`
`266`	`266`	`func (c *ContainerContext) Mounts() string {`
`267`	`267`	`var name string`
`268`		`- var mounts []string`
	`268`	`+ mounts := make([]string, 0, len(c.c.Mounts))`
`269`	`269`	`for _, m := range c.c.Mounts {`
`270`	`270`	`if m.Name == "" {`
`271`	`271`	`name = m.Source`
`@@ -289,7 +289,7 @@ func (c *ContainerContext) LocalVolumes() string {`
`289`	`289`	`}`
`290`	`290`	`}`
`291`	`291`
`292`		`- return fmt.Sprintf("%d", count)`
	`292`	`+ return strconv.Itoa(count)`
`293`	`293`	`}`
`294`	`294`
`295`	`295`	`// Networks returns a comma-separated string of networks that the container is`
`@@ -299,7 +299,7 @@ func (c *ContainerContext) Networks() string {`
`299`	`299`	`return ""`
`300`	`300`	`}`
`301`	`301`
`302`		`- networks := []string{}`
	`302`	`+ networks := make([]string, 0, len(c.c.NetworkSettings.Networks))`
`303`	`303`	`for k := range c.c.NetworkSettings.Networks {`
`304`	`304`	`networks = append(networks, k)`
`305`	`305`	`}`
`@@ -316,7 +316,7 @@ func DisplayablePorts(ports []types.Port) string {`
`316`	`316`	`last uint16`
`317`	`317`	`}`
`318`	`318`	`groupMap := make(map[string]*portGroup)`
`319`		`- var result []string`
	`319`	`+ var result []string //nolint:prealloc`
`320`	`320`	`var hostMappings []string`
`321`	`321`	`var groupMapKeys []string`
`322`	`322`	`sort.Slice(ports, func(i, j int) bool {`
`@@ -331,7 +331,7 @@ func DisplayablePorts(ports []types.Port) string {`
`331`	`331`	`hostMappings = append(hostMappings, fmt.Sprintf("%s:%d->%d/%s", port.IP, port.PublicPort, port.PrivatePort, port.Type))`
`332`	`332`	`continue`
`333`	`333`	`}`
`334`		`- portKey = fmt.Sprintf("%s/%s", port.IP, port.Type)`
	`334`	`+ portKey = port.IP + "/" + port.Type`
`335`	`335`	`}`
`336`	`336`	`group := groupMap[portKey]`
`337`	`337`
`@@ -372,7 +372,7 @@ func formGroup(key string, start, last uint16) string {`
`372`	`372`	`if ip != "" {`
`373`	`373`	`group = fmt.Sprintf("%s:%s->%s", ip, group, group)`
`374`	`374`	`}`
`375`		`- return fmt.Sprintf("%s/%s", group, groupType)`
	`375`	`+ return group + "/" + groupType`
`376`	`376`	`}`
`377`	`377`
`378`	`378`	`func comparePorts(i, j types.Port) bool {`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ func Ellipsis(s string, maxDisplayWidth int) string {`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`var (`
`44`		`- display []int`
	`44`	`+ display = make([]int, 0, len(rs))`
`45`	`45`	`displayWidth int`
`46`	`46`	`)`
`47`	`47`	`for _, r := range rs {`
Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ func (c *volumeContext) Labels() string {`
`100`	`100`	`return ""`
`101`	`101`	`}`
`102`	`102`
`103`		`- var joinLabels []string`
	`103`	`+ joinLabels := make([]string, 0, len(c.v.Labels))`
`104`	`104`	`for k, v := range c.v.Labels {`
`105`	`105`	`joinLabels = append(joinLabels, k+"="+v)`
`106`	`106`	`}`
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ func (c *networkContext) Labels() string {`
`101`	`101`	`return ""`
`102`	`102`	`}`
`103`	`103`
`104`		`- var joinLabels []string`
	`104`	`+ joinLabels := make([]string, 0, len(c.n.Labels))`
`105`	`105`	`for k, v := range c.n.Labels {`
`106`	`106`	`joinLabels = append(joinLabels, k+"="+v)`
`107`	`107`	`}`
Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ func (c *secretContext) Labels() string {`
`108`	`108`	`if mapLabels == nil {`
`109`	`109`	`return ""`
`110`	`110`	`}`
`111`		`- var joinLabels []string`
	`111`	`+ joinLabels := make([]string, 0, len(mapLabels))`
`112`	`112`	`for k, v := range mapLabels {`
`113`	`113`	`joinLabels = append(joinLabels, k+"="+v)`
`114`	`114`	`}`