harris.c desperately needs refactoring
ec429 opened this issue · 1 comments
ec429 commented
harris.c is over 5000 lines, a large chunk of which is GUI building (which ideally should be generated by something like the (unfinished, unreleased) atg builder 'atgen'). Also, its flow control is a mess, with large-scale units like run_raid and loader just being goto labels in a giant main().
This makes development work involve far too much scrolling around the file trying to find where something is implemented.
ec429 commented
This is pretty much done apart from the code generation bit (the GUI is still all hand-coded). harris.c currently clocks in at 371 lines, and while plenty of them don't really belong there, it's not a serious problem any more.