Wine发补丁前的check list

Wine

补丁格式

• PATCH 1/3 不要写错
• Signed-off-by: YongHao Hu < christopherwuy@gmail.com>
• commit message有没有typo

BUG

• 确定demangle没有复制漏错

实现

• 基本每个函数都要写trace
• TRACE里注意分清需要打印指针还是字符串 (char*) dbg_str()
• 用tools/里的工具update其他spec,最好grep一下以防想当然漏掉了
• 不要漏调用规范,thiscall跟cdecl的调用规范要注意以及调用参数
• %2时可以用&1 r%2& 1与r&1一样
• 使用api类型而不是内置,如ULONGLONG代替unsigned long long
• Struct的顺序要注意,尤其是里面数据类型不一致时
• 注意C89标准
  
  • 注释使用/* */而不是//
  • nameless union struct(wine里有DUMMYUNIONNAME)

测试

Add errno and SetLastError test.

换系统时注意

• 换系统时, 留意邮件发送端的名称
• git config -l的配置