Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/games/battlestar games/battlestar: Replace snprintf(3) with ...



details:   https://anonhg.NetBSD.org/src/rev/e26407300760
branches:  trunk
changeset: 1007105:e26407300760
user:      fox <fox%NetBSD.org@localhost>
date:      Thu Feb 06 22:09:43 2020 +0000

description:
games/battlestar: Replace snprintf(3) with strlcpy(3) for better performance.

Reviewed by: kamil@

diffstat:

 games/battlestar/parse.c |  12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diffs (54 lines):

diff -r 6a2834748b73 -r e26407300760 games/battlestar/parse.c
--- a/games/battlestar/parse.c  Thu Feb 06 20:17:04 2020 +0000
+++ b/games/battlestar/parse.c  Thu Feb 06 22:09:43 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: parse.c,v 1.16 2020/02/05 20:11:54 fox Exp $   */
+/*     $NetBSD: parse.c,v 1.17 2020/02/06 22:09:43 fox Exp $   */
 
 /*
  * Copyright (c) 1983, 1993
@@ -34,7 +34,7 @@
 #if 0
 static char sccsid[] = "@(#)parse.c    8.2 (Berkeley) 4/28/95";
 #else
-__RCSID("$NetBSD: parse.c,v 1.16 2020/02/05 20:11:54 fox Exp $");
+__RCSID("$NetBSD: parse.c,v 1.17 2020/02/06 22:09:43 fox Exp $");
 #endif
 #endif                         /* not lint */
 
@@ -120,7 +120,7 @@
                        for (i = n + 1; i < wordcount; i++) {
                                wordtype[i - 1] = wordtype[i];
                                wordvalue[i - 1] = wordvalue[i];
-                               snprintf(words[i - 1], WORDLEN, "%s", words[i]);
+                               strlcpy(words[i - 1], words[i], WORDLEN);
                        }
                        wordcount--;
                }
@@ -140,7 +140,7 @@
                        for (i = n + 1; i < wordcount; i++) {
                                wordtype[i - 1] = wordtype[i];
                                wordvalue[i - 1] = wordvalue[i];
-                               snprintf(words[i - 1], WORDLEN, "%s", words[i]);
+                               strlcpy(words[i - 1], words[i], WORDLEN);
                        }
                        wordcount--;
                }
@@ -164,7 +164,7 @@
                                wordtype[n + 1] = wordtype[n - 1];
                                wordtype[n - 1] = OBJECT;
                                strcpy(tmpword, words[n - 1]);
-                               snprintf(words[n - 1], WORDLEN, "%s", words[n + 1]);
+                               strlcpy(words[n - 1], words[n + 1], WORDLEN);
                                strcpy(words[n + 1], tmpword);
                                flag = 1;
                }
@@ -177,7 +177,7 @@
                                for (i = n + 1; i < wordcount; i++) {
                                        wordtype[i - 1] = wordtype[i + 1];
                                        wordvalue[i - 1] = wordvalue[i + 1];
-                                       snprintf(words[i - 1], WORDLEN, "%s", words[i + 1]);
+                                       strlcpy(words[i - 1], words[i + 1], WORDLEN);
                                }
                                wordcount--;
                                wordcount--;



Home | Main Index | Thread Index | Old Index