Use MAX/MIN over cumbersome statements #268

issue dcousens opened this issue on July 8, 2015

dcousens commented at 9:07 PM on July 8, 2015: contributor

Reference: https://github.com/bitcoin/secp256k1/pull/266#discussion_r34113224

May be relevant to a few sections, but this is generally a much more readable pattern.
sipa commented at 9:08 PM on July 8, 2015: contributor

PR welcome ;)
dcousens commented at 9:09 PM on July 8, 2015: contributor

@sipa of course, but, unfortunately priorities. Maybe try out the help wanted tag? :P
gmaxwell commented at 6:13 PM on August 27, 2015: contributor

Macro based MAX/MIN ends up evaluating the arguments multiple times; if they have side-effects that can be surprising. Just a thought.
dcousens commented at 3:27 AM on January 3, 2017: contributor

So many other fish to fry (HNY haha)
dcousens closed this on Jan 3, 2017

Contributors