2016-07-01 17:11 GMT+09:00 Andrey Ryabinin <[email protected]>:
>
>
> On 07/01/2016 10:53 AM, [email protected] wrote:
>> From: Joonsoo Kim <[email protected]>
>>
>> If we move an item on qlist's tail, we need to update qlist's tail
>> properly. curr->next can be NULL since it is singly linked list
>> so it is invalid for tail. curr is scheduled to be moved so
>> using prev would be correct.
>
> Hmm.. prev may be the element that moved in 'to' list. We need to assign the 
> last element
> from which is in ther 'from' list.

You're right. Also, I find another bug on this function.
I manage them on v2 and sent.

Thanks.

Reply via email to