[nagiosplug] git-notify: Drop the $sent_notices variable

 Module: nagiosplug
 Branch: master
 Commit: e067f2d8ac807fd1de2fdbc77e76641c486700b6
 Author: Holger Weiss <[email protected]>
   Date: Sat Nov  7 02:23:32 2009 +0100
    URL: http://nagiosplug.git.sf.net/git/gitweb.cgi?p=nagiosplug/nagiosplug;a=commit;h=e067f2d

git-notify: Drop the $sent_notices variable

Now that we don't ignore empty commits anymore, there's no need to keep
track of the number of commits actually notified about, as that will
always be equal to the number of commits returned by get_new_commits().

---

 tools/git-notify |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/tools/git-notify b/tools/git-notify
index d53461c..548f7da 100755
--- a/tools/git-notify
+++ b/tools/git-notify
@@ -47,9 +47,6 @@ my $cia_address = "cia\@cia.navi.cx";
 # debug mode
 my $debug = 0;
 
-# number of generated (non-CIA) notifications
-my $sent_notices = 0;
-
 # configuration parameters
 
 # base URL of the gitweb repository browser (can be set with the -u option)
@@ -393,7 +390,6 @@ sub send_ref_notice($$@)
 
     mail_notification($commitlist_address, "$refname $reftype $action",
         "text/plain; charset=us-ascii", @notice);
-    $sent_notices++;
 }
 
 # send a commit notice to a mailing list
@@ -467,7 +463,6 @@ sub send_commit_notice($$)
     $subject .= ": " . truncate_str(${$info{"log"}}[0],50);
     $_ = decode($info{"encoding"}, $_) for @notice;
     mail_notification($commitlist_address, $subject, "text/plain; charset=UTF-8", @notice);
-    $sent_notices++;
 }
 
 # send a commit notice to the CIA server
@@ -538,7 +533,6 @@ sub send_global_notice($$$)
     }
 
     mail_notification($commitlist_address, "New commits on branch $ref", "text/plain; charset=UTF-8", @$notice);
-    $sent_notices++;
 }
 
 # send all the notices
@@ -583,7 +577,7 @@ sub send_all_notices($$$)
         {
             send_global_notice( $refname, $old_sha1, $new_sha1 ) if $commitlist_address;
         }
-        else
+        elsif (@$commits > 0)
         {
             foreach my $commit (@$commits)
             {
@@ -591,7 +585,7 @@ sub send_all_notices($$$)
                 send_cia_notice( $refname, $commit ) if $cia_project_name;
             }
         }
-        if ($sent_notices == 0 and $commitlist_address)
+        elsif ($commitlist_address)
         {
             @notice = ( "Old SHA1: $old_sha1", "New SHA1: $new_sha1" );
             send_ref_notice( $ref, "modified", @notice );


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july