• fidoconf содержит дубликат subscribeCheck()

    From Nil A@2:5015/46 to All on Wed Mar 6 02:15:18 2024
    Hello, All!

    Зачем так писать, когда у нас есть функция с одинаковым именем, одинаковыми параметрами, и делающая примерно одно и тоже (возвращает 1, 2, 3, иногда просто 2 и 2, за что тоже надо бы руки оторвать) ?
    fidoconf/src/afixcmd.c: subscribeCheck(s_area * area, s_link * link) fidoconf/src/linked.c: int subscribeCheck(s_area area, s_link * link)
    А вы уверены, что нужную функцию линкер подставит?

    Как минимум, можно в fidoconf/src/linked.c объявить функцию static, чтобы на там и осталась, но лучше таки разобраться.

    Best Regards, Nil
    --- GoldED+/LNX 1.1.5
    * Origin: Linux 2.6.32-042stab145.3 (2:5015/46)
  • From Michael Dukelsky@2:5020/1042 to Nil A on Wed Mar 6 12:40:46 2024
    Привет, Nil!

    06 March 2024 02:15, Nil A послал(а) письмо к All:

    Зачем так писать, когда у нас есть функция с одинаковым именем, одинаковыми параметрами, и делающая примерно одно и тоже (возвращает
    1, 2, 3, иногда просто 2 и 2, за что тоже надо бы руки оторвать)
    ? fidoconf/src/afixcmd.c: subscribeCheck(s_area * area, s_link * link) fidoconf/src/linked.c: int subscribeCheck(s_area area, s_link * link)
    А вы уверены, что нужную функцию линкер подставит?

    По поводу того, что функция возвращает какие-то числа вместо 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)