Merge pull request #23 from go-zoox/fix/config-onporogress-need-be-a-…

…func fix: config onporogress need be a func instead of point to func
go-zoox · Aug 20, 2023 · a899cb1 · a899cb1
2 parents f2dc1bb + 629698e
commit a899cb1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/config.go b/config.go
@@ -47,7 +47,7 @@ type Config struct {
 	//
 	Context context.Context
 	//
-	OnProgress *func(percent int64, current, total int64)
+	OnProgress func(percent int64, current, total int64)
 	//
 	BasicAuth BasicAuth
 	//

diff --git a/execute.go b/execute.go
@@ -415,7 +415,7 @@ func (f *Fetch) Execute() (*Response, error) {
 			progress := &Progress{
 				Total:    resp.ContentLength,
 				Current:  0,
-				Reporter: *f.config.OnProgress,
+				Reporter: f.config.OnProgress,
 			}
 
 			_, err = io.Copy(io.MultiWriter(file, progress), reader)

diff --git a/fetch.go b/fetch.go
@@ -74,7 +74,7 @@ func (f *Fetch) SetDownloadFilePath(filepath string) *Fetch {
 
 // SetProgressCallback sets the progress callback
 func (f *Fetch) SetProgressCallback(callback func(percent int64, current, total int64)) *Fetch {
-	f.config.OnProgress = &callback
+	f.config.OnProgress = callback
 	return f
 }