Deprecate RPC getblocknumber #632

pull mndrix wants to merge 1 commits into bitcoin:master from mndrix:deprecate-getblocknumber changing 1 files +4 −2
  1. mndrix commented at 12:30 AM on November 11, 2011: contributor

    This RPC is exactly identical to getblockcount. This duplication dates back to commit 22f721dbf23cf5ce9e3ded9bcfb65a3894cc0f8c when Satoshi created the RPC interface.

    There's no need to have both, so we standardize on "count" which matches the naming convention in getconnectioncount.

  2. tcatm commented at 1:04 AM on November 11, 2011: none

    NACK I agree with deprecating getblocknumber, however deprecation should not remove it from pCallTable yet. This would be a seperate patch for a later version. You should add comments like "// deprecated" to the calltable entry and the function itself so we can easily remove the code later.

    midstate and hash1 are an example of RPC features that are deprecated but not removed yet.

  3. mndrix commented at 3:26 AM on November 11, 2011: contributor

    Good point. I'll repair the patch and resubmit tomorrow.

  4. Deprecate RPC getblocknumber
    This RPC is exactly identical to getblockcount.  This duplication
    dates back to commit 22f721dbf23cf5ce9e3ded9bcfb65a3894cc0f8c when
    Satoshi created the RPC interface.
    
    There's no need to have both, so we standardize on "count" which
    matches the naming convention in getconnectioncount.
    
    Following the tradition established with previously deprecated APIs,
    getblocknumber continues to work, but it's not listed in the help
    system.
    29c8b9416d
  5. laanwj commented at 2:47 PM on November 12, 2011: member

    ACK

  6. gavinandresen referenced this in commit 0310cd6403 on Nov 21, 2011
  7. gavinandresen merged this on Nov 21, 2011
  8. gavinandresen closed this on Nov 21, 2011

  9. coblee referenced this in commit 987fde84a3 on Jul 17, 2012
  10. sanch0panza referenced this in commit 5e6adb21a9 on May 17, 2018
  11. dexX7 referenced this in commit adfb0edf94 on Aug 25, 2018
  12. rajarshimaitra referenced this in commit e2b92b092d on Aug 5, 2021
  13. DrahtBot locked this on Sep 8, 2021
Contributors

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-22 06:16 UTC

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