***SPAM*** Re: [ofa-general] ***SPAM*** [PATCH] infiniband-diags/mcm_rereg_test.c: Add missing mad_rpc_close_port call

Hal Rosenstock hal.rosenstock at gmail.com
Thu Mar 12 03:30:49 PDT 2009


On Wed, Mar 11, 2009 at 4:54 PM, Sean Hefty <sean.hefty at intel.com> wrote:
>>>> +#if 1
>>>> +     mad_rpc_close_port(srcport);
>>>> +#else
>>>>       umad_unregister(port, agent);
>>>>       umad_close_port(port);
>>>>       umad_done();
>>>> +#endif
>>>
>>> Why use "#if 1" here?  Why not just replace the umad calls with
>>> mad_rpc_close_port?
>>
>>Just consistency with what's been done in other places in that file.
>>Other similar places are ifdef'd so I kept that.
>
> My vote is to not add an #if 1 and keep dead code around.  If #if 1 appears in
> other places, we should probably remove that dead code as well.

Someone made the decision to want to be able to switch back and forth
earlier. This should be directed to them. It's certainly easy to
eliminate the old code.



More information about the general mailing list