search

mgastner / cartogram-cpp

Cartogram generator in C++
MIT License
8 stars 4 forks source link

Arrange functions in header files by use rather than alphabetically #123

Closed adisidev closed 3 weeks ago

adisidev commented 2 years ago

This can be applied to inset_state.h and cartogram_info.h. Otherwise, looking at the header file does not easily convey what all the class's purpose is.

adisidev commented 3 weeks ago

After trying to do this, I have realised it is not easy to judge what should go where. This can be quite subjective.

Alphabetical ordering already has some implicit clustering (like putting all the write_s together. Perhaps, this is enough.