Re: [PATCH v2] net: hyperv: else branch not necessary

2015-01-27 Thread David Miller
From: Nicholas Mc Guire Date: Sun, 25 Jan 2015 19:08:37 +0100 > As the if completes with a unconditional goto the else branch > is not needed here. > > Signed-off-by: Nicholas Mc Guire > --- > > v2: added missing subsystem string in subject line - patch unchanged The feedback you were given w

RE: [PATCH v2] net: hyperv: else branch not necessary

2015-01-26 Thread Haiyang Zhang
bject: [PATCH v2] net: hyperv: else branch not necessary > > As the if completes with a unconditional goto the else branch > is not needed here. > > Signed-off-by: Nicholas Mc Guire Signed-off-by: Haiyang Zhang Thank you! > --- > > v2: added missing subsystem string in sub

[PATCH v2] net: hyperv: else branch not necessary

2015-01-25 Thread Nicholas Mc Guire
As the if completes with a unconditional goto the else branch is not needed here. Signed-off-by: Nicholas Mc Guire --- v2: added missing subsystem string in subject line - patch unchanged All paths of execution that did not exit through the if branch will go through the else branch so no need