Re: [PATCH v1 5/7] mv: remove BOTH from enum update_mode
- From
Derrick Stolee <derrickstolee@github.com>
- Date
- Jul 19, 2022, 18:00 UTC
- Message-ID
- <4996a896-cf92-7fc7-40d3-dc3e74e804a4@github.com>
- In-Reply-To
- <20220719132809.409247-6-shaoxuan.yuan02@gmail.com>
On 7/19/2022 9:28 AM, Shaoxuan Yuan wrote:
> Since BOTH is not practically used anywhere in the code and its meaning > is unclear, remove it.
It's literally not used any where. (I would just drop "practically".)
Also, now that enum update_mode is a flag-based enum, the 0 value cannot have any meaning to it except "none of the other flags are on". Thanks, -Stolee