Hi Dave,

Could you re-indent your code. You've created a lot of unnecessary diffs in 
your patch and there is too much interferences for a clean review.


i.e.

On 07/07/2015 09:21 PM, Dave Zhu (yanbzhu) wrote:
> -     if (bind_conf->default_ctx) {
> -             memprintf(err, "%sthis version of openssl cannot load multiple 
> SSL
> certificates.\n",
> -                       err && *err ? *err : "");
> -             return 1;
> -     }
> +            if (bind_conf->default_ctx) {
> +                memprintf(err, "%sthis version of openssl cannot load
> multiple SSL certificates.\n",
> +                        err && *err ? *err : "");
> +                goto end;
> +            }

R,
Emeric

Reply via email to