unsigned int

Unsigned short is maybe the value range is to small for big meshes. Should be changed to Unsigned int.

I think this cloud be deleted. If you want to keep it, move it into RAS_MeshObject.cpp in RAS_MeshObject::EndConversion().

Look like that GetVertexOffset() is now unused, because GetVertexOffsetAbsolute() is now used instead. One of the methods should be deleted.
I prefer to keep GetVertexOffset() and modify the code to return the absolute offset.

I think that it's useful to keep the function GetVertexOffset(), It can be used later for mesh generation ect …

You need to change the iterator to unsigned int too.

Note:
I don't made a comment on this line because it was obviously for me that this need to be changed too. I don't want to make a comment for every line that need to be changed too.

GetVertexOffset() is now not unused anywhere. One of the methods should be deleted.
I prefer to delete GetVertexOffsetAbsolute() and keep GetVertexOffset() and modify the code to return the absolute offset (return m_offset[i] + m_darray->m_offset;). Then the modifications in KX_PolyProxy.cpp are not necessary.
Don't forget to change the the return type to unsigned int .

unsigned int for the return type.

why ? the 'i' value is the current vertex array index, it can't be more than 2 (Line, Tri, Quad).

What do you think to rewrite the function ? :
GetVertexOffset(int index, bool absolute=true)

I am sorry, I don't checked what the maximum value of m_displayArrays is. In that case you can leave unsigned short.

I think we don't need to do this. There is actually no usage for getting a non absolute vertex offset.
We should not keep unused or add unused functionality to methods.

int RAS_Polygon::GetVertexOffset(int i)
{
    return m_offset[i] + m_darray->m_offset;
}

You can also keep your new GetVertexOffsetAbsolute(), and delete the old one, if you like the new method name more.

lists not listes

display

avoid doing cleanup mixed with functional changes. (not ultra important, though. for next time)

To be clear : i prefer keep the name "GetVertexOffsetAbsolute" and not "GetVertexOffset" because we have a setter named "SetVertexoffset" and a getter with the same name must be return the same value, but here is not the case.