FPSCounter: Flush the logs every second and close them when the renderer is shut down.

We also move `FPSCounter::Update()` to RenderBase, where it is called after `SwapImpl()`, does this affect our results?
Expanding with more logging info should be done in the FPSCounter class,
no? It would be unnecessary to inherit just for that, I think.
On Jul 16, 2014 3:06 PM, "Jules Blok" <notifications@github.com> wrote:

> In Source/Core/VideoCommon/FPSCounter.h:
>
> > @@ -4,12 +4,32 @@
> >
> >  #pragma once
> >
> > -namespace FPSCounter
> > +#include <fstream>
> > +
> > +#include "Common/Timer.h"
> > +
> > +class FPSCounter
>
> Maybe you want to expand it with more logging information? I normally
> reserve final for situations where inheriting the class would have
> unintended consequences.
>
> If you think it should be final I'll make it final though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dolphin-emu/dolphin/pull/595/files#r15020356>.
>