Nп/п : 19 из 19
От : Michael Dukelsky 2:5020/1042 06 мар 24 12:40:46
К : Nil A 06 мар 24 12:53:02
Тема : fidoconf содержит дубликат subscribeCheck()
----------------------------------------------------------------------------------
@MSGID: 2:5020/1042 65e83ccd
@REPLY: 2:5015/46 65e7b583
@CHRS: CP866 2
@TZUTC: 0300
@RealName: Михаил Дукельский
@TID: hpt/lnx 1.9 2023-02-24
Привет, Nil!
06 March 2024 02:15, Nil A послал(а) письмо к All:
NA> Зачем так писать, когда у нас есть функция с одинаковым именем,
NA> одинаковыми параметрами, и делающая примерно одно и тоже (возвращает
NA> 1, 2, 3, иногда просто 2 и 2, за что тоже надо бы руки оторвать)
NA> ? fidoconf/src/afixcmd.c: subscribeCheck(s_area * area, s_link * link)
NA> fidoconf/src/linked.c: int subscribeCheck(s_area area, s_link * link)
NA> А вы уверены, что нужную функцию линкер подставит?
По поводу того, что функция возвращает какие-то числа вместо enum.
Такого говнокода во всём проекте полно.
Что касается двух одинаковых функций. linked - это отдельное
приложение с кодом в одном файле linked.c. Заголовочный файл afixcmd.h там не
включается. Так что линкер другую функцию не подставит. По-видимому, не хотели
тащить в linked.c всё остальное из afixcmd.h, поэтому повторили одну
функцию. Наверно, можно было выделить эту функцию в отдельный файл и включать
его там, где надо.
Если сделаешь патч, будет здорово.
Желаю успехов, Nil!
За сим откланиваюсь, Michael.
... node (at) f1042 (dot) ru
--- GoldED+/LNX 1.1.5-b20231008
* Origin: ==<<.f1042.ru.>>== (2:5020/1042)
SEEN-BY: 50/109 104/117 301/1 341/66 460/58 463/68
467/888 4500/1 5000/111
SEEN-BY: 5001/100 5005/49 5015/42 46 5020/620 715
830 846 848 1042 4441 12000
SEEN-BY: 5025/121 5030/49 115 1081 5053/51 5054/8
5058/104 5059/37 5061/133
SEEN-BY: 5083/1 444 6090/1
@PATH: 5020/1042 4441