Question

获取源码

Mercurial 是一个源码存储工具。可使用以下命令源码仓库：

hg clone http://hg.nginx.org/nginx

格式化变更

变更应根据 nginx 使用的代码样式进行格式化。代码格式化不应该依赖于诸如语法高亮或自动换行等编辑器功能。以下是一些基本规则：

• 最大文本宽度为 80 个字符
• 缩进为四个空格
• 没有 tab（制表符）
• 文件中的逻辑代码块用两行空行分隔

参照现有 nginx 源码的格式，在您的代码中模仿此样式。如果风格与周围的代码相一致，则更改更容易被接受。

提交更改以创建一个 Mercurial 变更集（changeset）。请确保指定的 电子邮件 和变更作者的真实姓名正确无误。

提交消息应是单行简述，后跟一行空行加描述内容。第一行最好不要超过 67 个符号。可以使用 hg export 命令获得一个结果变更集 patch：

# HG changeset patch
# User Filipe Da Silva <username@example.com>
# Date 1368089668 -7200
#    Thu May 09 10:54:28 2013 +0200
# Node ID 2220de0521ca2c0b664a8ea1e201ce1cb90fd7a2
# Parent  822b82191940ef309cd1e6502f94d50d811252a1
Mail: removed surplus ngx_close_connection() call.

It is already called for a peer connection a few lines above.

diff -r 822b82191940 -r 2220de0521ca src/mail/ngx_mail_auth_http_module.c
--- a/src/mail/ngx_mail_auth_http_module.c    Wed May 15 15:04:49 2013 +0400
+++ b/src/mail/ngx_mail_auth_http_module.c    Thu May 09 10:54:28 2013 +0200
@@ -699,7 +699,6 @@ ngx_mail_auth_http_process_headers(ngx_m

           p = ngx_pnalloc(s->connection->pool, ctx->err.len);
           if (p == NULL) {
-            ngx_close_connection(ctx->peer.connection);
             ngx_destroy_pool(ctx->pool);
             ngx_mail_session_internal_server_error(s);
             return;

提交前

提交更改前，有几点值得思考：

• 建议的变更应能在大部分的 支持平台 上正常工作。
• 尽量说明清楚为什么需要更改，如果可以的话，请提供一个用例。
• 通过测试套件传递您的更改是确保不会导致回归的好方法。可以使用以下命令克隆测试仓库：

hg clone http://hg.nginx.org/nginx-tests

提交变更

提议的更改应发送到 nginx 开发 邮件列表。提交更改集的首选便捷方法是使用 patchbomb 扩展。

许可证

提交变更意味着授予项目一个权限以在一个适当的许可证下使用它。