I don't disagree.  The original patch was just that and internal commentary led to this version.  They both have a little bit of unsavoryness, but I'm certainly okay prioritizing the usability aspect.

Actually neither one seems particularly unsavory to me.  However, I've never really liked having boolean flag parameters like this in C++ since there's no explanation at the call site about what they're doing (unless the caller is thoughtful enough to put in a comment)... unlike in python where you can get some documentation by using keyword params.  It seems like a worthwhile tradeoff when it enables you to reuse a large complex function by allowing some minor detail to be tweaked by the caller though.  In this case, though, where we're only talking about a one-line function anyway, and we can get the same level of code commonality by having schedRelBreak() just call schedBreak(), I don't really see any advantage to this current version.

Updated with the function named as suggested.