Download & Extend
Poormanscron » Issues

Remove useless code

Posted by kiamlaluno on January 30, 2009 at 3:46pm
Project:Poormanscron
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

In poormanscron_exit() $message is initialized to NULL but it then immediately set to a string, basing on the result returned by drupal_run_cron().
Initializing the variable to NULL is useless.

Login or register to post comments

Comments

#1

Posted by Dave Reid on April 20, 2009 at 5:28pm
Status:active» needs review

Patch attached for review.

AttachmentSizeStatusTest resultOperations
366757-poormanscron-D6.patch633 bytesIgnored: Check issue status.NoneNone

#2

Posted by kiamlaluno on April 20, 2009 at 6:02pm

The patch seems to work.

#3

Posted by Rob Loach on May 6, 2009 at 5:07am
Status:needs review» fixed

Committed, thanks! http://drupal.org/cvs?commit=207516

#4

Posted by System Message on May 20, 2009 at 5:10am
Status:fixed» closed (fixed)

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