On 30/09/2019 16:36, Jes Sorensen wrote: > [...] > Applied thanks! > > I fixed up one minor nit rather than having to do the merry-go-round by > email one more time: > >> diff --git a/Monitor.c b/Monitor.c >> index 036103f..cf0610b 100644 >> --- a/Monitor.c >> +++ b/Monitor.c > [snip] > >> @@ -1116,7 +1119,8 @@ int WaitClean(char *dev, int verbose) >> rv = read(state_fd, buf, sizeof(buf)); >> if (rv < 0) >> break; >> - if (sysfs_match_word(buf, clean_states) <= 4) >> + if (sysfs_match_word(buf, clean_states) >> + < (int)ARRAY_SIZE(clean_states)-1) > > I moved the < up to the correct line where it belongs, and added spaces > ") - 1)" > > Cheers, > Jes Thanks a lot Jes, much appreciated! Cheers, Guilherme -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel