remove unnecessary comment_count_notify_settings_save

I like to link to the commit message when I make a comment saying I've fixed something - http://drupalcode.org/sandbox/arijit/1127432.git/commit/110bb1a - that helps others to review the change and in this case you didn't remove all the code.

See
97 $form['#submit'] = array('comment_count_notify_settings_save');

The system_settings_form will add its own submit handler that saves form values into a variable based on the name of the form element.

$form['interval'] = array( is saved to a variable called "interval" (not Interval).

I suggest removing the custom submit function, and renaming all of your form keys and the corresponding variables so they are unique to this module. The Drupal best practice is to prefix all function names and variable names with the module name. So, interval should be comment_count_notify_interval.

If you do that change and then create a fresh test site and fresh install of the module, save the settings form and look in the variables table in the database. There you can find the names and values of all variables on your site.

Looks great!

I'm still really concerned about the variable names and function names: interval, email, subject, getAdminMail(). These are all very likely to cause conflicts.

Pardon me, I see that topic is discussed in #1136970: Better variable and function names .

Note that when you fix something the best Status is "fixed." This leaves it visible in the queue for two weeks at which point it gets automatically closed by an automated process.

Leaving it in the queue makes it visible to people who may be looking for the same issue and helps prevent duplicate issues.