User Details
- User Since
- Nov 13 2014, 6:24 PM (521 w, 1 d)
Thu, Oct 31
Mon, Oct 21
Sep 9 2024
Aug 29 2024
@corvink your last comment included "I'm going to test this before merging.". I'm assuming you meant "before accepting" and then I will merge onto main and push the commit.
Address review feedback
Aug 28 2024
@corvink thank you for the reviews. Hopefully these last changes should address all of your feedback.
Address review feedback
Address review feedback
Aug 24 2024
Addressing further review feedback
Aug 23 2024
Thank you for the review feedback. I think I address most of it.
Addressing review feedback from corvink
Aug 21 2024
Aug 16 2024
Mar 31 2024
Mar 25 2024
Mar 24 2024
Mar 14 2024
Aug 16 2023
Oct 17 2021
Oct 16 2021
Oct 9 2021
Sep 18 2021
Dec 13 2020
Dec 18 2019
Dec 17 2019
Dec 15 2019
Here is more context in the related bug:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203874#c29
Sep 1 2019
Aug 13 2019
Thanks @jpaetzel for the ping. Over the weekend I updated my build systems to facilitate a build/test of this code. One outstanding issue, I've been meaning to circle back with @imp on how we should implement the licensing. I'll send a PM to work it out. I'll let @vbhakta_vmware.com comment on any remaining code changes.
Aug 10 2019
Thanks! Useful and looks good to me.
Apr 8 2018
Apr 5 2018
Mar 27 2018
Mar 25 2018
Per imp, abandon this review as existing license is sufficient.
Mar 23 2018
Looks good to me. I'll get it committed.