----------------------------------------------------------------------------------
@MSGID: 2:5034/10.1 f955819f
@REPLY: 2:5015/46 65bbb3ae
@CHRS: CP866 2
@PID: wfido 0.0.1/a
@TID: FTN::Pkt 1.02
@Posted: 01 Feb 24 19:00:41
Hello, Nil!
NA> Как такие места найти?
NA> а). Во время код-ревью внимательно вникать
NA> б). Запустить под valgrind - он такие вещи отловит, но
программа выполняется оооочень медленно, потому что там виртуальная машина
работает
NA> в). Собрать с санитайзером адресов, gcc -fsanitize=address,
замедление программы будет небольшое, но будет, и тоже покажет проблемные места
во время выполнения программы
Да Женя примерно тож самое и говорил
https://5034.ru/wfido/?area=RU.QICO&message=9454434b1080a75acfaf6320dd455a42
Ну то есть менять можно но с умом, то есть не везде все подряд.
https://5034.ru/diff.log
^^^^^^ 37 раз встречается memmove, в принципе можно внести в
последнюю ветку, то там помимо этого еще много отличий,
косметику бы конечно как отфильтровать, поди какой-нить плагин в VS
умеет косметику править?
Типа:
- if ( isalpha( *buf )) {/* Skip day of week name "Mon, " */
+if ( isalpha( *buf ))/* Skip day of week name "Mon, " */
Очень сильно bink.c отличается например, остальное так себе,
относительно не много ftn.c, zmodem.c, nodelist.c, modem.c,
protfm.c. В новой версии побольше дебага добавлено. А так отличия в
принципе не сильно большие (ОТНОСИТЕЛЬНО).
Вот убрать бы из дифа мусор косметики (стили кодирования, разбивки
по строкам) и дифф бы сжался раз в 20-30,
тогда мои бы глаза не плакали кровью :(
Ну как бы 37 раз copy на move поменять это не трудно, но в вот
с остальным как быть хз, без дяди программиста нереал.
С наилучшими пожеланиями, Sergey Anohin.
--- wfido
* Origin:
https://5034.ru/wfido (2:5034/10.1)
SEEN-BY: 50/8 109 606 240/1120 301/1 341/66 455/19
463/68 467/888 469/122
SEEN-BY: 4500/1 5001/100 5005/49 5010/352 5015/42
46 5020/113 290 545 715 830
SEEN-BY: 5020/846 848 1042 4441 9696 12000
5022/128 5023/24 5030/49 115 1081
SEEN-BY: 5030/1900 5034/10 13 5036/26 5053/51 58
5054/8 5058/104 5061/133
SEEN-BY: 5075/128 5083/1 444
@PATH: 5034/10 13 5020/715 1042 4441