[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4E312270.1020102@gmail.com>
Date: Thu, 28 Jul 2011 16:48:48 +0800
From: WeipingPan <panweiping3@...il.com>
To: David Miller <davem@...emloft.net>
CC: netdev@...r.kernel.org
Subject: Re: [PATCH net-next 0/2] vlan:remove duplicate <linux/if_vlan.h>
On 07/28/2011 04:38 PM, David Miller wrote:
> From: Weiping Pan<panweiping3@...il.com>
> Date: Thu, 28 Jul 2011 16:37:38 +0800
>
>> As net/8021q/vlan.h already includes<linux/if_vlan.h>,
>> so other files that includes vlan.h need not to include it.
> Please do not do this.
>
> If someone wants to remove this implicit header dependency,
> that someone will need to add all of the includes back.
>
> It is better for people to include all the headers they need,
> rather than rely upon such indirect inclusions.
Ok, thanks.
Weiping Pan
> I am not applying these patches.
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists