Показать сообщение отдельно
  #23  
Старый 02.03.2021, 20:52
Andrei Dzedolik
Guest
 
Сообщений: n/a
По умолчанию Re: FidoGate fbox & binkd

Andrei Dzedolik написал(а) к Zhenja Kaliuta в Mar 21 17:20:48 по местному времени:

Greetings, traveler.


AD>>>> Хоршо, github + я отпишусь сюда, как сделаю.

[... skip ...]

ZK> А это, на коммент ответишь? Ну я могу и сам поправить.

Давай здесь обсудим, так сказать публично :)

С 1. -- 4. я все понял, как будет возможность - переделаю и пришлю.

А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в fidogate.conf или как-то еще?

Комментарий для справки:

Could it be possible to change the patch a bit:

1. Reword the commit message: make a header (first line separated) mentioning part of the functionality (tick) and do it in imperative way;

2. I understsand that the original code is spaghetti, but for me it still looks better to use a helper to both not "repeat yourself" and isolate conditional compilation (but see note below);

3. There is streq() macro which makes code more readable due to absence of reverse condition;

4. May be it makes sense to strcat() the suffix after the base prepared? It makes obvious what is changed in the string so no need to parse printf format and arguments on reading the code;

5. I do not see any reason why it must be compile time configuration. Would the runtime option be better?


\aID

--- Msged/ 6.1.2
Ответить с цитированием