Nп/п : 63 из 87
От : Vitaliy Aksyonov 1:104/117 05 мар 24 22:24:28
К : Michael Dukelsky 05 мар 24 08:28:01
Тема : Re: strlen
----------------------------------------------------------------------------------
@MSGID: 1:104/117 65e7fe4f
@REPLY: 2:5020/1042 65e77b23
@CHRS: CP866 2
@TZUTC: -0700
@TID: hpt/lnx 1.9 2022-07-03
Привет, Michael!
05 Mar 24 23:00, ты писал(а) мне:
VA>>> Залил очередной патч. Получил пачку вопросов от Codacy. Починил
VA>>> почти все. Остался вопрос с strlen.
VA>>> Codacy считает его опасным и вся инспекция не проходит, но при
VA>>> этом они пишут, что эта проверка экспериментальная.
VA>>> Один из вариантов поправить - передать в ту функцию std::string.
VA>>> Может у кого есть мысли получше? :)
VA>> Что делать с этой дурацкой проверкой? В эхотаге миллион вызовов
VA>> strlen. Если Codacy будет возбуждаться на них - придется
VA>> переписать всю работу со строками. Я понимаю, если бы оно
VA>> корректно ловило случаи, когда функция вызывается на буффере, где
VA>> может не быть `\0`, но в данном конкретном случае там абсолютно
VA>> валидный строковый литерал, который гарантированно содержит `\0`.
VA>> Может выключить эту проверку? Или сделать её некритичной?
MD> Если есть возможность сделать её некритичной, то, конечно, можно так и
MD> сделать, чтобы не пропустить действительно нужное предупреждение. Я не
MD> знаю, как в Codacy относятся к письмам разработчиков. Если так же, как
MD> в Microsoft, то писать туда бесполезно. Но можно попробовать, вдруг
MD> там вменяемые люди.
Переделал, чтобы не возмущался. Но это будет проблемой в будущем.
Эхотаг не так просто избавить от использования этой функции. Я покопаюсь в
настройках, может можно что-то сделать.
Best regards,
Vitaliy Aksyonov.
... Объявление в общественном мyжском тyалете: "Мимо писсyаров не СРАТЬ!!!".
--- GoldED+/LNX 1.1.5-b20240305-beta
* Origin: Aurora, Colorado (1:104/117)
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: 104/117 5020/1042 4441