On 20.12.21 01:18, Peter Smith wrote:
For some reason the current HEAD PublicationActions is a struct of boolean representing combinations of the 4 different "publication actions".I felt it is more natural to implement boolean flag combinations using a bitmask instead of a struct of bools. IMO using the bitmask also simplifies assignment and checking of said flags.
I don't see why this is better. It just makes the code longer and adds more punctuation and reduces type safety.
