> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Tuesday, May 8, 2018 15:45
> On Tue, 8 May 2018 22:26:21 +0000
> Dexuan Cui <de...@microsoft.com> wrote:
> 
> > @@ -63,6 +63,9 @@ static __u32 vmbus_get_next_version(__u32
> current_version)
> >     case (VERSION_WIN10):
> >             return VERSION_WIN8_1;
> >
> > +   case (VERSION_WIN10_V5):
> > +           return VERSION_WIN10;
> > +
> >     case (VERSION_WS2008):
> 
> In a future patch, we should get rid of the use of parenthesis on this switch
> statement.  Lots of other cleanup of this code is possible as well.

I agree.
We can make another patch to clean up this function. Maybe we should
replace the function with an array of versions.

Thanks,
-- Dexuan
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to