This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git

The following commit(s) were added to refs/heads/master by this push:
       new  0e4cb82   Confusing to reuse variable for different purpose
0e4cb82 is described below

commit 0e4cb826c460caf120a674bb45613bbdd2582173
Author: Sebb <[email protected]>
AuthorDate: Sat Jun 3 12:11:27 2017 +0100

    Confusing to reuse variable for different purpose
---
 www/status/monitors/svn.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/status/monitors/svn.rb b/www/status/monitors/svn.rb
index d82282f..ef6a06e 100644
--- a/www/status/monitors/svn.rb
+++ b/www/status/monitors/svn.rb
@@ -51,8 +51,8 @@ def Monitor.svn(previous_status)
 
   # read cron log
   log = File.expand_path('../../../logs/svn-update', __FILE__)
-  data = File.open(log) {|file| file.flock(File::LOCK_EX); file.read}
-  updates = data.split(%r{\n(?:/\w+)*/srv/svn/})[1..-1]
+  fdata = File.open(log) {|file| file.flock(File::LOCK_EX); file.read}
+  updates = fdata.split(%r{\n(?:/\w+)*/srv/svn/})[1..-1]
 
   status = {}
   seen_level = {}

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to