Alexander Potashev <aspotashev@xxxxxxxxx> writes: > This breaks theoretical usage of custom format string in custom die > routines: > 1. A custom die routine might not support %s. > 2. If the die routine adds a custom format specifier, vsnprintf > will fail. I do not think that flies even as a theory. The "custom" 39a3f5ea talks about is more about "Instead of sending it to stderr, let's pop up an info window and give the message there" kind of customization. Existing die() that prints errno does so by giving strerror() to "%s", so no matter how custom your die routine is, you must support '%s' for its output to make sense to humans. If you "not support %s", then you will ignore (or choke at) such strerror() result without this patch anyway. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html