cleanup TODOs, FIXMEs

korny · korny · commit cb41b00d5673 · 2013-06-16T01:01:12.000+02:00
diff --git a/bin/coderay b/bin/coderay
@@ -155,8 +155,9 @@ when 'highlight', nil
         puts boom.message
       end
       # puts "I don't know this plugin: #{boom.message[/Could not load plugin (.*?): /, 1]}."
-    rescue CodeRay::Scanners::Scanner::ScanError  # FIXME: rescue Errno::EPIPE
-      # this is sometimes raised by pagers; ignore [TODO: wtf?]
+    rescue CodeRay::Scanners::Scanner::ScanError
+      # this is sometimes raised by pagers; ignore
+    # FIXME: rescue Errno::EPIPE
     ensure
       file.close if output_file
     end
diff --git a/lib/coderay.rb b/lib/coderay.rb
@@ -166,7 +166,6 @@ class << self
     #
     # See also demo/demo_simple.
     def scan code, lang, options = {}, &block
-      # FIXME: return a proxy for direct-stream encoding
       TokensProxy.new code, lang, options, block
     end
     
diff --git a/lib/coderay/encoders/html/numbering.rb b/lib/coderay/encoders/html/numbering.rb
@@ -75,7 +75,7 @@ def self.number! output, mode = :table, options = {}
           line_number = start
           output.gsub!(/^.*$\n?/) do |line|
             line_number_text = bolding.call line_number
-            indent = ' ' * (max_width - line_number.to_s.size)  # TODO: Optimize (10^x)
+            indent = ' ' * (max_width - line_number.to_s.size)
             line_number += 1
             "<span class=\"line-numbers\">#{indent}#{line_number_text}</span>#{line}"
           end
diff --git a/lib/coderay/encoders/statistic.rb b/lib/coderay/encoders/statistic.rb
@@ -67,7 +67,6 @@ def text_token text, kind
       @type_stats['TOTAL'].count += 1
     end
     
-    # TODO Hierarchy handling
     def begin_group kind
       block_token ':begin_group', kind
     end
diff --git a/lib/coderay/token_kinds.rb b/lib/coderay/token_kinds.rb
@@ -44,7 +44,7 @@ module CodeRay
     :important           => 'important',          # CSS, Taskpaper
     :include             => 'include',            # C, Groovy, Java, Python, Sass
     :inline              => 'inline',             # nested code, eg. inline string evaluation; lots of scanners
-    :inline_delimiter    => 'inline-delimiter',   # used instead of :inline > :delimiter FIXME: Why?
+    :inline_delimiter    => 'inline-delimiter',   # used instead of :inline > :delimiter FIXME: Why use inline_delimiter?
     :instance_variable   => 'instance-variable',  # Ruby
     :integer             => 'integer',            # most scanners
     :key                 => 'key',                # lots of scanners, used together with :value

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,6 @@ class << self`
`166`	`166`	`#`
`167`	`167`	`# See also demo/demo_simple.`
`168`	`168`	`def scan code, lang, options = {}, &block`
`169`		`- # FIXME: return a proxy for direct-stream encoding`
`170`	`169`	`TokensProxy.new code, lang, options, block`
`171`	`170`	`end`
`172`	`171`