[SLOF] [PATCH slof v2 1/4] net-snk: Fix coding style

Alexey Kardashevskiy aik at ozlabs.ru
Thu Jan 21 20:01:38 AEDT 2016


On 01/21/2016 07:37 PM, Thomas Huth wrote:
> On 21.01.2016 02:39, Alexey Kardashevskiy wrote:
>> This restyles function prototypes:
>> - return types on the same line;
>> - opening brace is on the next line.
>>
>> This replaces [><]* with "*" as >< are also used to resolve merge
>> conflicts.
>>
>> This removes trailing spaces.
>>
>> This should cause no behavioural change.
>>
>> Signed-off-by: Alexey Kardashevskiy <aik at ozlabs.ru>
>> ---
>>   clients/net-snk/app/netlib/dhcp.c     | 134 ++++++++++++++++------------------
>>   clients/net-snk/app/netlib/ethernet.c |  37 +++++-----
>>   clients/net-snk/app/netlib/ipv4.c     |  87 +++++++++-------------
>>   clients/net-snk/app/netlib/ipv4.h     |   2 +-
>>   clients/net-snk/app/netlib/ipv6.c     |  60 +++++----------
>>   clients/net-snk/app/netlib/tcp.c      |  16 ++--
>>   clients/net-snk/app/netlib/tftp.c     |  22 +++---
>>   clients/net-snk/app/netlib/udp.c      |  20 ++---
>>   8 files changed, 161 insertions(+), 217 deletions(-)
> ...
>> diff --git a/clients/net-snk/app/netlib/ethernet.c b/clients/net-snk/app/netlib/ethernet.c
>> index bbfd6d1..b73239e 100644
>> --- a/clients/net-snk/app/netlib/ethernet.c
>> +++ b/clients/net-snk/app/netlib/ethernet.c
> ...
>> @@ -176,9 +175,9 @@ send_ether(int fd, void* buffer, int len)
>>    * @see                fill_dnshdr
>>    * @see                fill_btphdr
>>    */
>> -void
>> -fill_ethhdr(uint8_t * packet, uint16_t eth_type,
>> -            const uint8_t * src_mac, const uint8_t * dest_mac) {
>> +void fill_ethhdr(uint8_t * packet, uint16_t eth_type,
>> +        const uint8_t * src_mac, const uint8_t * dest_mac)
>
> Indentation looks wrong here ... wouldn't it be nicer to align the
> parameters in the second line with the opening "(" ?


I like the kernel style better but in general it does not change a thing. 
Let's vote here what style to use - kernel or QEMU :)


> (this also happens at some other spots in the patch)
>
> Apart from that, the patch looks fine to me.


-- 
Alexey


More information about the SLOF mailing list