http://reviews.csiden.org/r/7/
Code cleanup by Matt Ahrens. I think I might have posted this for review a long long time ago already and then forgotten about it, but I can't find the old thread. Note about reviewboard: I'm going to start posting diffs using reviewboard instead of webrevs, it's much easier for me to track what I have out for review and what feedback I need to respond to when I have multiple reviews out at the same time. If you want you can continue send feedback via email, but it would make my life much easier if you registered for an account on that reviewboard server (http://reviews.csiden.org/account/register/) and provide your feedback there ( http://www.reviewboard.org/docs/manual/1.7/users/reviews/reviewing-diffs/). I think you'll find the review process goes much nicer in reviewboard than over email. Also, once you've registered feel free to use that reviewboard instance for your own reviews for illumos-gate as well ( http://www.reviewboard.org/docs/rbtools/dev/). Chris
_______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
