Adapt MxRegion.h

[foxtacles]

This adapts our MxRegion implementation to the recently surfaced MxRegion.h file. It gives proper names to entities and merges the MxRegionCursor and everything from MxRegionList into MxRegion.

We can wait until we have the "entropy" detection in place to verify nothing breaks, although in this case the entropy seems relatively low.

[jonschz]

Isn't this a TEMPLATE?

[foxtacles]

Technically it's both synthetic and template. I'm not sure if we decided what takes preference in cases like these, I think it's relatively inconsistent in the code base so far

[disinvite]

Can remove the if here for a better beta match.

[disinvite]

If you change this to if (a.HasMatch()) and reverse the if/else blocks, it matches the beta better. No apparent effect on lego1 though.

[disinvite]

Looks good! I haven't found any regressions so far.

Some more beta addrs here if you want to add them. Not all should be comment annotations, I just did it that way for simplicity:

// FUNCTION: BETA10 0x10148fe8
// MxRegion::~MxRegion

// FUNCTION: BETA10 0x10149535
// MxRegion::Intersects

// FUNCTION: BETA10 0x1014aa46
// MxSpan::IntersectsH

// SYNTHETIC: BETA10 0x1014b0b0
// MxSpan::`scalar deleting destructor'

// FUNCTION: BETA10 0x1014b0f0
// MxSpan::~MxSpan

// FUNCTION: BETA10 0x1014b1d0
// MxRegion::IsEmpty

// TEMPLATE: BETA10 0x1014b210
// MxList<MxSpan *>::GetNumElements

// SYNTHETIC: BETA10 0x1014b230
// MxRegion::`scalar deleting destructor'

// FUNCTION: BETA10 0x1014b3b0
// MxSpan::GetMin

// FUNCTION: BETA10 0x1014b3f0
// MxSpan::GetMax

// FUNCTION: BETA10 0x1014b910
// MxSegment::GetMin

// FUNCTION: BETA10 0x1014b930
// MxSegment::GetMax

// FUNCTION: BETA10 0x1014c360
// MxSegment::MxSegment

// TEMPLATE: BETA10 0x1014c910
// ?Next@?$MxListCursor@PAVMxSegment@@@@QAEEXZ