Re: [Xen-devel] [PATCH v5 13/17] xen/arm: Instruction prefetch abort (X) mem_event handling

[Tamas K Lengyel <tamas.lengyel@xxxxxxxxxxxx>]

On Thu, Sep 11, 2014 at 11:23 PM, Julien Grall <julien.grall@xxxxxxxxxx> 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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

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