diff options
| author | Rahul P. Chaudhari <rahul100885@gmail.com> | 2012-01-30 14:41:22 +0530 |
|---|---|---|
| committer | Rahul P. Chaudhari <rahul100885@gmail.com> | 2012-01-30 14:43:52 +0530 |
| commit | 6831ab11106c582a67bb5109921629059e690610 (patch) | |
| tree | 01b80595c5cbacede4c7c7e4a93639fb21563307 /railties | |
| parent | 30327a00a0a809ca1c8d7dc0fe3261159346f066 (diff) | |
| download | rails-6831ab11106c582a67bb5109921629059e690610.tar.gz rails-6831ab11106c582a67bb5109921629059e690610.tar.bz2 rails-6831ab11106c582a67bb5109921629059e690610.zip | |
Used block to make sure file get auto closed after use
Diffstat (limited to 'railties')
| -rw-r--r-- | railties/lib/rails/code_statistics.rb | 30 |
1 files changed, 16 insertions, 14 deletions
diff --git a/railties/lib/rails/code_statistics.rb b/railties/lib/rails/code_statistics.rb index 435ea83ad8..03538b2422 100644 --- a/railties/lib/rails/code_statistics.rb +++ b/railties/lib/rails/code_statistics.rb @@ -37,24 +37,26 @@ class CodeStatistics #:nodoc: next unless file_name =~ pattern - f = File.open(directory + "/" + file_name) comment_started = false - while line = f.gets - stats["lines"] += 1 - if(comment_started) - if line =~ /^=end/ - comment_started = false - end - next - else - if line =~ /^=begin/ - comment_started = true + + File.open(directory + "/" + file_name) do |f| + while line = f.gets + stats["lines"] += 1 + if(comment_started) + if line =~ /^=end/ + comment_started = false + end next + else + if line =~ /^=begin/ + comment_started = true + next + end end + stats["classes"] += 1 if line =~ /^\s*class\s+[_A-Z]/ + stats["methods"] += 1 if line =~ /^\s*def\s+[_a-z]/ + stats["codelines"] += 1 unless line =~ /^\s*$/ || line =~ /^\s*#/ end - stats["classes"] += 1 if line =~ /^\s*class\s+[_A-Z]/ - stats["methods"] += 1 if line =~ /^\s*def\s+[_a-z]/ - stats["codelines"] += 1 unless line =~ /^\s*$/ || line =~ /^\s*#/ end end |