[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-awt-dev
Subject:    Re: <AWT Dev> Fwd: LinearRGBConverter bug
From:       Andrey Kuzmenko <andreykuzmenko () gmail ! com>
Date:       2016-01-18 14:21:47
Message-ID: CAFDwXmZmCSx7QF5-ve1VYSExZsqrcDtb=Ua-g+eecC39t=sFoA () mail ! gmail ! com
[Download RAW message or body]

True. Sorry, it's the code from GNU Classpath!

2016-01-18 12:37 GMT+00:00 Mario Torre <neugens.limasoftware@gmail.com>:

> I couldn't find a LinearRGBConverter in OpenJDK but indeed that name
> was somewhat familiar...
>
> I believe you want to contact the GNU Classpath project, this isn't an
> OpenJDK issue.
>
> Cheers,
> Mario
>
> 2016-01-18 13:01 GMT+01:00 Andrey Kuzmenko <andreykuzmenko@gmail.com>:
> > Hi guys, I think there is a bug in LinearRGBConverter class. In this
> method:
> >
> > public float[] toRGB(float[] in);
> >
> > The following lines:
> >
> > if (n <= 0.00304f)
> >   out[i] = in[0] * 12.92f;
> >
> > Should be:
> >
> > if (n <= 0.00304f)
> >   out[i] = n * 12.92f;
> >
> > (What it does now is propagate Red to all channels at the very low
> > intensity, which is not noticeable much, but nevertheless is a bug, I
> > think.)
> >
> > Cheers,
> > Andrey
> >
> >
>
>
>
> --
> pgp key: http://subkeys.pgp.net/ PGP Key ID: 80F240CF
> Fingerprint: BA39 9666 94EC 8B73 27FA  FC7C 4086 63E3 80F2 40CF
>
> Java Champion - Blog: http://neugens.wordpress.com - Twitter: @neugens
> Proud GNU Classpath developer: http://www.classpath.org/
> OpenJDK: http://openjdk.java.net/projects/caciocavallo/
>
> Please, support open standards:
> http://endsoftpatents.org/
>

[Attachment #3 (text/html)]

<div dir="ltr">True. Sorry, it&#39;s the code from GNU Classpath!</div><div \
class="gmail_extra"><br><div class="gmail_quote">2016-01-18 12:37 GMT+00:00 Mario \
Torre <span dir="ltr">&lt;<a href="mailto:neugens.limasoftware@gmail.com" \
target="_blank">neugens.limasoftware@gmail.com</a>&gt;</span>:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">I couldn&#39;t find a LinearRGBConverter in OpenJDK but \
indeed that name<br> was somewhat familiar...<br>
<br>
I believe you want to contact the GNU Classpath project, this isn&#39;t an<br>
OpenJDK issue.<br>
<br>
Cheers,<br>
Mario<br>
<div class="HOEnZb"><div class="h5"><br>
2016-01-18 13:01 GMT+01:00 Andrey Kuzmenko &lt;<a \
href="mailto:andreykuzmenko@gmail.com">andreykuzmenko@gmail.com</a>&gt;:<br> &gt; Hi \
guys, I think there is a bug in LinearRGBConverter class. In this method:<br> \
&gt;<br> &gt; public float[] toRGB(float[] in);<br>
&gt;<br>
&gt; The following lines:<br>
&gt;<br>
&gt; if (n &lt;= 0.00304f)<br>
&gt;     out[i] = in[0] * 12.92f;<br>
&gt;<br>
&gt; Should be:<br>
&gt;<br>
&gt; if (n &lt;= 0.00304f)<br>
&gt;     out[i] = n * 12.92f;<br>
&gt;<br>
&gt; (What it does now is propagate Red to all channels at the very low<br>
&gt; intensity, which is not noticeable much, but nevertheless is a bug, I<br>
&gt; think.)<br>
&gt;<br>
&gt; Cheers,<br>
&gt; Andrey<br>
&gt;<br>
&gt;<br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
pgp key: <a href="http://subkeys.pgp.net/" rel="noreferrer" \
                target="_blank">http://subkeys.pgp.net/</a> PGP Key ID: 80F240CF<br>
Fingerprint: BA39 9666 94EC 8B73 27FA   FC7C 4086 63E3 80F2 40CF<br>
<br>
Java Champion - Blog: <a href="http://neugens.wordpress.com" rel="noreferrer" \
target="_blank">http://neugens.wordpress.com</a> - Twitter: @neugens<br> Proud GNU \
Classpath developer: <a href="http://www.classpath.org/" rel="noreferrer" \
                target="_blank">http://www.classpath.org/</a><br>
OpenJDK: <a href="http://openjdk.java.net/projects/caciocavallo/" rel="noreferrer" \
target="_blank">http://openjdk.java.net/projects/caciocavallo/</a><br> <br>
Please, support open standards:<br>
<a href="http://endsoftpatents.org/" rel="noreferrer" \
target="_blank">http://endsoftpatents.org/</a><br> \
</font></span></blockquote></div><br></div>



[prev in list] [next in list] [prev in thread] [next in thread]