Hi Sergey,

> I prefer not to introduce more ad-hoc prefixes in the log summary.
> Something like moving the "macOS" part to the end should be fine.

That's fine with me.

> style: this breaks a perfect indentation of two spaces after type;
> further, it appears to be unsorted by type; I'd put it after u_long

Good catch, thanks!

> This makes the following slight update to the patch.
> If you're okey with it, I will commit it then.

LGTM.

Best regards,
Piotr Sikora
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to