Skip to content

Commit

Permalink
Merge pull request #139 from lukejahnke/master
Browse files Browse the repository at this point in the history
Fixes for warnings (related to issue 106)
  • Loading branch information
colszowka committed Jul 31, 2012
2 parents 14219ee + 0a3fd58 commit f9979f0
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/simplecov/configuration.rb
Expand Up @@ -138,7 +138,7 @@ def at_exit(&block)
# the SimpleCov.root is this.
#
def project_name(new_name=nil)
return @project_name if @project_name and new_name.nil?
return @project_name if defined? @project_name and @project_name and new_name.nil?
@project_name = new_name if new_name.kind_of?(String)
@project_name ||= File.basename(root.split('/').last).capitalize.gsub('_', ' ')
end
Expand All @@ -148,8 +148,8 @@ def project_name(new_name=nil)
# are joined and combined into a single coverage report
#
def use_merging(use=nil)
@use_merging = use unless use.nil? # Set if param given
@use_merging = true if @use_merging != false
@use_merging = use unless use.nil?
@use_merging = true unless defined? @use_merging and @use_merging == false
end

#
Expand Down

0 comments on commit f9979f0

Please sign in to comment.