Menu

#116 CBL_REF::LUT_8u incorrect

1.3-dev
open-fixed
Ramesh
5
2009-02-04
2009-01-30
Anonymous
No

I'm not sure if this is the correct place to post this, but yeah, the function CBL_REF::LUT_8u looks like this:

CBL_INLINE void CBL_REF::LUT_8u( __m128i & reg, U8* table )
{
XMM128 & r = reinterpret_cast<XMM128&>(reg);
for (int i = 0; i < 16; i++)
r.u8[0] = table[r.u8[0]];
}

but should probably look like this:

CBL_INLINE void CBL_REF::LUT_8u( __m128i & reg, U8* table )
{
XMM128 & r = reinterpret_cast<XMM128&>(reg);
for (int i = 0; i < 16; i++)
r.u8[i] = table[r.u8[i]];
}

Nothing major, but it's bound to frustrate someone eventually. Thanks!

-Scott

PS Oh.. well my e-mail is popsoftheyear@yahoo.com but I don't need notified. Thanks though.

Discussion

  • Ravi Korsa

    Ravi Korsa - 2009-02-04

    It's corrected. Thanks for pointing this out.

     
  • Ramesh

    Ramesh - 2009-02-04
    • assigned_to: nobody --> ravi-korsa
     
  • Ravi Korsa

    Ravi Korsa - 2009-02-04
    • status: open --> open-fixed
     
  • Ravi Korsa

    Ravi Korsa - 2009-02-04
    • assigned_to: ravi-korsa --> jrameshbe
     

Log in to post a comment.