LGTM and I just have a wording nit that was already there before your patch. When reading it I really wondered why we need to "pollute" the flags.

Wouldn't a function that performs a "backup" make sense, analogous to ..._restore? And then rename the pollute function to something like enable_everything or enable_all_options?

That would make things clearer IMHO.  Can easily be a follow-up of course.

--
Regards
Robin

Reply via email to