bin/review-check in review-1.6.0 vs bin/review-check in review-1.7.0

- old
+ new

@@ -29,29 +29,18 @@ rescue Errno::EPIPE exit 0 end def main - $KCODE = 'UTF-8' unless defined?(Encoding) @config = ReVIEW::Configure.values - @config.merge!({ - "inencoding" => "UTF-8", - "outencoding" => "UTF-8" - }) @book = ReVIEW::Book::Base.load @book.config = @config modes = nil files = ARGV unless ARGV.empty? opts = OptionParser.new opts.version = ReVIEW::VERSION - opts.on('--inencoding=ENCODING', 'Set input encoding. (UTF-8, EUC, JIS, and SJIS)') {|enc| - @config["inencoding"] = enc - } - opts.on('--outencoding=ENCODING', 'Set output encoding. (UTF-8[default], EUC, JIS, and SJIS)') {|enc| - @config["outencoding"] = enc - } opts.on('-a', '--all-chapters', 'Check all chapters.') { files = @book.chapters.map {|ent| ent.path } } opts.on('-s', '--section N', 'Check section N. (deprecated)') {|n| ents = @book.parts[Integer(n) - 1] or @@ -93,43 +82,43 @@ files.each do |path| File.open(path) {|f| each_paragraph(f) do |para, lineno| s = para.join('') if m = re.match(s) - next if m[0] == $ReVIEW_utils_word_ok + next if m[0] == @review_utils_word_ok next if neg and neg =~ s str, offset = find_line(para, re) out = sprintf("%s:%d: %s\n", path, lineno + offset, str) - print convert_outencoding(out, @config["outencoding"]) + print out end end } end end def find_line(lines, re) # single line? lines.each_with_index do |line, idx| - return line.gsub(re, '<<<\&>>>'), idx if re =~ line + return line.gsub(re, '<<<\&>>>'), idx if re =~ line end # multiple lines? i = 0 while i < lines.size - 1 str = lines[i] + lines[i+1] - return str.gsub(re, '<<<\&>>>'), i if re =~ str + return str.gsub(re, '<<<\&>>>'), i if re =~ str i += 1 end raise 'must not happen' end def words_re(rc) words = [] nega = [] File.foreach(rc) do |line| - next if line[0,1] == '#' + next if line[0,1] == '#' if / !/ =~ line line, n = *line.split(/!/, 2) nega.push n.strip end words.push line.strip @@ -137,17 +126,15 @@ return Regexp.compile(words.join('|')), nega.empty?() ? nil : Regexp.compile(nega.join('|')) end def each_paragraph(f) - $ReVIEW_utils_word_ok = nil + @review_utils_word_ok = nil while line = f.gets - line = convert_inencoding(line, @config["inencoding"]) - case line when /\A\#@ok\((.*)\)/ - $ReVIEW_utils_word_ok = $1 + @review_utils_word_ok = $1 when /\A\#@/ ; when %r[\A//caption\{(.*?)//\}] yield [$1], f.filename, f.lineno when %r<\A//\w.*\{\s*\z> @@ -166,10 +153,10 @@ break if %r<\A(?:=|//[\w\}])> =~ line next if %r<\A\#@> =~ line buf.push line.strip end yield buf, lineno - $ReVIEW_utils_word_ok = nil + @review_utils_word_ok = nil end end end def each_paragraph_line(f, &block)