If the format string is invalid, PyUnicode_FromFormatV() formats the valid arguments and then copies the raw format string. Errors are silently ignored.

I propose to raise a ValueError if the format string is invalid. This change may "break" existing applications, but I hope that nobody relies on this "bug" :-)

Attached patch implements my proposition.

The full test suite pass with the patch, so hopefully, CPython doesn't rely on this bug :-)

Hum, my patch is incomplete: "%.s" is not seen as an invalid format.

I don't know the reason for the previous behavior, but it's documented:

http://docs.python.org/dev/c-api/unicode.html?highlight=pyunicode_fromformatv#PyUnicode_FromFormat


"An unrecognized format character causes all the rest of the format string to be copied as-is to the result string, and any extra arguments discarded."

Updated patch: detect also invalid "%.s" format and update the documentation.

> I don't know the reason for the previous behavior, but it's documented:

Oh, I missed the doc. Fixed in the new patch.

Barry wrote on python-dev mailing list:
"If it's documented to behave that way, why would you still consider it a bug?
The current behavior is clearly intentional, the function is working as
intended, and there may be code out there that depends on this documented
functionality, or at least, it won't be prepared to handle the new exception."
http://mail.python.org/pipermail/python-dev/2012-October/122122.html

So I'm closing the issue.