@MSGID: 2:5020/1042 65e77b23
@REPLY: 1:104/117 65e73805
@CHRS: CP866 2
@TZUTC: 0300
@TID: hpt/lnx 1.9 2023-02-24
Hello Vitaliy,

05 March 2024, Vitaliy Aksyonov wrote to Michael Dukelsky:

 VA>> Залил очередной патч. Получил пачку вопросов от Codacy. Починил
 VA>> почти все. Остался вопрос с strlen.

 VA>> Codacy считает его опасным и вся инспекция не проходит, но при
 VA>> этом они пишут, что эта проверка экспериментальная.

 VA>> Один из вариантов поправить - передать в ту функцию std::string.
 VA>> Может у кого есть мысли получше? :)

 VA> Что делать с этой дурацкой проверкой? В эхотаге миллион вызовов
 VA> strlen. Если Codacy будет возбуждаться на них - придется переписать
 VA> всю работу со строками. Я понимаю, если бы оно корректно ловило
 VA> случаи, когда функция вызывается на буффере, где может не быть `\0`,
 VA> но в данном конкретном случае там абсолютно валидный строковый
 VA> литерал, который гарантированно содержит `\0`. Может выключить эту
 VA> проверку? Или сделать её некритичной?

 Если есть возможность сделать её некритичной, то, конечно, можно так
и сделать, чтобы не пропустить действительно нужное предупреждение. Я не
знаю, как в Codacy относятся к письмам разработчиков. Если так же, как в
Microsoft, то писать туда бесполезно. Но можно попробовать, вдруг там вменяемые
люди.

Michael

... node (at) f1042 (dot) ru
--- GoldED+/LNX 1.1.5-b20231008
 * Origin: ==<<.f1042.ru.>>== (2:5020/1042)
SEEN-BY: 104/117 460/58 463/68 5005/49 5015/46 255
5019/40 5020/715 848 1042
SEEN-BY: 5020/4441 12000 5023/24 5025/121 5030/49
1081 5053/58 5058/104
SEEN-BY: 6078/80
@PATH: 5020/1042 4441