Project

General

Profile

Actions

Bug #7231

closed

StringIO inconsistently raises IOError or RuntimeError if the string is frozen

Added by brixen (Brian Shirai) over 11 years ago. Updated about 11 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 1.9.3p286 (2012-10-12 revision 37165) [x86_64-darwin10.8.0]
Backport:
[ruby-core:48530]

Description

StringIO#ungetc, #ungetbyte raise RuntimeError if the data string is frozen. StringIO#write raises IOError.

Why?

1.9.3p286 :001 > require 'stringio'
=> true
1.9.3p286 :002 > s = StringIO.new x = "abc"
=> #StringIO:0x00000101020038
1.9.3p286 :003 > x.freeze
=> "abc"
1.9.3p286 :004 > s.ungetbyte 0x42
RuntimeError: can't modify frozen String
from (irb):4:in ungetbyte' from (irb):4 from /Users/brian/.rvm/rubies/ruby-1.9.3-p286/bin/irb:16:in '
1.9.3p286 :005 > s.ungetc "w"
RuntimeError: can't modify frozen String
from (irb):5:in ungetc' from (irb):5 from /Users/brian/.rvm/rubies/ruby-1.9.3-p286/bin/irb:16:in '
1.9.3p286 :006 > s.write "d"
IOError: not modifiable string
from (irb):6:in write' from (irb):6 from /Users/brian/.rvm/rubies/ruby-1.9.3-p286/bin/irb:16:in '

Thanks,
Brian


Files

patch.diff (1.21 KB) patch.diff Glass_saga (Masaki Matsushita), 12/25/2012 12:01 PM

Updated by mame (Yusuke Endoh) over 11 years ago

Indeed. I think that it should raise IOError constantly. Any opinion?
Anyone could create a patch, please?

--
Yusuke Endoh

Updated by usa (Usaku NAKAMURA) about 11 years ago

  • Category set to ext
  • Status changed from Open to Assigned
  • Assignee set to nobu (Nobuyoshi Nakada)
  • Target version set to 2.0.0

Updated by Glass_saga (Masaki Matsushita) about 11 years ago

How about the patch?

Actions #4

Updated by Anonymous about 11 years ago

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

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


  • ext/stringio/stringio.c (strio_ungetc): raise IOError instead of RuntimeError
    if the string is frozen.
    [Bug #7231] [ruby-core:48530]

  • ext/stringio/stringio.c (strio_ungetbyte): ditto.

  • test/stringio/test_stringio.rb: a test for above.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0