[OpenJDK 2D-Dev] Review Request for JDK-6386906 : Faulty rounding code in BMPImageReader.decodeRLE4()

[Jim Graham]

Why use ceil() at all?  If you want Math.ceil(intval/2.0f), use 
"((intval + 1) / 2)"...

			...jim

On 6/13/2016 7:00 AM, Jayathirth D V wrote:
> Hi,
>
>
>
> _Please review the following fix in JDK9:_
>
> _ _
>
> Bug : https://bugs.openjdk.java.net/browse/JDK-6386906
>
>
>
> Webrev : http://cr.openjdk.java.net/~jdv/6386906/webrev.00/
>
>
>
> Issue : We are dividing an integer with another integer and trying to
> perform ceil() on resultant value which will be an integer. Resultant
> value will not have fractional part and ceil() operation is invalid.
>
>
>
> Solution : Update the code to divide the integer with floating point
> variable so that ceil() operation becomes valid.
>
>
>
> Thanks,
>
> Jay
>
>
>