On Thu, Sep 11, 2014 at 11:23 PM, Julien Grall <julien.grall@linaro.org> wrote:

Hello Tamas,

On 10/09/14 06:28, Tamas K Lengyel wrote:

+ default:
+ break;
+ }

I don't think the default: break; is useful here. I would drop it.

I seem to recall some compilers warning me in the past if there was no default case (in a separate project), but that might just have been an overly restrictive setting I had.

Tamas

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel