diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-05-25 13:29:38 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-05-25 13:29:38 +0300 |
commit | 1390964b68b4b35d7049401c484e71faf17c15c8 (patch) | |
tree | 60271512e5d7c3d9981ecbe39d1ab240a3951c38 | |
parent | a016ce8a04da3df46cbfdd1efaae719853efcf1d (diff) |
gitaly-lfs-smudge: Remove unused parameterpks-lfs-smudge-refactor
Remove the unused `to io.Writer` parameter from `handleSmudge()`. We
alreay return an `io.ReadCloser` that the contents can be read from, so
there is no need to receive a writer.
-rw-r--r-- | cmd/gitaly-lfs-smudge/lfs_smudge.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cmd/gitaly-lfs-smudge/lfs_smudge.go b/cmd/gitaly-lfs-smudge/lfs_smudge.go index 74852bf3a..99b79175a 100644 --- a/cmd/gitaly-lfs-smudge/lfs_smudge.go +++ b/cmd/gitaly-lfs-smudge/lfs_smudge.go @@ -21,7 +21,7 @@ func smudgeContents(cfg smudge.Config, to io.Writer, from io.Reader) (returnedEr ctx, finished := tracing.ExtractFromEnv(context.Background()) defer finished() - output, err := handleSmudge(ctx, cfg, to, from) + output, err := handleSmudge(ctx, cfg, from) if err != nil { return fmt.Errorf("smudging contents: %w", err) } @@ -38,7 +38,7 @@ func smudgeContents(cfg smudge.Config, to io.Writer, from io.Reader) (returnedEr return nil } -func handleSmudge(ctx context.Context, cfg smudge.Config, to io.Writer, from io.Reader) (io.ReadCloser, error) { +func handleSmudge(ctx context.Context, cfg smudge.Config, from io.Reader) (io.ReadCloser, error) { logger := log.ContextLogger(ctx) ptr, contents, err := lfs.DecodeFrom(from) |