I think it was already called vSwap in the backends, that might just be for
consistency.


On Fri, Feb 14, 2014 at 8:55 PM, Tony Wasserka <notifications@github.com>wrote:

> In Source/Core/VideoBackends/D3D/Render.cpp:
>
> > @@ -725,7 +724,7 @@ void formatBufferDump(const u8* in, u8* out, int w, int h, int p)
> >  }
> >
> >  // This function has the final picture. We adjust the aspect ratio here.
> > -void Renderer::Swap(u32 xfbAddr, u32 fbWidth, u32 fbHeight,const EFBRectangle& rc,float Gamma)
> > +void Renderer::vSwap(u32 xfbAddr, u32 fbWidth, u32 fbHeight,const EFBRectangle& rc,float Gamma)
>
> Whatever that "v" is supposed to tell me (I guess it was just the quickest
> way to give it a name different than "Swap"?), please don't use function
> names like that. It's just confusing, especially since you could just take
> the fractions of a second to type a more descriptive name.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/dolphin-emu/dolphin/pull/37/files#r9761566>
> .
>



-- 
Pierre "delroth" Bourdon <delroth@gmail.com>
Software Engineer @ Zürich, Switzerland
http://code.delroth.net/