On 07/02/2013 11:08 AM, Jakub Filak wrote:
Yes, I agree. See the next patch.
Anyway, new_id could be -1 and we should check it. So, this patch is probably unnecessary but makes the code more robust (and I though you like robust code).
Pushed both patches. Thanks!