Adding NewAutoProgress so total can be set. Made Update() thread safe#21
Adding NewAutoProgress so total can be set. Made Update() thread safe#21
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the AutoProgress functionality by adding thread-safe progress updates and explicit return values. The key changes introduce a constructor function for setting the total value upfront and make the Update() method safe for concurrent use.
- Added
NewAutoProgress()constructor to initialize AutoProgress with a total value - Made
Update()method thread-safe using mutex locks - Replaced naked returns with explicit return statements across multiple functions
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.