Hi Marchand


>It could (should?) have been a single patch.
--Yes, it can.I think the test unit is missing the offsets test content, so add 
this patch, and can also test the previous patch.
>A link to patchwork would avoid me wasting time looking for it.

>I guess this is the patch Thomas merged last night.
--I can see a link in my sent message,As in the screenshot below:
--But I'm not sure why you didn't get the link when you got the email, I'm 
sorry.



>On the patch itself, the title is vague.
>It should summarize what the change adds to the unit tests.

>test/ipfrag: check fragment offsets
--Thank you for reminding me, do I need to send another patch to modify the 
title?
Huichao Cai

Reply via email to