Joseph Walton-Rivers authored 3 years ago and Joseph Walton-Rivers committed 3 years ago

This is a merged commit of a bunch of typos and whatnot.

This is only a first approxmiation of what I would like for CI. Ideally,
it'd be it's own step that can pass/fail regardless of the build status.
With this setup, it's integrated into the build (and also running tidy
over our vendored dependencies, which are outside our scope).

Something like this: http://www.stablecoder.ca/2018/10/30/full-cmake-helper-suite.html
might be better for the future...

Mesh now has a (fairly primative) method for removing unused and
duplicated verticies. This means its possible to use conbined
geometry without duplicate verticies.

In my test scene, this halved the number of verticies in the combined
mesh.

There are plenty of other optimisations that could be implemented. Some
that would be nice to have but I don't have time for:

* Removal of internal faces in combined meshes
* Optimise triangle order so GL_TRIANGLE_STRIP or the like can be used

The lighting bug in the previous commit was caused by the fact that the
combined (averaged) normals don't work if there should be a clear seam
this commit duplicates each vertex.

* ditch has 3 sides high and one low
* point has 1 side high and three low

This adds to the two versions already present:

* slope has 2 sides high and 2 sides low
* cube is just a cube (4 sides high)

* The planes and cube are from first priciples so probably don't follow
  best practice with regards to ordering
* Renderer includes some matrixes for testing (spin objects, fixed prespective)
* Model renderer uses backface culling (to check winding order of
  polygons