attributes before handling errors

_Pickler_New() now calls PyObject_GC_Del() instead of Py_DECREF() on error,
because the pickle object is created using PyObject_GC_New().

Fix a crash in the destructor when an attribute is not initiallized.

The caller gets an error code and can raise a classic Python exception.

If an attribute is not initialized, the destructor can crash

type->tp_str must not point to slot_tp_str() if type has no __str__ attribute,
so there is no reason for slot_tp_str() to fallback on slot_tp_str() on lookup
error. Moreover, calling PyErr_Clear() may hide a real bug like MemoryError.

If __str__ attribute is removed, slots must be updated (which is done by
type_setattro()).

The problem was I was only checking for decimal digits after the third '?',
not for *hex* digits :(.

This changeset also fixes a couple of comment typos, deletes an unused
function relating to encoded word parsing, and removed an invalid
'if' test from the folding function that was revealed by the tests
written to validate this issue.

more bytes than requested.

more bytes than requested.

do with byte strings.

Added tests for Tcl.split() and Tcl.splitline().

do with byte strings.

Added tests for Tcl.split() and Tcl.splitline().

not to stderr.  Patch by Berker Peksag and Michael Dickens.

Patch by Dmi Baranov.

PyModule_GetDef() returns an error.

PyModule_GetDef() returns an error.

Also fixes a resource warning that occurred when the fallback is taken.

Patch by Serhiy Storchaka.

(We couldn't figure out how to write tests for this.)

Patch by Andrew Rowe.

building the tuple (type, value, traceback) passed to the callback.

PyTuple_Pack() does crash if an argument is NULL.

addresses.

addresses.

in set2list()

Don't mark old extra memory dead before calling realloc(). realloc() can fail
and realloc() must not touch the original buffer on failure.

So mark old extra memory dead only on success if the new buffer did not move
(has the same address).