Messages in this thread | | | Subject | Re: [PATCH 2/6] statfs: use << to align with fs header | From | Randy Dunlap <> | Date | Fri, 13 Apr 2018 10:55:23 -0700 |
| |
On 04/13/2018 10:35 AM, Andreas Dilger wrote: > On Apr 13, 2018, at 10:11 AM, Christian Brauner <christian.brauner@ubuntu.com> wrote: >> >> Consistenly use << to define ST_* constants. This also aligns them with >> their MS_* counterparts in fs.h > > IMHO, using (1 << 10) makes the code harder to debug. If you see a field > in a structure like 0x8354, it is non-trivial to map this to the ST_* > flags if they are declared in the form (1 << 10) or BIT(10). If they are > declared in the form 0x100 (as they are now) then it is trivial that the > ST_APPEND flag is set in 0x8354, and easy to understand the other flags. > > So, my preference would be to NOT land this or the previous patch.
That makes sense to me.
-- ~Randy
| |