<div dir="ltr">Forgot to CC wine-devel..<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 21, 2017 at 12:47 AM, Aaryaman Vasishta <span dir="ltr"><<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, Apr 20, 2017 at 6:51 AM, Akarsha Sehwag <span dir="ltr"><<a href="mailto:akarsha15010@iiitd.ac.in" target="_blank">akarsha15010@iiitd.ac.in</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi<br>Could someone take out a little time to review the code once?<br>Bug I intend to solve: <a href="https://bugs.winehq.org/show_bug.cgi?id=39640" target="_blank">https://bugs.winehq.org/show_b<wbr>ug.cgi?id=39640</a><br><br><div>I am new to wine. So, I would appreciate suggestions regarding my coding style if that's not-so-perfect according to wine standards. Or any other problems with the code I have written. I apologize for submitting an untested code earlier. I hope now it works fine.<br></div></div></blockquote></span><div>No worries! your efforts in contributing is always appreciated :)</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>/*----------------------------<wbr>---------------CODE-----------<wbr>------------------------------<wbr>---------*/<br></div></div></blockquote><div>Try using diffs/patches instead of manually copying them in this format. Diffs are easier to apply and test with using git. That's how Stefan was able to detect compilation errors in the patches you sent over on wine-patches. I suggest sending another email with the changes in a diif format if possible.</div><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><b>//DOUBT: </b>Would sizeof(list->pid) be better or sizeof(int)? <br>//Since in the former case,<font face="arial, helvetica, sans-serif"> <span style="color:rgb(0,0,0);font-size:13px;white-space:pre-wrap">since list->pid is actually a pointer to DWORD, so //sizeof(list->pid) would return 8 bytes and not 4 bytes.</span></font></div><div><br></div><div>}<br></div></div></blockquote></span><div>Probably sizeof(DWORD) would be fine as well. In general, try following what the surrounding, previously committed code is doing.</div><div>FWIW, pointers don't necessarily have to be 8 bytes in size. The size of a pointer depends on whether your application is compiled as 32 or 64 bit. It's possible to e.g. compile a program as 32 bit on a 64 bit machine.</div><div>For wine in 64 bit: <a href="https://wiki.winehq.org/FAQ#Is_there_a_64_bit_Wine.3F" target="_blank">https://wiki.winehq.org/<wbr>FAQ#Is_there_a_64_bit_Wine.3F</a></div><div><br></div><div>Cheers,</div><div>Aaryaman</div></div></div></div>
</blockquote></div><br></div></div>