GMSHInterface: _include_stations_as_constraints
FileIO/Legacy/MeshIO.h: _edge_length[]

Shuffle and repeat tests on Travis

[T] Replace static var in tests with an outer def.

Running this test multiple times using
--gtest_repeat caused an error because of the
static modifier.

I would suggest not to use static in tests,
because the state of the test depends on execution
order.

Static analysis fixes part 1
  

Optimize element neighbor computation

http://www.viva64.com/en/d/0278/print/

http://www.viva64.com/en/d/0216/print/

On 32-bit/64-bit platform, structure size can be reduced by rearranging the fields according to their sizes in decreasing order.

http://www.viva64.com/en/d/0150/print/

http://www.viva64.com/en/d/0212/print/

http://www.viva64.com/en/d/0115/print/

There is no sense in testing the pointer against null, as the memory was allocated using the 'new' operator.

http://www.viva64.com/en/d/0293/print/

http://www.viva64.com/en/d/0254/print/

http://www.viva64.com/en/d/0183/print/

http://www.viva64.com/en/d/0170/print/

http://www.viva64.com/en/d/0146/print/

http://www.viva64.com/en/d/0097/print/

http://www.viva64.com/en/d/0303/print/

http://www.viva64.com/en/d/0294/print/

http://www.viva64.com/en/d/0165/print/

http://www.viva64.com/en/d/0274/print/

http://www.viva64.com/en/d/0289/print/

Check only if there is a neighbor.

This is used when the face_id of the neigbor is known.