On 24/02/14 01:36, Marek Lindner wrote:
> On Monday 24 February 2014 00:00:18 Antonio Quartulli wrote:
>> --- a/send.c
>> +++ b/send.c
>> @@ -255,8 +255,8 @@ int batadv_send_skb_unicast(struct batadv_priv
>> *bat_priv, struct batadv_orig_node *orig_node,
>> unsigned short vid)
>> {
>> - struct batadv_unicast_packet *unicast_packet;
>> struct ethhdr *ethhdr;
>> + struct batadv_unicast_packet *unicast_packet;
>> int ret = NET_XMIT_DROP;
>>
>> if (!orig_node)
>
> Don't we want longer declarations before shorter declarations ?Yes, but the point is that there is no patch doing this change. This line was moved during a merge... If you want to rearrange the variable declaration I'd suggest to do it with one patch changing them all. Cheers, -- Antonio Quartulli
signature.asc
Description: OpenPGP digital signature
