On Fri, Apr 5, 2013 at 9:41 AM, Rico Schüller <kgbric...@web.de> wrote:
> On 05.04.2013 09:13, Nozomi Kodama wrote: > >> s = 0.75f; >> - if ( order > 2 ) >> - s += 5.0f / 16.0f; >> - if ( order > 4 ) >> - s -= 3.0f / 32.0f; >> + if (order > 2) >> + s = 1.0625f; >> + if (order > 4) >> + s = 0.96875f; >> s /= D3DX_PI; >> > > Doesn't the compiler do that already? Most likely,since those are literal constants. Besides, 5.0f / 16.0f is more readable than a pre-computed result. So I don't think this patch is really useful. Frédéric