[wine-zh] Re: add PfBindInterfaceToIPAddress func stub
- From: Qian Hong <fracting@xxxxxxxxx>
- To: wine-zh <wine-zh@xxxxxxxxxxxxx>, penghao <penghao@xxxxxxxxxxxxxxx>
- Date: Wed, 23 Jul 2014 17:10:35 +0800
彭浩你好, 谢谢你的工作!
有几个小建议:
1. 在git中设置 format.suffix=.txt , 这样生成的补丁默认是 .txt 结尾的, 在邮箱/网页上review更方便
(Wine项目也有很多人用 .patch扩展名, 这个严格地说算是个人喜好, 我觉得.txt比.patch好)
2. 在发送邮件的时候CC自己, 这样别人点击 reply all的时候, 就会CC到你
3. 设置 Reply-To: wine-devel <wine-devel@xxxxxxxxxx> , 这样别人点击回复的时候,
可以回复到wine-devel.
4. git commit message最好用大小写分明的完整语句, 如 Add PfBindInterfaceToIPAddress
stub implementation. (不要在commit
message中用func这样的缩写,要用就用完整的function这个单词. 至于动词的时态,则没有统一,
用Added的和用Add的都有.)
5. 如果一组补丁之间有联系, 那么用git format patch + 邮件客户端的方式,可以做到自动发出一系列邮件,
这组邮件的标题自动按 [Patch 1/3] xxxx, [Patch 2/3] xxxx, [Patch 3/3] xxxx 的形式起名.
我原来是想让你用wine-zh作为实验场, 把这几个细节改好, 不过看到你已经发到wine-patch了, 那就算了,
下次再注意这些细节就好了.下次就不必发到wine-zh了.
2014-07-23 16:50 GMT+08:00 彭浩 <penghao@xxxxxxxxxxxxxxx>:
> 各位好, 我武汉深之度这边参与wine项目的人员, 很高兴认识大家.
> 之前因为别的事情耽搁了, 所以这个补丁拖了一段时间. 现按照facting的要求分拆成三个补丁
>
> ---
> dlls/iphlpapi/iphlpapi.spec | 2 +-
> dlls/iphlpapi/iphlpapi_main.c | 9 +++++++++
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
>
--
Regards,
Qian Hong
-
http://www.winehq.org
Other related posts:
