diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-01-19 20:15:13 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-01-19 20:15:13 +0000 |
commit | 64ec8521820b1f8b684ad0c2a5907b160d3fc2f8 (patch) | |
tree | a6cfeaf6300dd79bba3260485206607e950b1c38 /enc/make_encdb.rb | |
parent | 388205aaa77bb8191172ca0b42f57c1ea019df29 (diff) | |
download | ruby-64ec8521820b1f8b684ad0c2a5907b160d3fc2f8.tar.gz ruby-64ec8521820b1f8b684ad0c2a5907b160d3fc2f8.tar.xz ruby-64ec8521820b1f8b684ad0c2a5907b160d3fc2f8.zip |
* enc/make_encdb.rb: fix duplication check.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@15135 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'enc/make_encdb.rb')
-rwxr-xr-x | enc/make_encdb.rb | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/enc/make_encdb.rb b/enc/make_encdb.rb index 15d5f3af3..6eda76e5c 100755 --- a/enc/make_encdb.rb +++ b/enc/make_encdb.rb @@ -9,27 +9,29 @@ # ENC_ALIAS("CP932", "Windows-31J") # -def check_duplication(encs, name, fn, line) - if encs.include?(name) - raise ArgumentError, "%s:%d: encoding %s is already registered" % [fn, line, name] +def check_duplication(defs, name, fn, line) + if defs[name] + raise ArgumentError, "%s:%d: encoding %s is already registered(%s:%d)" % + [fn, line, name, *defs[name]] + else + defs[name.upcase] = [fn,line] end end count = 0 lines = [] encodings = [] +defs = {} encdir = ARGV[0] outhdr = ARGV[1] || 'encdb.h' Dir.open(encdir) {|d| d.grep(/.+\.[ch]\z/)}.sort.each do |fn| open(File.join(encdir,fn)) do |f| orig = nil name = nil - encs = [] f.each_line do |line| if (/^OnigEncodingDefine/ =~ line)..(/"(.*?)"/ =~ line) if $1 - check_duplication(encs, $1, fn, $.) - encs << $1.upcase + check_duplication(defs, $1, fn, $.) encodings << $1 count += 1 end @@ -41,19 +43,18 @@ Dir.open(encdir) {|d| d.grep(/.+\.[ch]\z/)}.sort.each do |fn| when /^ENC_REPLICATE\(\s*"([^"]+)"\s*,\s*"([^"]+)"/ raise ArgumentError, '%s:%d: ENC_REPLICATE: %s is not defined yet. (replica %s)' % - [fn, $., $2, $1] unless encs.include?($2.upcase) + [fn, $., $2, $1] unless defs[$2.upcase] count += 1 when /^ENC_ALIAS\(\s*"([^"]+)"\s*,\s*"([^"]+)"/ raise ArgumentError, '%s:%d: ENC_ALIAS: %s is not defined yet. (alias %s)' % - [fn, $., $2, $1] unless encs.include?($2.upcase) + [fn, $., $2, $1] unless defs[$2.upcase] when /^ENC_DUMMY\(\s*"([^"]+)"/ count += 1 else next end - check_duplication(encs, $1, fn, $.) - encs << $1.upcase + check_duplication(defs, $1, fn, $.) lines << line.sub(/;.*/m, ";\n") if line end end |