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.

CommentFileSizeAuthor
check_markup_cleanup.patch1 KBDavid_Rothstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.