actions/typos
1
0
Fork 0
mirror of https://github.com/crate-ci/typos.git synced 2024-12-26 01:22:08 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #286 from epage/crash

Browse source 
fix(cli): Don't crash on races
This commit is contained in:
Ed Page 2021-06-15 17:31:59 -05:00 committed by GitHub
commit 10497270f1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
CHANGELOG.md
View file

@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
<!-- next-header -->
## [Unreleased] - ReleaseDate
#### Bug Fixes
- Fix a crash from hitting a race condition
## [1.0.8] - 2021-06-15
## [1.0.7] - 2021-06-15

21
src/bin/typos-cli/report.rs
View file

@ -59,21 +59,12 @@ impl<'r> MessageStatus<'r> {
impl<'r> Report for MessageStatus<'r> {
fn report(&self, msg: Message) -> Result<(), std::io::Error> {
let _ = self.typos_found.compare_exchange(
false,
msg.is_correction(),
atomic::Ordering::Relaxed,
atomic::Ordering::Relaxed,
);
let _ = self
.errors_found
.compare_exchange(
false,
msg.is_error(),
atomic::Ordering::Relaxed,
atomic::Ordering::Relaxed,
)
.unwrap();
if msg.is_correction() {
self.typos_found.store(true, atomic::Ordering::Relaxed);
}
if msg.is_error() {
self.errors_found.store(true, atomic::Ordering::Relaxed);
}
self.reporter.report(msg)
}
}