admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Use guard clause instead of ||.

Browse Source
This commit is contained in:
Markus Reiter 2018-07-19 13:36:27 +02:00
parent fec884d8da
commit ad7be58655
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Library/Homebrew/download_strategy.rb
View File

@ -69,7 +69,8 @@ class AbstractDownloadStrategy
def safe_system(*args) def safe_system(*args)
if @shutup if @shutup
quiet_system(*args) || raise(ErrorDuringExecution.new(args, status: $CHILD_STATUS)) return if quiet_system(*args)
raise(ErrorDuringExecution.new(args, status: $CHILD_STATUS))
else else
super(*args) super(*args)
end end

3
Library/Homebrew/utils.rb
View File

@ -302,7 +302,8 @@ end
# Kernel.system but with exceptions # Kernel.system but with exceptions
def safe_system(cmd, *args, **options) def safe_system(cmd, *args, **options)
Homebrew.system(cmd, *args, **options) || raise(ErrorDuringExecution.new([cmd, *args], status: $CHILD_STATUS)) return if Homebrew.system(cmd, *args, **options)
raise(ErrorDuringExecution.new([cmd, *args], status: $CHILD_STATUS))
end end
# prints no output # prints no output