[Swan-dev] [PATCH libreswan] pluto: Rework nic offload detection code

Paul Wouters paul at nohats.ca
Sun Nov 5 11:40:57 UTC 2017


On Nov 5, 2017, at 15:55, Aviv Heller <aviv at avivh.com> wrote:

>>> I've made some minor changes. Could you review these before I commit
>> them? Also please next time send a patch as attachment, so there isn't
>> whitespace translations that I need to do surgery on :)
> 
> Hi Paul,
> 
> Thanks for taking the time to review my patch.
> 
> A couple of cosmetic points, based on the updated version of your patch:
> 1) At netlink_esp_hw_offload struct definition and the 'if (i >= cmd->len)' on line 908, the indentation was done with spaces instead of tabs (sorry for the trouble, I will use a .patch file next time:).

We use tabs :)

> 2) At 926 'bool ret = false' instead of FALSE.

We use the capital letter versions of those too, so that was also a conscious decision.

I mostly meant the flipped if/then to confirm you were okay with that :)

Paul




More information about the Swan-dev mailing list