Author: cy
Date: Fri Nov  3 13:08:29 2017
New Revision: 325357
URL: https://svnweb.freebsd.org/changeset/base/325357

Log:
  While discussing the new gets_s.c in D12785, ed@ suggested putting
  {}'s around the if (c == EOF) block to prevent potential 'trailing else'
  issues from being introduced when refactoring. As my gets_s() code
  is based on this, it makes sense to fix the same issue here first
  here and now, then do an svn copy again to capture this history).
  
  Suggested by: ed@ in D12785

Modified:
  head/lib/libc/stdio/gets.c

Modified: head/lib/libc/stdio/gets.c
==============================================================================
--- head/lib/libc/stdio/gets.c  Fri Nov  3 12:52:59 2017        (r325356)
+++ head/lib/libc/stdio/gets.c  Fri Nov  3 13:08:29 2017        (r325357)
@@ -61,13 +61,13 @@ gets(char *buf)
                warned = 1;
        }
        for (s = buf; (c = __sgetc(stdin)) != '\n'; ) {
-               if (c == EOF)
+               if (c == EOF) {
                        if (s == buf) {
                                ret = NULL;
                                goto end;
                        } else
                                break;
-               else
+               } else
                        *s++ = c;
        }
        *s = 0;
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to