summaryrefslogtreecommitdiff
path: root/libpod
diff options
context:
space:
mode:
authorMiloslav Trmač <mitr@redhat.com>2018-07-28 05:29:43 +0200
committerAtomic Bot <atomic-devel@projectatomic.io>2018-08-01 18:22:59 +0000
commitbf0ab88eacf6b390b6fe6bda74cbb38bbc82a1a9 (patch)
tree15c21c65c9260df8b0fcb105e27d7f370d75e766 /libpod
parentdae620066284c4c8c35d73941dfecdbc00a2b1b6 (diff)
downloadpodman-bf0ab88eacf6b390b6fe6bda74cbb38bbc82a1a9.tar.gz
podman-bf0ab88eacf6b390b6fe6bda74cbb38bbc82a1a9.tar.bz2
podman-bf0ab88eacf6b390b6fe6bda74cbb38bbc82a1a9.zip
Use []pullRefPair instead of []*pullRefPair
We are passing the values, don't really need the pointer sharing semantics, and the structures are small enough, and the arrays short enough, that we very likely lose on the indirect accesses more than we save on quicker copying of the slices when extending them. Value semantics is safer anyway. Should not change behavior. Signed-off-by: Miloslav Trmač <mitr@redhat.com> Closes: #1176 Approved by: rhatdan
Diffstat (limited to 'libpod')
-rw-r--r--libpod/image/pull.go14
1 files changed, 7 insertions, 7 deletions
diff --git a/libpod/image/pull.go b/libpod/image/pull.go
index 0ecfbacb8..890df8f50 100644
--- a/libpod/image/pull.go
+++ b/libpod/image/pull.go
@@ -165,7 +165,7 @@ func refNamesFromImageReference(ctx context.Context, srcRef types.ImageReference
}
// refPairsFromImageReference returns a list of pullRefPair for a single ImageReference, depending on the used transport.
-func (ir *Runtime) refPairsFromImageReference(ctx context.Context, srcRef types.ImageReference, imgName string, sc *types.SystemContext) ([]*pullRefPair, error) {
+func (ir *Runtime) refPairsFromImageReference(ctx context.Context, srcRef types.ImageReference, imgName string, sc *types.SystemContext) ([]pullRefPair, error) {
refNames, err := refNamesFromImageReference(ctx, srcRef, imgName, sc)
if err != nil {
return nil, err
@@ -179,7 +179,7 @@ func (ir *Runtime) refPairsFromImageReference(ctx context.Context, srcRef types.
// pulled.
func (i *Image) pullImage(ctx context.Context, writer io.Writer, authfile, signaturePolicyPath string, signingOptions SigningOptions, dockerOptions *DockerRegistryOptions, forceSecure bool) ([]string, error) {
// pullImage copies the image from the source to the destination
- var pullRefPairs []*pullRefPair
+ var pullRefPairs []pullRefPair
sc := GetSystemContext(signaturePolicyPath, authfile, false)
srcRef, err := alltransports.ParseImageName(i.InputName)
if err != nil {
@@ -317,7 +317,7 @@ func refNamesFromPossiblyUnqualifiedName(inputName string) ([]pullRefName, error
// refPairsFromPossiblyUnqualifiedName looks at a decomposed image and determines the possible
// image references to try pulling in combination with the registries.conf file as well
-func (i *Image) refPairsFromPossiblyUnqualifiedName() ([]*pullRefPair, error) {
+func (i *Image) refPairsFromPossiblyUnqualifiedName() ([]pullRefPair, error) {
refNames, err := refNamesFromPossiblyUnqualifiedName(i.InputName)
if err != nil {
return nil, err
@@ -325,16 +325,16 @@ func (i *Image) refPairsFromPossiblyUnqualifiedName() ([]*pullRefPair, error) {
return i.imageruntime.pullRefPairsFromRefNames(refNames)
}
-// pullRefPairsFromNames converts a []pullRefName to []*pullRefPair
-func (ir *Runtime) pullRefPairsFromRefNames(refNames []pullRefName) ([]*pullRefPair, error) {
+// pullRefPairsFromNames converts a []pullRefName to []pullRefPair
+func (ir *Runtime) pullRefPairsFromRefNames(refNames []pullRefName) ([]pullRefPair, error) {
// Here we construct the destination references
- res := make([]*pullRefPair, len(refNames))
+ res := make([]pullRefPair, len(refNames))
for i, rn := range refNames {
destRef, err := is.Transport.ParseStoreReference(ir.store, rn.dstName)
if err != nil {
return nil, errors.Wrapf(err, "error parsing dest reference name %#v", rn.dstName)
}
- res[i] = &pullRefPair{
+ res[i] = pullRefPair{
image: rn.image,
srcRef: rn.srcRef,
dstRef: destRef,