Hi Maxim,

>> +#define NGX_DEFAULT_CIPHERS  "DEFAULT"
>> +
>> +
>
> I tend to think it would be better to omit this for clarity, and
> just use the "DEFAULT" string constant in
> ngx_conf_merge_str_value():
>
> (...)
>
> You are ok with this?  If yes, I'll just push the fixed version.

I think that's more consistent with ngx_{mail,http}_ssl_module this
way, but I don't feel strongly about it, so feel free to commit your
version... I won't mind :)

Best regards,
Piotr Sikora

_______________________________________________
nginx-devel mailing list
[email protected]
http://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to