Make vertex loader testable

This branch contains some core refactorings and global dependencies removal to make VertexLoader testable. A sample unit test has been included, and while it does not test much (a simple position conversion test) it works fine, even testing JIT :)
I was generating more than 2 uids before, forgot to kill that var.


On Sat, Aug 2, 2014 at 2:55 AM, Tony Wasserka <notifications@github.com>
wrote:

> In Source/UnitTests/VideoCommon/VertexLoaderTest.cpp:
>
> > @@ -0,0 +1,110 @@
> > +#include <set>
> > +
> > +#include "Common/Common.h"
> > +#include "VideoCommon/DataReader.h"
> > +#include "VideoCommon/VertexLoader.h"
> > +
> > +// Needs to be included later because it defines a TEST macro that conflicts
> > +// with a TEST method definition in x64Emitter.h.
> > +#include <gtest/gtest.h>  // NOLINT
> > +
> > +TEST(VertexLoaderUID, UniqueEnough)
> > +{
> > +	std::set<VertexLoaderUID> uids;
>
> Being unfamiliar with GoogleTest, what is the point of this set? I only
> see two tests happening in this function, and the set seems mostly
> irrelevant to them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dolphin-emu/dolphin/pull/716/files#r15728395>.
>



-- 
Pierre "delroth" Bourdon <delroth@gmail.com>
Software Engineer @ Zürich, Switzerland
http://code.delroth.net/