The macro PULL_HEIGHTMAP is bad

[no-lex]

This macro uses its arguments to change the control logic within the macro. It gets used twice over the span of three lines and can clearly be expanded without being repetitive (did i mention that the macro's control logic is modified by the arguments passed to it?).

Located in octaedit.cpp.