Skip to content

Commit cf96153

Browse files
authored
main.c: do not use printf variant if not needed (#20418)
1 parent bc458e5 commit cf96153

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

main/main.c

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,8 @@ PHPAPI char *php_get_version(sapi_module_struct *sapi_module)
127127
{
128128
smart_string version_info = {0};
129129
smart_string_append_printf(&version_info,
130-
"PHP %s (%s) (built: %s) (%s)\n",
131-
PHP_VERSION, sapi_module->name, php_build_date,
130+
"PHP " PHP_VERSION " (%s) (built: %s) (%s)\n",
131+
sapi_module->name, php_build_date,
132132
#ifdef ZTS
133133
"ZTS"
134134
#else
@@ -148,8 +148,12 @@ PHPAPI char *php_get_version(sapi_module_struct *sapi_module)
148148
#endif
149149
);
150150
smart_string_appends(&version_info, "Copyright (c) The PHP Group\n");
151-
if (php_build_provider()) {
152-
smart_string_append_printf(&version_info, "Built by %s\n", php_build_provider());
151+
152+
const char *build_provider = php_build_provider();
153+
if (build_provider) {
154+
smart_string_appends(&version_info, "Built by ");
155+
smart_string_appends(&version_info, build_provider);
156+
smart_string_appendc(&version_info, '\n');
153157
}
154158
smart_string_appends(&version_info, get_zend_version());
155159
smart_string_0(&version_info);
@@ -160,7 +164,7 @@ PHPAPI char *php_get_version(sapi_module_struct *sapi_module)
160164
PHPAPI void php_print_version(sapi_module_struct *sapi_module)
161165
{
162166
char *version_info = php_get_version(sapi_module);
163-
php_printf("%s", version_info);
167+
PHPWRITE(version_info, strlen(version_info));
164168
efree(version_info);
165169
}
166170

0 commit comments

Comments
 (0)