manual: Fix termios.c example. (Bug 31078)
Checks
Context |
Check |
Description |
redhat-pt-bot/TryBot-apply_patch |
success
|
Patch applied to master at the time it was sent
|
redhat-pt-bot/TryBot-32bit |
success
|
Build for i686
|
linaro-tcwg-bot/tcwg_glibc_build--master-aarch64 |
warning
|
Patch is already merged
|
linaro-tcwg-bot/tcwg_glibc_check--master-arm |
warning
|
Patch is already merged
|
linaro-tcwg-bot/tcwg_glibc_check--master-aarch64 |
warning
|
Patch is already merged
|
linaro-tcwg-bot/tcwg_glibc_build--master-arm |
warning
|
Patch is already merged
|
Commit Message
Remove the unused 'char *name;' from the example.
Use write instead of putchar to write input as it is read.
Example tested on x86_64 by compiling and running the example.
Tested by building the manual pdf and reviewing the results.
---
manual/examples/termios.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Comments
* Carlos O'Donell:
> Remove the unused 'char *name;' from the example.
>
> Use write instead of putchar to write input as it is read.
>
> Example tested on x86_64 by compiling and running the example.
>
> Tested by building the manual pdf and reviewing the results.
Looks okay.
Reviewed-by: Florian Weimer <fweimer@redhat.com>
Thanks,
Florian
On 11/20/23 16:38, Florian Weimer wrote:
> * Carlos O'Donell:
>
>> Remove the unused 'char *name;' from the example.
>>
>> Use write instead of putchar to write input as it is read.
>>
>> Example tested on x86_64 by compiling and running the example.
>>
>> Tested by building the manual pdf and reviewing the results.
>
> Looks okay.
>
> Reviewed-by: Florian Weimer <fweimer@redhat.com>
Pushed. Bug marked fixed. Thank you :-}
@@ -34,7 +34,6 @@ void
set_input_mode (void)
{
struct termios tattr;
- char *name;
/* Make sure stdin is a terminal. */
if (!isatty (STDIN_FILENO))
@@ -70,7 +69,7 @@ main (void)
if (c == '\004') /* @kbd{C-d} */
break;
else
- putchar (c);
+ write (STDOUT_FILENO, &c, 1);
}
return EXIT_SUCCESS;