Minor code cleanup for caching in check_markup()

The code in check_markup() defines a $cache variable (boolean) but then only uses it once. While working on another issue, I realized that the code would be simpler and more readable if it simply didn't bother defining that as a separate variable all. So I've moved that to a separate issue here.

Patch is attached.