FidoGate fbox & binkd

Zhenja Kaliuta написал(а) к Sergey Poziturin в Mar 21 12:53:04 по местному времени:

Нi, Sergey!

On Mon, 01 Mar 2021 11:57:42 +0300 Sergey Poziturin writes:

ZK>>>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>>>> пользуется ;) Активнее!!
SA>>>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP>>> Чо в портах было в 2002 году, то и взял. Чо вы :))
ZK>> если кроме шуток, то он сабжи часто режет в необычных местах.
SP> Знаю, в планах пересобрать чо посвежее.

Если что, я рядом. Плюс irc или мессенджер какой.

Там вроде как фундаментальных несовместимостей я не натворил, но ключики
в areas немного поменяли смысл.

--- Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Zhenja Kaliuta написал(а) к Andrei Dzedolik в Mar 21 18:01:24 по местному времени:

Нi, Andrei!

On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:

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

SA>> Ты главное проверь чтоб работало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.

AD> Поправил / потестил у себя - полет нормальный, работает и как было
AD> и как стало :) Pull request ушел в github.


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


--- Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

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

Zhenja Kaliuta написал(а) к Andrei Dzedolik в Mar 21 18:47:01 по местному времени:

Нi, Andrei!

On Tue, 02 Mar 2021 17:20:48 +0100 Andrei Dzedolik writes:

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

AD> [... skip ...]
ZK>> А это, на коммент ответишь? Ну я могу и сам поправить.
AD> Давай здесь обсудим, так сказать публично :)

Так публично ж, https://github.com/ykaliuta/fidogate/pull/14

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

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

Угу. cfgetstring() в начале всех утилит как пример.
И в пример добавить (configs/fidogate.conf.sample.in), и в доку.

PS: эх, советы раздавать не лень :))

--- Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Andrei Dzedolik написал(а) к Zhenja Kaliuta в Mar 21 18:46:52 по местному времени:

Greetings, traveler.

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

ZK> Угу. cfgetstring() в начале всех утилит как пример.
ZK> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.

Понял-принял, как сделаю - отпишусь :)

ZK> PS: эх, советы раздавать не лень :))
;)

\aID

--- Msged/ 6.1.2

Andrei Dzedolik написал(а) к Zhenja Kaliuta в Mar 21 22:35:46 по местному времени:

* Originally by: Andrei Dzedolik (2:463/1331.1), 02 Mar 21 18:46.

Greetings, traveler.

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

ZK>> Угу. cfgetstring() в начале всех утилит как пример.
ZK>> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.

AD> Понял-принял, как сделаю - отпишусь :)

Переделал с учетом замечаний, создал новый pull. Старый закрыл. сорри за путаницу, я не заметел, как оно/я его обновил новым коммитом - так не было задумано :)


\aID

--- Msged/ 6.1.2