[PATCH] ngx_mail_ssl_module / ecdh_curve
Maxim Dounin
mdounin at mdounin.ru
Mon May 20 23:31:39 UTC 2013
Hello!
On Mon, May 20, 2013 at 11:37:39PM +0200, Filipe Da Silva wrote:
[...]
> > > # HG changeset patch
> > > # User fds <fdasilvayy at gmail.com>
> >
> > Could you please use a proper name here? Just "fds" looks wierd.
> >
> > > # Date 1368197625 -7200
> > > # Node ID f6c3e219648f47eaa65ab43a8189af31f7babc69
> > > # Parent 7ffe7f3be4045dbc7478248370b473271660e473
> > > Missing call to ngx_ssl_ecdh_curve in mail_ssl_module
> >
> > Please use trailing dot and "Mail:" prefix for mail-related
> > changes.
> >
> > That is, I would like to see something like this here:
> >
> > Mail: missing ngx_ssl_ecdh_curve() call.
[...]
> # HG changeset patch
> # User F. da Silva<fdasilvayy at gmail.com>
At least space after a name seems to be missing.
What about "Filipe Da Silva <fdasilvayy at gmail.com>", as used in
the From address of your messages?
> # Date 1368197625 -7200
> # Fri May 10 16:53:45 2013 +0200
> # Node ID 24369ba28ede7baea634de2dfbab899925830f34
> # Parent cfab1e7e4ac2f0d17199ee1d49ac4647b63746d3
> Mail: Missing ngx_ssl_ecdh_curve call.
No capitalization after after a prefix, please. Parentheses after
a function name is also generally good idea to explicitly mark
functions. That is, I would like to see what was previously
suggested:
Mail: missing ngx_ssl_ecdh_curve() call.
(Please let me know if you'll prefer me to fix it myself. It
would be better if you'll fix it yourself though - especially if
you are planning to submit more patches.)
(Please also consider switching off html in your messages to
nginx-devel at . Thank you.)
--
Maxim Dounin
http://nginx.org/en/donation.html
More information about the nginx-devel
mailing list