Discussion:
Changes in golded+ sources
(слишком старое сообщение для ответа)
golded+ inspector
2023-10-02 20:53:02 UTC
Permalink
===========================
replace malloc with new, fix warnings
Author: Vitaliy Aksyonov <***@i.ua>
Date: 2023-10-02 10:42:21 +0300
Committed by: GitHub <***@github.com>
Files:
M golded.spec
M golded3/gccfgg.cpp
M golded3/gccfgg4.cpp
M golded3/gckeys.cpp
M golded3/gclang.cpp
M golded3/gearea.cpp
M golded3/gecarb.cpp
M golded3/gecfgg.h
M golded3/gecmfd.cpp
M golded3/gedoss.cpp
M golded3/geinit.cpp
M golded3/geline.cpp
M golded3/gelmsg.cpp
M golded3/gemlst.cpp
M golded3/gemrks.cpp
M golded3/genode.cpp
M golded3/gepost.cpp
M golded3/geprot.h
M golded3/geqwks.cpp
M golded3/gerand.cpp
M golded3/geread.cpp
M golded3/geread2.cpp
M golded3/gesoup.cpp
M golded3/gesrch.cpp
M golded3/geusrbse.cpp
M golded3/gmarea.cpp
M golded3/gmarea.h
M goldlib/gall/gftnall.h
M goldlib/gall/gftnnlfd.cpp
M goldlib/gcfg/gedacfg.cpp
M goldlib/gcfg/gedacfg.h
M goldlib/gcfg/gs_max3.h
M goldlib/gcfg/gxareas.cpp
M goldlib/gcfg/gxezy102.cpp
M goldlib/gcfg/gxezy110.cpp
M goldlib/gcfg/gxfd.cpp
M goldlib/gcfg/gxfm092.cpp
M goldlib/gcfg/gxfm100.cpp
M goldlib/gcfg/gxfm116.cpp
M goldlib/gcfg/gximail4.cpp
M goldlib/gcfg/gximail5.cpp
M goldlib/gcfg/gximail6.cpp
M goldlib/gcfg/gxmax3.cpp
M goldlib/gcfg/gxqecho.cpp
M goldlib/gcfg/gxquick.cpp
M goldlib/gcfg/gxra.cpp
M goldlib/gcfg/gxsuper.cpp
M goldlib/gcfg/gxtmail.cpp
M goldlib/gcfg/gxwtr.cpp
M goldlib/gcfg/gxxbbs.cpp
M goldlib/gcfg/gxxmail.cpp
M goldlib/gcui/gwinmenu.cpp
M goldlib/gmb3/gmo_msg.h
M goldlib/gmb3/gmoezyc3.cpp
M goldlib/gmb3/gmoezyc4.cpp
M goldlib/gmb3/gmohuds.h
M goldlib/gmb3/gmosmb1.cpp
M goldlib/gmb3/gmosqsh2.cpp
M goldlib/gmb3/gmosqsh3.cpp
M goldlib/gmb3/gmosqsh4.cpp
M goldlib/gmb3/gmowcat3.cpp
M goldlib/gmb3/gmowcat4.cpp
M goldlib/gmb3/gmoxbbs3.cpp
M goldlib/gmb3/gmoxbbs4.cpp
M goldlib/hunspell/affixmgr.cxx
M goldlib/hunspell/affixmgr.hxx
M goldlib/hunspell/csutil.cxx
M goldlib/hunspell/hunspell.cxx
M goldlib/hunspell/suggmgr.cxx
M goldlib/hunspell/suggmgr.hxx
M goldlib/uulib/uuint.h
M goldlib/uulib/uunconc.cpp
M goldlib/uulib/uuscan.cpp
M goldlib/uulib/uuutil.cpp
M goldnode/goldnode.cpp
M srcdate.h
Nil A
2023-10-02 22:45:14 UTC
Permalink
Hello, golded+!

Monday October 02 2023 23:53, from golded+ inspector -> All:

gi> replace malloc with new, fix warnings
gi> Author: Vitaliy Aksyonov <***@i.ua>
gi> Date: 2023-10-02 10:42:21 +0300
gi> Committed by: GitHub <***@github.com>

Во-первых, я не вижу в этом гитхаб интерфейсе простую вещь, кто заапрувил
пулреквест, и ваще надо бы правило написать, что там два человека делают палец
вверх кодревью, после чего можно мёрджить. Ровно поэтому я не зареган на
гитхабе - странный интерфейс. Ну так то я по работе там имел два аккаунта,
чтобы в опен-сорц отдать, но их форкфлоу немного странно.
79 files changed, 483 insertions(+), 476 deletions(-
Во-вторых, это не ident или clang-format -style=... вызов, чтобы вот так вот
просто поверить наслово, при отсутствии хоть каких-то юниттестов.

Best Regards, Nil
Vitaliy Aksyonov
2023-10-02 18:59:50 UTC
Permalink
Привет, Nil!
79 files changed, 483 insertions(+), 476 deletions(-
NA> Во-вторых, это не ident или clang-format -style=... вызов, чтобы вот
NA> так вот просто поверить наслово, при отсутствии хоть каких-то
NA> юниттестов.

Ты чё, не веришь джентельменам на слово? ;)
Дык, код доступен. Посмотри мои изменения, если что не нравится - обсудим.

Best regards,
Vitaliy Aksyonov.

... А help читать - не барское занятье?
Michael Dukelsky
2023-10-03 19:02:52 UTC
Permalink
Привет, Nil!

03 October 2023 01:45, Nil A послал(а) письмо к golded+ inspector:

gi>> replace malloc with new, fix warnings
gi>> Author: Vitaliy Aksyonov <***@i.ua>
gi>> Date: 2023-10-02 10:42:21 +0300
gi>> Committed by: GitHub <***@github.com>

NA> Во-первых, я не вижу в этом гитхаб интерфейсе простую вещь, кто
NA> заапрувил пулреквест,

Запрос на включение (pull request) одобрил (approved) я. Почему гитхаб
подсовывает себя, не знаю. Для печати содержимого поля "Commited by:" я
использую git log --format="... %cn ...".

NA> и ваще надо бы правило написать, что там два
NA> человека делают палец вверх кодревью, после чего можно мёрджить.

Два человека для проверки кода - это в наших условиях слишком много. И
одного-то можно не набрать. Тут же есть не только С/С++, но и make и CMake. И
для таких проверок кода могут потребоваться разные люди. Правило написать
недолго. Кто будет код проверять? Ты будешь хотя бы C/C++? А кто будет
проверять make?

NA> Ровно
NA> поэтому я не зареган на гитхабе - странный интерфейс. Ну так то я по
NA> работе там имел два аккаунта, чтобы в опен-сорц отдать, но их форкфлоу
NA> немного странно.

В проверке кода могут участвовать только зарегистрированные пользователи,
имеющие право записи в данный репозиторий.

Желаю успехов, Nil!
За сим откланиваюсь, Michael.

... node (at) f1042 (dot) ru
Michael Dukelsky
2023-10-03 19:10:54 UTC
Permalink
Привет, Nil!

03 October 2023 22:02, Michael Dukelsky послал(а) письмо к Nil A:

NA>> Во-первых, я не вижу в этом гитхаб интерфейсе простую вещь, кто
NA>> заапрувил пулреквест,

Посмотреть, кто это сделал, можно так. На странице репозитория нажать Pull
requests, потом нажать на Closed. Появится список закрытых запросов на
включение. Нажать на название нужного запроса. Откроется запрос и рядом с
сиреневым словом Merged будет написано, кто влил эти коммиты.

Желаю успехов, Nil!
За сим откланиваюсь, Michael.

... node (at) f1042 (dot) ru
Nil A
2023-10-03 19:44:04 UTC
Permalink
Hello, Michael!

Tuesday October 03 2023 22:10, from Michael Dukelsky -> Nil A:

NA>>> Во-первых, я не вижу в этом гитхаб интерфейсе простую вещь, кто
NA>>> заапрувил пулреквест,

MD> Посмотреть, кто это сделал, можно так. На странице репозитория нажать
MD> Pull requests, потом нажать на Closed. Появится список закрытых
MD> запросов на включение. Нажать на название нужного запроса. Откроется
MD> запрос и рядом с сиреневым словом Merged будет написано, кто влил эти
MD> коммиты.

Ооо, спасибо. А то мы вдвоём с Виталиком не смогли разгадать гитхабовский
интерфейс.

Best Regards, Nil

Loading...