summaryrefslogtreecommitdiffstats
tag namev2.17.1 (7bf8fe6cc9bb79f56edc145c60e2a4c94255a536)
tag date2022-03-20 09:43:07 +0800
tagged bySebastian Thiel <sebastian.thiel@icloud.com>
tagged objectcommit 48c446294e...
### Improvements to aggregate progress reporting
Previously, aggregate mode progress reports were handled by an infinitely-looping thread carrying a 64-bit atomic of the current count, which it would print periodically. This resulted in #99 - breaking on platforms without 64-bit atomics, for which a feature was added to disable it. It also implied a race condition, where the "Enumerating ..." message could be printed after results had been gathered but before dua exited. Additionally, part of the status message could be left on the display if the first line of a report was too short to cover it. This commit should resolve these: * The 64-bit atomic counter is replaced with an 8-bit AtomicBool * All printing is controlled from the main thread * The first line is cleared prior to printing a report The only notable drawback I see with this approach is that progress reporting can sometimes be delayed, since the display is only evaluated for update during periods the aggregation loop makes progress. The practical difference appears relatively minor. Since this should resolve #99, the aggregate-scan-progress feature is removed. Special thanks to [@Freaky](https://github.com/Freaky) for the contribution! ### BREAKING change for package maintainers The `aggregate-scan-progress` feature was removed as it shouldn't be required anymore. ### Commit Statistics - 8 commits contributed to the release over the course of 55 calendar days. - 57 days passed between releases. - 0 commits where understood as [conventional](https://www.conventionalcommits.org). - 0 issues like '(#ID)' where seen in commit messages ### Commit Details * **Uncategorized** - prepare changelog (fc1e10a) - Improve aggregate progress reporting (7d83f96) - update dependencies (9a1da6b) - upgrade sysinfo (0b6b52f) - Adjust to changes in clap (f9df024) - dependency update (0d9fbd3) - Update clap to official release (b029dc5) - Upgrade to TUI 0.17 (9ce96ac)