Does anything speak against just throwing directly form checkHeader?
What I envisioned is checkHeader (bool do_not_throw) and then we call it with checkHeader ((flags & OpenFlags::Force) != OpenFlags::Force). And in the function, if do_not_throw is false and we encounter an error we throw.
What I envisioned is
checkHeader (bool do_not_throw)
and then we call it withcheckHeader ((flags & OpenFlags::Force) != OpenFlags::Force)
. And in the function, ifdo_not_throw
is false and we encounter an error we throw.