Да, нужно доработать патч. Согласен.
С уважением,
Владимир Коробов
-Original Message-
From: Slawa Olhovchenkov
Sent: Wednesday, October 5, 2022 1:10 PM
To: Korobov Vladimir via nginx-ru
Subject: Re: Исправления срабатываний статического анализатора.
On Wed, Oct 05, 2022 at
Да, патч нужно дорабатывать. Согласен.
С уважением,
Владимир Коробов
-Original Message-
From: Evgeniy Berdnikov
Sent: Wednesday, October 5, 2022 12:33 PM
To: nginx-ru@nginx.org
Subject: Re: Исправления срабатываний статического анализатора.
On Wed, Oct 05, 2022 at 09:24:19AM +,
On Wed, Oct 05, 2022 at 09:24:19AM +, Korobov Vladimir via nginx-ru wrote:
> Окей, неправально выразился.
> Я понимаю, что делает этот патч и он не делает хуже.
окей, возьмем скажем второй кусок патча.
diff -r ba5cf8f73a2d -r ee42d7efc9a6
src/http/modules/ngx_http_proxy_module.c
---
On Wed, Oct 05, 2022 at 09:24:19AM +, Korobov Vladimir via nginx-ru wrote:
> Окей, неправально выразился.
> Я понимаю, что делает этот патч и он не делает хуже.
Я написал, почему этот патч делает хуже, причём СИЛЬНО хуже чем было.
И Максим Дунин высказался. Вы можете что-то возразить
Окей, неправально выразился.
Я понимаю, что делает этот патч и он не делает хуже.
С уважением,
Владимир Коробов
-Original Message-
From: Slawa Olhovchenkov
Sent: Wednesday, October 5, 2022 12:13 PM
To: Korobov Vladimir via nginx-ru
Subject: Re: Исправления срабатываний статического
On Wed, Oct 05, 2022 at 05:22:10AM +, Korobov Vladimir via nginx-ru wrote:
> Кажется, что да, что-то делают. Цели успокоить анализатора нет, тем более
> анализатор генерирует несколько сотен предупреждений, а исправляются патчем
> только несколько.
погоди-погоди, как так "кажется"?
ны