search

OpenGeoscience / vtkMap

vtkMap
23 stars 13 forks source link

Code cleanup 171113 #111

Closed johnkit closed 6 years ago

johnkit commented 6 years ago

This branch started out to add missing "override" declarations, based on warning messages in the nightly build on TalosIV. But it then prompted me to also remove some redundant code in vtkMapTile visibility, and some dead/unused "Bin" code in vtkMapTile.

alvarosan commented 6 years ago

Looks good @johnkit , +1. Remember to apply clang-format.

johnkit commented 6 years ago

Thanks for reminding me about clang-format. I thought I had your pre-commit hook installed, but apparently not. Too bad that clang-format inserts indent after macro calls, but at least it's consistent.