Discussion:
4374 dn_free_ranges should use range_tree_t
Christopher Siden
2014-04-14 04:08:05 UTC
Permalink
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



-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/23047029-187a0c8d
Modify Your Subscription: https://www.listbox.com/member/?member_id=23047029&id_secret=23047029-2e85923f
Powered by Listbox: http://www.listbox.com

Loading...