Project

General

Profile

Bug #9062

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

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

Status:
Closed
Priority:
Normal
Assignee:
ruby -v:
ruby 2.1.0dev (2013-10-29 trunk 43466) [x86_64-linux]
[ruby-dev:47779]

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 3 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 3 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 3 years ago

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

History

#1 [ruby-dev:47782] Updated by Hiroshi SHIBATA almost 3 years ago

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

#2 Updated by Kouhei Sutou almost 3 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 [ruby-dev:47785] Updated by Kouhei Sutou almost 3 years ago

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

Also available in: Atom PDF