Hi, Le mardi 20 mai 2014 à 13:25 +0200, Bart Van Assche a écrit : > On 05/20/14 10:33, Yann Droneaud wrote: > > Please find a slightly modified version of your patch to simplify > > a bit the error paths (no backward goto's) and to reduces the amount > > of lines touched. > > > > I wasn't able to explain it clearly enough in the previous patch review, > > so I've made the changes directly in the file and propose the modified > > patch for you to review. > > > > As it's only suggestion, feel free to integrate the changes in your > > patch or discard them. > > Hello Yann, > > It seems like our opinions about backward goto's are different :-) I > thought these are common at the end of error handling code in the Linux > kernel. Anyway, what matters to me is that a fix gets upstream, not > which fix. But please do not expect me to spend more time testing a > patch that has been reworked only because of the coding style aspects > mentioned in your e-mail. >
Having a shorter diff might help to get the patch applied upstream. YMMV. Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html