Bug #9062

lib/rexml/parsers/streamparser.rb に require "rexml/parsers/baseparser" を追加してほしい

Added by Ippei Obayashi almost 2 years ago. Updated almost 2 years ago.

[ruby-dev:47779]
Status:Closed
Priority:Normal
Assignee:Kouhei Sutou
ruby -v:ruby 2.1.0dev (2013-10-29 trunk 43466) [x86_64-linux] Backport:1.9.3: UNKNOWN, 2.0.0: UNKNOWN

Description

lib/rexml/parsers/streamparser.rb は内部で REXML::Parsers::BaseParser を使っていますが
rexml/parsers/baseparser を require していません。そのため StreamParser を使うとき
rexml/parsers/streamparser に加えて rexml/parsers/baseparser を require する必要があります。

面倒ですし streamparser.rb が baseparser を require するとよいと思うのですがどうでしょうか。

Associated revisions

Revision 43480
Added by Kouhei Sutou almost 2 years ago

  • lib/rexml/parsers/streamparser.rb: Add dependency file require. [Bug #9062] Reported by Ippei Obayashi. Thanks!!!

Revision 43480
Added by Kouhei Sutou almost 2 years ago

  • lib/rexml/parsers/streamparser.rb: Add dependency file require. [Bug #9062] Reported by Ippei Obayashi. Thanks!!!

History

#1 Updated by Hiroshi SHIBATA almost 2 years ago

  • Assignee set to Kouhei Sutou
  • Target version set to 2.1.0

#2 Updated by Kouhei Sutou almost 2 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r43480.
Ippei, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/rexml/parsers/streamparser.rb: Add dependency file require. [Bug #9062] Reported by Ippei Obayashi. Thanks!!!

#3 Updated by Kouhei Sutou almost 2 years ago

たしかにそのとおりですね!
追加しました!

Also available in: Atom PDF