Requesting feedback on boost::algorithm::is_clamped
Hi everyone, I have made a MR for boost::algorithm::is_clamped. Details are on github https://github.com/boostorg/algorithm/pull/86. MR is in draft state since doc changes are missing, but tests and code should™ be correct. If you have no time for review of code I would highly appreciate the feedback on the usability of this function. In my work I needed it a few times within last year, but I would like to get feedback from 5-10 other professional developers from diverse backgrounds.
On Fri, Aug 20, 2021 at 12:56 PM Ivan Matek via Boost
I would like to get feedback from 5-10 other professional developers from diverse backgrounds.
Well I'm first generation Italian-American and politically conservative. Anyway, I left some comments on your pull request. Regards
On Mon, Aug 23, 2021 at 4:47 PM Vinnie Falco
On Fri, Aug 20, 2021 at 12:56 PM Ivan Matek via Boost
wrote: I would like to get feedback from 5-10 other professional developers from diverse backgrounds.
Well I'm first generation Italian-American and politically conservative.
Explains why you prefer to standardize ASIO instead of libunifex ;) Anyway, I left some comments on your pull request.
Thank you, I have resolved the ones I knew how to resolve(technical ones),
most of unresolved are just awaiting resolve from you if you agree with my reply. I have no idea what to do about copyright so that remains open. Sorry for lag, was busy during past week. I have a few questions about working in boost. 1. From what I can see b2 does not work when I just fork and clone algorithm repo. 😞 I cloned modular boost and b2 works ok there, but I did not figure out how to point algorithm submodule in modular-boost to my algorithm forked boost/algorithm. Does somebody knows how to make this workflow nicer? Now I am manually c/p stuff from algorithm fork to modular boost submodule directory and running b2(for tests) from there. 2. Does somebody know how to tell b2 to use certain C++ standard(equivalent of --std= for clang/gcc or /std: for msvc)? 3. Is Marshall the only person that can approve changes in algorithm, or is there some OWNERS file I can look up to for who to ping for approval?
On Sun, Aug 29, 2021 at 2:23 PM Ivan Matek
...most of unresolved are just awaiting resolve from you if you agree with my reply. I have no idea what to do about copyright so that remains open.
If you are creating a whole new file I would put your name on the copyright at the top. As for my other comments, they are just advisory and should not be considered as a blocker or an endorsement. In other words, its up to the library maintainer :) Thanks
participants (2)
-
Ivan Matek
-
Vinnie Falco