# Code Review: warning1

**URL:** <https://www.ruby-forum.com/t/code-review-warning1/145909>\
**Category:** IronRuby\
**Created:** [August 14, 2008, 11:05pm UTC](https://www.ruby-forum.com/t/code-review-warning1/145909 "2008-08-14T23:05:29Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![Seo\_S](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/s/439d5e/32.png) [@Seo\_S](https://www.ruby-forum.com/u/Seo_S)\
**Post date:** [August 14, 2008, 11:05pm UTC](https://www.ruby-forum.com/t/code-review-warning1/145909/1 "2008-08-14T23:05:29Z")

</div>

Attached patch fixes 4 unused local variable warnings, and possibly a  
bug in ExpressionTreeVisitor. For BinaryExpression node, return value  
of visiting node.Conversion was not passed. Isn’t that a bug?

All warnings were found by Mono C# compiler.