From bc374a76530826e7f701c6643a59423a2c912be1 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Mon, 21 Dec 2015 05:05:09 +0000 Subject: enable "frozen_string_literal: true" While we're in the area, make a wording change from "GPLv3 or later" to "GPL-3.0+", as the latter is favored by SPDX.org --- lib/dtas/source/sox.rb | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'lib/dtas/source/sox.rb') diff --git a/lib/dtas/source/sox.rb b/lib/dtas/source/sox.rb index 6f02f99..701a36a 100644 --- a/lib/dtas/source/sox.rb +++ b/lib/dtas/source/sox.rb @@ -1,5 +1,7 @@ # Copyright (C) 2013-2015 all contributors -# License: GPLv3 or later (https://www.gnu.org/licenses/gpl-3.0.txt) +# License: GPL-3.0+ (https://www.gnu.org/licenses/gpl-3.0.txt) +# frozen_string_literal: true +# encoding: binary require_relative '../../dtas' require_relative '../source' require_relative '../replaygain' @@ -39,7 +41,7 @@ class DTAS::Source::Sox # :nodoc: end def try(infile, offset = nil, trim = nil) - err = "" + err = "".b cmd = %W(soxi -s #{infile}) s = qx(@env.dup, cmd, err_str: err, no_raise: true) return if err =~ /soxi FAIL formats:/ @@ -65,8 +67,8 @@ class DTAS::Source::Sox # :nodoc: tmp = {} case @infile when String - qx(@env, %W(soxi -a #@infile)).split(/\n/n).each do |line| - key, value = line.split(/=/n, 2) + qx(@env, %W(soxi -a #@infile)).split("\n").each do |line| + key, value = line.split('=', 2) key && value or next # TODO: multi-line/multi-value/repeated tags tmp[key.upcase] = value -- cgit v1.2.3-24-ge0c7