OpenSettlers#4
I really need to spend more time. Today I struggled a lot with bitstrings. And still didn’t find an answer for what I was looking for, but I created a map for command without redundant info. So no more flags, only command name, data and size.
After push thou I saw a better solution - I could do pattern match on functions, so I could eliminate switch. I get a lot of mess with all those maps. I think that rethinking/refactoring it should happen sooner than I initially thought.
Also “size” field seems a bit off. Additionally I don’t feel as comfortable enough with specification as I was expecting, so I guess I’ll spend some more time with it - this should also get me up to speed.
Written on April 23, 2017