I did the change because I dont see the point having extra-parentheses around a single term (ServerDll , or ServerDll->HardErrorCallback), it makes reading difficult (but Im of course ok with such things : if ((var_1 >= a) && (var_1 <= b)) ), and furthermore this thing happens only in these instructions : if ((ServerDll) && (ServerDll->A_Callback)) or if ((CsrLoadedServerDll[i]) && (CsrLoadedServerDll[i]->SizeOfProcessData)) in the code
Regards, Hermès BÉLUSCA - MAÏTO De : [email protected] [mailto:[email protected]] De la part de Alex Ionescu Envoyé : mercredi 14 novembre 2012 05:34 À : ReactOS Development List Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 57701: [CSRSRV] - Use a variable ServerDll instead of using each time CsrLoadedServerDll[i], as it is done in some other places. - Clean the code (remove extra-parentheses). - Zero-out t... Would you mind not "cleaning up formatting" by going against our coding style and doing the very thing I even taught you not to do during your initial patches/code reviews? Thanks. Best regards, Alex Ionescu On Tue, Nov 13, 2012 at 1:08 PM, <[email protected]> wrote: - if ((ServerDll) && (ServerDll->HardErrorCallback)) + if (ServerDll && ServerDll->HardErrorCallback)
_______________________________________________ Ros-dev mailing list [email protected] http://www.reactos.org/mailman/listinfo/ros-dev
