# Code Review: bugs-1

**URL:** <https://www.ruby-forum.com/t/code-review-bugs-1/146179>\
**Category:** IronRuby\
**Created:** [August 18, 2008, 7:56pm UTC](https://www.ruby-forum.com/t/code-review-bugs-1/146179 "2008-08-18T19:56:47Z")\
**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 18, 2008, 7:56pm UTC](https://www.ruby-forum.com/t/code-review-bugs-1/146179/1 "2008-08-18T19:56:47Z")

</div>

tfpt review “/shelveset:bugs-1;REDMOND\jflam”  
Comment :  
Fixes Rakefile to correctly generate case-sensitive filenames during  
compilation  
Fixes String#reverse and taint flow  
Fixes File#zero? behavior (works correctly for directory names, and  
handles NUL correctly)  
Stubs out implementation of File#symlink do throw a  
NotImplementedError (not sure if we can do this without P/Invoke)

---

<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 18, 2008, 8:40pm UTC](https://www.ruby-forum.com/t/code-review-bugs-1/146179/2 "2008-08-18T20:40:51Z")

</div>

Looks good.