RTEMS | Only restore terminal settings on exit when captured (!300)

Gedare Bloom (@gedare) gitlab at rtems.org
Tue Nov 12 16:31:39 UTC 2024



Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/300 was reviewed by Gedare Bloom

--
  
Gedare Bloom started a new discussion on cpukit/libmisc/shell/shell.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/300#note_114793

 > -    if (tcsetattr(fileno(stdin), TCSADRAIN, &previous_term) < 0) {
 > +    if (have_previous_term == 0
 > +        && tcsetattr(fileno(stdin), TCSADRAIN, &previous_term) < 0) {

I would just change this to:

\`\`\`else if (have_previous_term == 0) {\`


-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/300
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20241112/e38dc3da/attachment-0001.htm>


More information about the bugs mailing list