On 6/23/2022 7:41 AM, Shaoxuan Yuan wrote:

As suggested by Derrick [1],
move the in-line definition of "enum update_mode" to the top
of the file and make it use "flags" mode (each state is a different
bit in the word).

nit: strange line-wrapping here.

Change the flag assignments from '=' (single assignment) to '|='
(additive). Also change flag evaluation from '==' to '&', etc.

Code looks good.

Thanks,
-Stolee