Generics changes in Grid

MountainRange / MULE

OSS Mule, Built To New Heights :sunrise_over_mountains:

GNU General Public License v3.0

5 stars 0 forks source link

Generics changes in Grid #58

Closed strburst closed 8 years ago

strburst commented 8 years ago

I've made some changes to the way Grid, VisualGrid, and WorldMap are defined to make it possible to use a PlainGrid implementation for generics. I'm sending this pull now because we could merge this soon so that we don't have to deal with JavaFX stuff while writing tests.

@jgkamat could you review 0266496 carefully? (Wow, a commit hash with only numbers; weird.)

This pull also contains miscellaneous changes like Javadoc cleanup; consult the commit messages.

strburst commented 8 years ago

Is this pull good to merge? Does anyone else have any last-minute feedback? @MountainRange/peaks

jgkamat commented 8 years ago

This LGTM, we should merge this.