Silence some warnings from pedantic static analysis tools, improve compatibility with C++ #228

pull gmaxwell wants to merge 1 commits into bitcoin-core:master from gmaxwell:staticanalysisfixups changing 4 files +9 −5
  1. gmaxwell commented at 9:14 AM on March 8, 2015: contributor

    C doesn't include the null in an array initilized from a string literal if it doesn't fit, in C++ this is invalid.

    The vararray style prototypes and init+calc also changed in this commit are not C89 enough for some tools.

  2. Silence some warnings from pedantic static analysis tools, improve compatibility with C++.
    C doesn't include the null in an array initilized from a
    string literal if it doesn't fit, in C++ this is invalid.
    
    The vararray style prototypes and init+calc also changed in
     this commit are not C89 enough for some tools.
    27bc1311af
  3. gmaxwell commented at 9:16 AM on March 8, 2015: contributor

    (Sorry, wrote this in PR and seemingly never got around to PRing it; presumably because I expected to have more tool-nits like this.)

  4. sipa commented at 10:29 AM on March 16, 2015: contributor

    ACK

  5. sipa merged this on Mar 16, 2015
  6. sipa closed this on Mar 16, 2015

  7. sipa referenced this in commit 3087bc4d75 on Mar 16, 2015
Contributors

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: 2026-05-19 06:53 UTC