> There was some discussion recently about converting these related > #defines to enums [0]. We might consider doing that here. > > If you read through that entire thread, you'd see that there were some > disagreements about whether using enums for sets of bits is a good idea > ([1] and [2]), but it is at least something worth considering while we > are on this topic. > > FWIW, I think it is a good idea to use an enum here.
[snip] > [0] > https://public-inbox.org/git/[email protected]/ > [1] > https://public-inbox.org/git/[email protected]/ > [2] https://public-inbox.org/git/[email protected]/ Thanks for the handy references. You know my opinion on bitflags as enums from reading them, but I think that we have already had that discussion and came to a conclusion. So don't use an enum here. The patch itself looks good, and I also prefer the bit shift format over octal.
