Fix/Refactor of neighbor array populating (issue #18)

[ootsby]

I tacked on a fix for application of the visual heuristic and removed an unnecessary (AFAICT) loop calling addNeighbors in the BSP mapgen.

[Hyftar]

Try squashing your last commit when fixing an existing commit instead of adding another commit. In this case this should be fairly straightforward; Just follow this tutorial.

Once you're done, just do a forced update using git push origin master -f (assuming origin is the name of your forked repo).

I think keeping (or at least striving to keep) a clean git history is quite important.

[ootsby]

Thanks for the pointer. However, as far as I'm aware, the recipient has the option to squash a PR when they take it if they wish. Alternatively they can bounce it if there are issues with the automatic process and request a squash then.

[shiffman]

Thanks for this!