Re: [PATCH 1/2] iproute2: tc/m_pedit.c - remove dead code

2015-06-25 Thread Stephen Hemminger
On Thu, 25 Jun 2015 02:03:02 -0700 Maciej Żenczykowski wrote: > From: Maciej Żenczykowski > > The initializers are simply not needed. > > These if-blocks are outright dead code, because '0 > unsigned' is always > false, so only else clause triggers and regardless of which clause triggers > it

[PATCH 1/2] iproute2: tc/m_pedit.c - remove dead code

2015-06-25 Thread Maciej Żenczykowski
From: Maciej Żenczykowski The initializers are simply not needed. These if-blocks are outright dead code, because '0 > unsigned' is always false, so only else clause triggers and regardless of which clause triggers it only updates 'ind' which is later unconditionally written to before being used