Uploaded image for project: 'Qpid Proton'
  1. Qpid Proton
  2. PROTON-1613

Default value for Messenger::recv is -1, not 2048

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Won't Fix
    • Affects Version/s: proton-c-0.17.0
    • Fix Version/s: None
    • Component/s: examples
    • Labels:
    • Flags:
      Patch

      Description

      diff --git a/tests/javascript/msgr-recv.js b/tests/javascript/msgr-recv.js
      index 24bdcae7..f1353685 100755
      --- a/tests/javascript/msgr-recv.js
      +++ b/tests/javascript/msgr-recv.js
      @@ -191,7 +191,7 @@ if (typeof process === 'object' && typeof require === 'function') {
           'Usage: msgr-recv [OPTIONS]\n' +
           ' -a <addr>[,<addr>]* \tAddresses to listen on [amqp://~0.0.0.0]\n' +
           ' -c # \tNumber of messages to receive before exiting [0=forever]\n' +
      -    ' -b # \tArgument to Messenger::recv(n) [2048]\n' +
      +    ' -b # \tArgument to Messenger::recv(n) [-1]\n' +
           ' -w # \tSize for incoming window [0]\n' +
           ' -e # \t# seconds to report statistics, 0 = end of test [0] *TBD*\n' +
           ' -R \tSend reply if \'reply-to\' present\n' +
      diff --git a/tests/tools/apps/c/msgr-recv.c b/tests/tools/apps/c/msgr-recv.c
      index eff5820c..b87f29a8 100644
      --- a/tests/tools/apps/c/msgr-recv.c
      +++ b/tests/tools/apps/c/msgr-recv.c
      @@ -52,7 +52,7 @@ static void usage(int rc)
           printf("Usage: msgr-recv [OPTIONS] \n"
                  " -a <addr>[,<addr>]* \tAddresses to listen on [amqp://~0.0.0.0]\n"
                  " -c # \tNumber of messages to receive before exiting [0=forever]\n"
      -           " -b # \tArgument to Messenger::recv(n) [2048]\n"
      +           " -b # \tArgument to Messenger::recv(n) [-1]\n"
                  " -w # \tSize for incoming window [0]\n"
                  " -t # \tInactivity timeout in seconds, -1 = no timeout [-1]\n"
                  " -e # \t# seconds to report statistics, 0 = end of test [0] *TBD*\n"
      diff --git a/tests/tools/apps/python/msgr-recv.py b/tests/tools/apps/python/msgr-recv.py
      index 757b19c2..52fe69c3 100755
      --- a/tests/tools/apps/python/msgr-recv.py
      +++ b/tests/tools/apps/python/msgr-recv.py
      @@ -34,7 +34,7 @@ usage = """
       Usage: msgr-recv [OPTIONS]
        -a <addr>[,<addr>]* \tAddresses to listen on [amqp://~0.0.0.0]
        -c # \tNumber of messages to receive before exiting [0=forever]
      - -b # \tArgument to Messenger::recv(n) [2048]
      + -b # \tArgument to Messenger::recv(n) [-1]
        -w # \tSize for incoming window [0]
        -t # \tInactivity timeout in seconds, -1 = no timeout [-1]
        -e # \t# seconds to report statistics, 0 = end of test [0] *TBD*
      

      There may possibly be more discrepancies, I spotted and then was able to find only this single one. The function in various bindings is usually not called Messenger::recv...

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              jdanek Jiri Daněk
            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: