search

japanoise / emsys

ersatz-emacs text editor
MIT License
3 stars 2 forks source link

Break up main.c #34

Open japanoise opened 1 month ago

japanoise commented 1 month ago

If I'm gonna work on emsys more I need to make it easier to work on, because life is short and I have other hobbies and spend an awful lot of time programming other things (day job).

One idea is to break up main.c. Currently it has kind of a mixed reason to exist, and is 2000 lines long which is painful to work with (a legacy of being based on kilo). Ideally, it would just have int main in it, but I think that's unrealistic. Here's some quick ideas as to how I could break it up:

Might have to put more shit in headers if I do this, though, argh.

japanoise commented 1 month ago

Would also like to document all the files in HACKING.md. Doing that always makes me feel like I've lowered the bus factor a little bit. emsys is not a complicated piece of code, but it is sloppily organized.

nicholascarroll commented 1 month ago

totally! Let me get a couple PRs through first because they are nearly ready

  1. 32 window scrolling.

  2. 19 visual mark

    also #33 which will be quick i might not have time until the weekend

nicholascarroll commented 1 month ago

Would also like to document all the files in HACKING.md. Doing that always makes me feel like I've lowered the bus factor a little bit. emsys is not a complicated piece of code, but it is sloppily organized.

yes please!