search

elfmz / far2l

Linux port of FAR v2
GNU General Public License v2.0
1.76k stars 174 forks source link

consider adding codefactor.io #829

Open unxed opened 3 years ago

unxed commented 3 years ago

https://www.codefactor.io/repository/github/elfmz/far2l/issues/

It looks pretty, shows clear and useful comments about any found issue, is free of charge for public repos and does automated PR reviews (!)

Upstream far already using it on github. It integrates the same way as Travis CI does.

unxed commented 3 years ago

Скормил этой штуке код своего форка putty, который с фаровскими экстеншнами. Забавно, она нашла кучу проблем в самой putty, и ни одной в экстеншнах! А я-то думал, это у меня там код кривой, и удивлялся ещё, что он работает и никто не жалуется. Да на фоне того, как устроен код самой putty (половина логики апликухи засунута в гигантскую WindowProc callback function, не зря мне это казалось странным), мою поддержку экстеншнов можно назвать идеальной, по ходу.

Не удивительно, что putty не развивается давным-давно фактически. Чтоб добавить туда экстеншны фара красиво, там всю архитектуру перефигачить надо (отделяя UI от логики, а логику от ввода/вывода, вот это вот всё), так что я, видимо, выбрал единственно возможный вариант, добавив их туда как угодно, лишь бы работало :) В итоге поддержка экстеншнов размазана между io и ui, но ведь бегает же!

singalen commented 3 years ago

If I may chime in, Far is kind of irrecoverable. With static analysis, I would go in this order: gcc warnings, clang warnings, clang-tidy, PVS-studio.