Bytes are signed in Java, so 137 is not what you want. Also,
Color.rgb() take ints, not bytes. So (byte) 137 cast as an int results
in rg == -119.
On Fri, May 22, 2009 at 1:09 PM, Gavin Aiken <gavin.ai...@imperial.ac.uk> wrote:
> Hey all,
>
> Before I submit this I just wanted to check it's not a java nuiance.
>
> The Color class can be used to extract rgb values from a pixel (32 bit int).
>
> I implemented a lookup table filter and used a byte to store each individual
> r g & b value to save a bit of space.
>
> Long story short;
>
> public void testColor(){
>
> int nr = (byte)0; // Simulation of lookup in array of byte values
> representing individual bands
> int ng = (byte)0;
> int nb = (byte)0;
> int nc = Color.rgb(nr, ng, nb);
>
> assertEquals(nr,Color.red(nc));
> assertEquals(ng,Color.green(nc));
> assertEquals(ng,Color.blue(nc));
>
> nr = (byte)0;
> ng = (byte)137;
> nb = (byte)0;
> nc = Color.rgb(nr, ng, nb);
>
> assertEquals(nr,Color.red(nc)); // FAIL HERE
> assertEquals(ng,Color.green(nc));
> assertEquals(nb,Color.blue(nc));
> }
>
> Failure in testColor:
> junit.framework.AssertionFailedError: expected:<0> but was:<255>
> at uk.ac.ic.doc.gea05.miffed.os.ColorTest.testColor(ColorTest.java:44)
>
>
> Any ideas?
>
> 137 is the magic number causing the error in my case, there may be more.
>
>
> Cheers!
>
> Gav
>
>
>
>
> >
>
--
Romain Guy
Android framework engineer
romain...@android.com
Note: please don't send private questions to me, as I don't have time
to provide private support. All such questions should be posted on
public forums, where I and others can see and answer them
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google
Groups "Android Developers" group.
To post to this group, send email to android-developers@googlegroups.com
To unsubscribe from this group, send email to
android-developers-unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/android-developers?hl=en
-~----------~----~----~----~------~----~------~--~---
