Looks good, will make creating new banks a lot easier.

My only question is why bankgen.py doesn't map 'S' to the C++ short type (line 27). Guessing there's a good reason, just curious.

Good point. We should set the types according to hipo::bank, perhaps with decltype, such as

decltype(std::declval<hipo::bank>().getInt(0, 0))

As an aside, here are the types (note there is some casting done internally):

I opened an issue in hipo for this: gavalian/hipo#71