You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Justin Ross (JIRA)" <ji...@apache.org> on 2018/05/02 22:30:00 UTC

[jira] [Closed] (PROTON-1613) Default value for Messenger::recv is -1, not 2048

     [ https://issues.apache.org/jira/browse/PROTON-1613?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Ross closed PROTON-1613.
-------------------------------
    Resolution: Won't Fix

> Default value for Messenger::recv is -1, not 2048
> -------------------------------------------------
>
>                 Key: PROTON-1613
>                 URL: https://issues.apache.org/jira/browse/PROTON-1613
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: examples
>    Affects Versions: proton-c-0.17.0
>            Reporter: Jiri Daněk
>            Priority: Trivial
>              Labels: messenger, patch
>
> {code}
> 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*
> {code}
> 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...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org