@scresto09 commented on this pull request.
+ /* when the number changes sign, the format is reset to avoid a display shift */ + if ((positive == FALSE && guessed_number >= 0) || (positive == TRUE && guessed_number < 0)) + { + format_length = 0; + } + else + { + format_length = digit_end - digit_start; + if (format_length > 12) + format_length = 0; + } + + g_snprintf(format_buf, sizeof(format_buf)-1, "%%0%d%c", format_length, use_hexa ? ( hexaCase == HEXA_CASE_UPPER ? 'X' : 'x' ) : 'd'); + + if ((buf = g_strdup_printf(format_buf, guessed_number)))
Hum, the code seems correct to me, and it’s really useful for me to use a format that isn’t a "string literal". I don’t think this is a problem. If you'd like, maybe I could add a DEFINE or a CFLAGS to disable this warning ?