[issue6697] Check that _PyUnicode_AsString() result is not NULL

[Alexander Belopolsky]

Alexander Belopolsky <belopolsky at users.sourceforge.net> added the comment:

On Tue, Dec 7, 2010 at 1:11 PM, Marc-Andre Lemburg
<report at bugs.python.org> wrote:
>>  I am not sure what
>> you mean by "a parser API."
>
> PyArg_Parse() et al. See the discussion earlier on this ticket.
>

I've just realized that.  It is the "u#" code.  Yes, I'll see if I can
use it instead of "U", but I think in the affected code the PyUnicode
object is needed as well.

>> ..  this is very similar to xmlparser_getattro
>> in _elementtree.c and I think should be handled the same way.
>
> Not sure what you mean here. If you ignore errors and don't clear
> the exception, it will pop up at some later point in processing
> and that's generally very confusing.
>

I mean not converting to char* at all and use
PyUnicode_CompareWithASCIIString() instead of strcmp().  I wish that
function had a shorter name, though, but it is not a big deal to spell
it out.  BTW, I don't think there is a way to use wchar_t* literals in
Python code, right?  As in Py_UNICODE_strcmp(name, L"version").

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue6697>
_______________________________________