<div dir="ltr"><div><div>Hi Stefan:</div><div><br></div><div>Thanks for your several suggestions .</div><div><br></div><div>I had planned to submit test first and make "Implement findstr.exe for Wine"</div><div> as my GSoC proposal. </div><div>Do you think it is ok to as a GSoC proposal?</div><div>Can you give me some suggestions? </div><div><br></div><div>Thanks again!</div></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-03-23 19:12 GMT+08:00 Stefan Dösinger <span dir="ltr"><<a href="mailto:stefandoesinger@gmail.com" target="_blank">stefandoesinger@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Hi,<br>
<br>
Thanks for your patch! I have a few comments:<br>
<br>
Am 2015-03-23 um 04:14 schrieb Kaipeng Zeng:<br>
> index 5f5e9d5..f3056c7 100755<br>
> --- a/configure<br>
> +++ b/configure<br>
You don't have to include the change to configure in your patch,<br>
Alexandre will update it himself.<br>
<br>
> diff --git a/<a href="http://configure.ac" target="_blank">configure.ac</a> b/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
> index d23227a..822affe 100644<br>
> ...<br>
>  WINE_CONFIG_PROGRAM(findstr,,[install])<br>
> +WINE_CONFIG_TEST(programs/findstr/tests)<br>
>  WINE_CONFIG_PROGRAM(hh,,[install])<br>
I don't think we have any precedent for having tests without having at<br>
least a stub implementation. Personally I don't see anything wrong<br>
with having just a test for now. Can you make sure that the test is<br>
properly built and included in winetest.exe?<br>
<br>
Do you intend to implement the program itself?<br>
<br>
> +static DWORD runcmd(const char* cmd, char *output)<br>
> ...<br>
> +    CloseHandle(hRead);<br>
> +    return TRUE;<br>
> +}<br>
I think it would be a good idea to return the return value of findstr<br>
here instead of always returning true when CreateProcess succeeds.<br>
<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
<br>
iQIcBAEBAgAGBQJVD/UWAAoJEN0/YqbEcdMwh8EP/37ev7ZrJHzmNy5AJfbuzY/s<br>
3wxUIKIo5hwrjJeiwpdFCWAtDQGMo3Hz8weLLyAaCpT4OVrzz/DvbWFLxX3zCl2Z<br>
9EhetkZkyfiPBUF1T+s/WdTse51J9qeP8mFI7NgQdwuwUOXyp2Ba7Kq8DVBM6sNW<br>
MtWsKdItE298jTzL21oQ59w9ar6YrFwRTcVcvLQ3JPmLKSfJ5MR2qT1SxIO6nVC9<br>
Omqc89R9Ba3jSFv+5c3l0Zn6zgnf/U5YBPmsOMYb9DWLNvD0ZgiBh5NZkLxYcV16<br>
NLHQWxJy3mO11RNyjSn1Ank26rR1MiM4kVT7tK3WvneM1IZW7Z/RugGaymtTgfBq<br>
vIIMKidz2hPrIwmuy4AYSEb9ScJIT+b2DCbwgJEWgmxTPl3mymW5ltXLx8yzcmhV<br>
y7GNYGVKuaH6CZeyHLimeLGA845qCzXW15vWxaB6PGw+lVlqpq4mWJnWcB2NOsC+<br>
Auw4vubTf8KTs2UHyQily9AaahthU5IkZA6sPBDpTlZCNlGpRXa+gW88WpKjd5wV<br>
5y9otOU49Fhp704uZAiVcX9ag5cTFaseu0AsTN/TKmYToA50bzAmJr+2/+nJEQIf<br>
lcahIWPeyYw1u7rF3cOEZfkN7FIx/ab1gv1hETLvyoKz/nnYb9NPztnnK3HDFNRP<br>
XJl+R82YChN/YV946HfO<br>
=6sQQ<br>
-----END PGP SIGNATURE-----<br>
</blockquote></div><br></div>