Prefer snprintf to sprintf

This also simplifies advancing `ptr`
This commit is contained in:
Rangi42
2024-09-15 00:00:39 -04:00
parent 02439b18c0
commit 9b3d19c3f2

View File

@@ -627,8 +627,7 @@ static std::tuple<DefaultInitVec<size_t>, std::vector<Palette>>
static char buf[sizeof(", $XXXX, $XXXX, $XXXX, $XXXX")]; static char buf[sizeof(", $XXXX, $XXXX, $XXXX, $XXXX")];
char *ptr = buf; char *ptr = buf;
for (uint16_t cgbColor : list) { for (uint16_t cgbColor : list) {
sprintf(ptr, ", $%04x", cgbColor); ptr += snprintf(ptr, sizeof(", $XXXX"), ", $%04x", cgbColor);
ptr += QUOTEDSTRLEN(", $XXXX");
} }
return &buf[QUOTEDSTRLEN(", ")]; return &buf[QUOTEDSTRLEN(", ")];
}; };