It turns out that it would fail every time if…

Consistent failure if I didn't put `next if typename.nil?` before the rest of the statement.  This compromise puts 2 `next if` statements, then an implied `else` since those were the two conditions I needed to check 😄

lib/logstash/codecs/collectd.rb

@@ -120,12 +120,13 @@ class LogStash::Codecs::Collectd < LogStash::Codecs::Base
       @logger.info("Getting Collectd typesdb info", :typesdb => path.to_s)
       File.open(path, 'r').each_line do |line|
         typename, *line = line.strip.split
-        next if typename.nil? || if typename[0,1] != '#' # Don't process commented or blank lines
+        @logger.debug("typename", :typename => typename.to_s)
+        next if typename.nil?
+        next if typename[0,1] == '#'
         v = line.collect { |l| l.strip.split(":")[0] }
         types[typename] = v
       end
     end
-    end
     @logger.debug("Collectd Types", :types => types.to_s)
     return types
   end # def get_types