diff mbox series

keytable: fix stdout/stderr inconsistency

Message ID BL3PR19MB65145B42CD9A31304D6DF5CED30C2@BL3PR19MB6514.namprd19.prod.outlook.com (mailing list archive)
State New, archived
Headers show
Series keytable: fix stdout/stderr inconsistency | expand

Commit Message

vricosti April 20, 2024, 3:01 p.m. UTC
This patch fix the keytable inconsistency about stdout/stderr.  

Signed-off-by: Vince Ricosti <vricosti@outlook.com>
---
 utils/keytable/keytable.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sean Young April 22, 2024, 10:40 a.m. UTC | #1
On Sat, Apr 20, 2024 at 05:01:14PM +0200, vricosti wrote:
> This patch fix the keytable inconsistency about stdout/stderr.  
> 
> Signed-off-by: Vince Ricosti <vricosti@outlook.com>
> ---
>  utils/keytable/keytable.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c
> index 9f7575d6..5f5b88dd 100644
> --- a/utils/keytable/keytable.c
> +++ b/utils/keytable/keytable.c
> @@ -1728,7 +1728,7 @@ static int get_rate(int fd, unsigned int *delay, unsigned int *period)
>         }
>         *delay = rep[0];
>         *period = rep[1];
> -       printf(_("Repeat delay: %d ms, repeat period: %d ms\n"), *delay, *period);
> +       fprintf(stderr,_("Repeat delay: %d ms, repeat period: %d ms\n"), *delay, *period);

You're right, there is a problem there. I've found a few more instances of
stdout being used rather than stderr, I'll post an update patch shortly.


Sean
diff mbox series

Patch

diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c
index 9f7575d6..5f5b88dd 100644
--- a/utils/keytable/keytable.c
+++ b/utils/keytable/keytable.c
@@ -1728,7 +1728,7 @@  static int get_rate(int fd, unsigned int *delay, unsigned int *period)
        }
        *delay = rep[0];
        *period = rep[1];
-       printf(_("Repeat delay: %d ms, repeat period: %d ms\n"), *delay, *period);
+       fprintf(stderr,_("Repeat delay: %d ms, repeat period: %d ms\n"), *delay, *period);
        return 0;
 }