convert limb access in seWedding*.tw
I am mentioning it here because it was an extreme case, but the basic problem pops up all over the place:
Lines that get longer and more complicated because of ever growing if-else constructs. In this case it was so extreme that there was even a comment above it wishing good luck reading this
, at which point I am wondering why it was still one line and not distributed over multiple lines, because once I did that it became pretty easy to read and the logic errors in it became obvious, whereas they where impossible to spot before.
I know that it was likely not just one contributor, but instead grown over time(and in some places I might have contributed to it myself) but I still wanted to mention it.
Merge request reports
Activity
After a point, it may be better to break text blocks like that down into overarching if-elses instead of in-line ones. I will agree that that is the biggest concern about this overhaul.
Another option is to see if any of those condition outcomes happen repeatedly. If they do, you may want to consider replacing it with a variable and just setting that variable once.
Edited by Pregmoddermentioned in commit 61f317d1