[lvc-project] Line with no value и прочее

[Alexey Khoroshilov]

Доброе утро, Дмитрий!

On 21.03.2024 08:31, Antipov, Dmitriy wrote:
> Я надеюсь, что все заинтересованные лица это уже видели:
>
> On Wed, 2024-03-20 at 14:38 +0100, Jiri Pirko wrote:
>> Wed, Mar 20, 2024 at 01:56:35PM CET, abelova at astralinux.ru wrote:
>>> skb is an optional parameter, so it may be NULL.
>>> Add check defore dereference in eth_hdr.
>>>
>>> Found by Linux Verification Center (linuxtesting.org) with SVACE.
>> Either drop this line which provides no value, or attach a link to the
>> actual report.
>>> It seems this fix is based on some kind of static analysis, but no real bug.
>> Yeah, I agree. That's the main reason I asked for the link to the report.
> Какова наша официальная позиция по этому вопросу? Я абсолютно уверен,
> что подобные комментарии мы будем получать и в дальнейшем, и думаю,
> что сейчас самое время выработать единообразный подход, которого
> будут придерживаться все участники проекта.
Да, вопрос изредка возникает. За три года это второй случай на моей памяти.

Типовой ответ:

It is an established practice for our project, you can find 700+ applied
patches with similar line:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/log/?qt=grep&q=linuxtesting.org

Этого как правило достаточно. Также если долистать до конца, то можно
обратить внимание, что первый из них был принят лично Линусом
Торвальдсом в 2010 году.

Хорошилов Алексей