Subject: Re: Couple of questions

---

Hi Nix,
you're no programming person? But looks like it
At least you find your way in code and know how to compile something. And your solution for the sentry looks good (and you found all relevant places).

For the "next/prev unit" function, that combines vehicles and buildings: Maybe two functions "getNextUnit/getPrevUnit" would be better suited. Else it might be difficult to merge the buildings and the vehicles lists in a consistent way. When you put it in one function, you could for example first search the vehicles list, then when its end is reached continue with the buildings. Just an idea...

Do you plan to send your code as patch to someone who can put it in the svn? Or is it mainly for your own use?
Anyway I would like to look at that unit-cycle topic (I didn't use that feature myself before, but now I like it) and implement it completely (also the "done" functionality) when I have time for it. But if you do all the changes, it would be ok for me.

---