# Code Review: bugs-2

**URL:** <https://www.ruby-forum.com/t/code-review-bugs-2/146607>\
**Category:** IronRuby\
**Created:** [August 22, 2008, 11:35pm UTC](https://www.ruby-forum.com/t/code-review-bugs-2/146607 "2008-08-22T23:35:50Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![John\_LSSIRONRUBY](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/4491bb/32.png) [@John\_LSSIRONRUBY](https://www.ruby-forum.com/u/John_LSSIRONRUBY)\
**Post date:** [August 22, 2008, 11:35pm UTC](https://www.ruby-forum.com/t/code-review-bugs-2/146607/1 "2008-08-22T23:35:50Z")

</div>

tfpt review “/shelveset:bugs-2;REDMOND\jflam”  
Comment :

- 

```
Fixes Rubyforge #21069: Array#delete on a frozen array throws a 

```

TypeError even if modification doesn’t take place

- 

```
Fixes Array#flatten frozen bug (related)

```

- 

```
Fixes additional spec failure in Array#flatten where to_ary not 

```

called on elements

- 

```
Punting for now on the fix for Array#flatten that dispatches by 

```

calling Array#flatten on elements - right now we don’t have a good  
mechanism to pass our ‘seen’ dictionary to callers across the dynamic  
invocation boundary (see TODO)

- 

```
Fixes Rubyforge #20125: MatchData#inspect returns incorrect 

```

representation of matchdata

---

<div class="post-metadata">

**Author:** ![John\_LSSIRONRUBY](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/4491bb/32.png) [@John\_LSSIRONRUBY](https://www.ruby-forum.com/u/John_LSSIRONRUBY)\
**Post date:** [August 23, 2008, 12:40am UTC](https://www.ruby-forum.com/t/code-review-bugs-2/146607/2 "2008-08-23T00:40:11Z")

</div>

Looks good.