Minimized the diff (I believe) as requested.

Codecov Report

Merging#140 (585f67a) intomaster (2af4a9c) willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master     #140   +/-   ##=======================================  Coverage   96.06%   96.06%           =======================================  Files          14       14             Lines         839      839           =======================================  Hits          806      806             Misses         33       33

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update2af4a9c...585f67a. Read thecomment docs.

Thanks! It's not exactly as I suggested yet, because:

• You added indentation to the min/max macros, so, there's still a (IMHO unneeded) diff there. I would remove the indent again so these macros are not changed at all.
• I also suggested (at least tried to) to move the rest of the functions and macros (constrain untilsq) down to wheremin andmax are defined, so they can share a singleifdef __cplusplus block, and are also in a place where you do not need to interrupt theextern "C" block.
• It seems one of the commits also removes some trailing whitespace in from the twotemplate<class T, class L> lines for the min/max template functions, I'd also either revert that, or put it in a separate commit.

Ideally, you'd also squash all your commits together into a single commit (except maybe for whitespace-only changes, those would better be in separate commits), not sure if you're familiar with merging commits / rewriting history in git?

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign ourContributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let usrecheck it.}