I tried to clean up my code (see attached r21T2). Essentially I moved functions around.
EDIT: second version r21T2 is more compliant to OTTD coding style (and it has better variables names)
3 files are concerned:
- smallmap_colours.h (new file)
- smallmap_gui.cpp (now smaller)
- viewport.cpp (now bigger)
But I'm not sure what's best: in viewport.cpp, I use extern variables to access things declared in smallmap_gui.cpp.
Should I move these variables to smallmap_gui.h instead?
adf88 wrote:Does the patch break network compatibility?
AFAIK it breaks nothing related to multiplayer.
adf88 wrote:Could you publish some older version which is closer to OpenTTD 1.1.4?
I certainly can do that
See zoom64_history_r1_to_r21.zip
adf88 wrote:I have also a suggestion: it would be nice if we could control industries visibility like in the map window.
The viewport map is already linked to the smallmap controls that show/hide specific industries. I may try to do a toolbar not tied to the smallmap...