<div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><p style="margin: 0;"><br></p><p style="margin: 0;"><br></p><p style="margin: 0;">Hi <span style="font-family: arial; white-space: pre-wrap;">Konstantin,</span></p><pre>>Could you explain why do we need that define at all?
>As I can read the code, right now IPV4_IPOPT_MANUAL is always defined,
>so all '#else' blocks are simply dead code.
>Is there any reason to keep it?
>If so, then the code probably need to be re-ordered somehow, 
>to make '#else' part to be enabled and executed: 
>let say a separate test-case(s), and/or separate function or extra parameter
<div>>for  test_get_ipv4_opt().</div><div><br></div><div>The code for the '#else' part is a relatively simple way to organize the test data, there is no need to stitch all the data together manually, but it is not as freely organized as the '#if' part of the code, and it was intended that as an alternative, I will remove the code of the '#else' part and resend the patch.</div><div><br></div><div>Huichao Cai</div></pre></div><br><br><span title="neteasefooter"><p> </p></span>