Nп/п : 8 из 10
От : Vitaliy Aksyonov 1:104/117 08 апр 25 07:14:14
К : Nil A 08 апр 25 16:22:02
Тема : Re: Pull request
----------------------------------------------------------------------------------
@MSGID: 1:104/117 67f52174
@REPLY: 2:5015/46 67f439d9
@CHRS: CP866 2
@TZUTC: -0600
@TID: hpt/lnx 1.9 2024-03-02
Привет, Nil!
07 Apr 25 23:45, ты писал(а) мне:
VA>> Народ, я отправил небольшой PR, который добавляет вычитку кладжа
VA>> CHRS в базах Synchronet. Кто-то может сделать review?
NA> Вроде визуально проблем нет, патч короткий.
NA> 1. Конечно там простынки одинаковые, может хотя бы на макрос в Си
NA> стиле заменить, если нельзя какой-то инлайн функцию написать?
Лучше, конечно, заменить на функцию. Но лень делать рефакторинг.
Может в другой раз. Там ещё был запрос на TZUTC.
NA> 2. Ругань на sprintf. У кого-то не соберётся, если использовать
NA> безопасный snprintf? Watcom C?
По-хорошему, это надо выпиливать очень много где. Но... см. п. 1.
Best regards,
Vitaliy Aksyonov.
... Ищу психиатра для работы с C++.
--- GoldED+/LNX 1.1.5-b20240309
* Origin: Aurora, Colorado (1:104/117)
SEEN-BY: 104/117 221/6 460/58 463/68 5000/111
5001/100 5015/46 255 5019/40
SEEN-BY: 5020/715 848 1042 4441 12000 5023/24
5030/49 1081 5053/58 5058/104
SEEN-BY: 6078/80
@PATH: 104/117 5020/1042 4441