In #805702: Cron should use locking framework the cron_semaphore using a variable was removed in favor of an implementation using the new lock api.

The function drupal_cron_cleanup and its registration using drupal_register_shutdown_function was never removed though. This is dead code now though. I'm providing a batch that cleans this up.

CommentFileSizeAuthor
#2 remove_cron_cleanup_1.patch1.23 KBdavyvdb
remove_cron_cleanup.patch1.02 KBdavyvdb

Comments

bojanz’s picture

bojanz commented
Status: Needs review » Needs work

Please roll a unified patch (diff -up).
Also, not sure if that contact.module hunk is supposed to be there.

davyvdb’s picture

davyvdb commented
Status: Needs work » Needs review
StatusFileSize
new remove_cron_cleanup_1.patch1.23 KB

Here you go.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Closing as duplicate of #1775488: drupal_cron_cleanup is not converted to lock.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Closed (duplicate)