I like the method, but I have some comments about the new method:
- datetime_totimestamp() is not well indented
- "PyObject *time" should be defined at the before the first
instruction
- why not using "if (time == NULL) return NULL;" directly instead of
using a block in case of time is not NULL?
- there are reference leaks: timetuple, timestamp and
PyFloat_FromDouble()
I wrote a similar patch before reading
add-datetime-totimestamp-method.diff which does exactly the same... I
attach my patch but both should be merged. |