Use MAX/MIN over cumbersome statements #268

issue dcousens openend this issue on July 8, 2015
  1. 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.

  2. sipa commented at 9:08 pm on July 8, 2015: contributor
    PR welcome ;)
  3. 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
  4. 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.
  5. dcousens commented at 3:27 am on January 3, 2017: contributor
    So many other fish to fry (HNY haha)
  6. dcousens closed this on Jan 3, 2017


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-04 08:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me