This isn't a huge win (I measured a 13ms improvement), but the fix is so simple I think it's worth doing.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, theme-rebuild-perf-fix.patch, failed testing.

msonnabaum’s picture

Status: Needs work » Needs review
msonnabaum’s picture

chx’s picture

Status: Needs review » Reviewed & tested by the community

Yes this looks worth doing.

Anonymous’s picture

nice find.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 0ac024d and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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