On 19/08/15 02:23, Michael Neuling wrote:
So this doesn't break when you add a new state, is it worth writing it as:

        if (ctx->status >= STARTED)
                return -EBUSY;

?

Yeah I think that would be more future proof, although it won't make a
difference with the current code.

Sounds reasonable, I'll submit a V2.


Andrew

--
Andrew Donnellan              Software Engineer, OzLabs
andrew.donnel...@au1.ibm.com  Australia Development Lab, Canberra
+61 2 6201 8874 (work)        IBM Australia Limited

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to