this line is intended to check that the caller is holding the appropriate locks. I have no hard proof that it is running in our release builds, but when I was doing local performance profiling on a non-debug build it was showing up. It doesn't hurt legibility too much to inline it.