Commit cd031ba2 authored by Jeff Muizelaar's avatar Jeff Muizelaar
Browse files

Bug 1676649 - Remove an unneeded cast and make build_colorant_matrix safe. r=aosmond 

Differential Revision: https://phabricator.services.mozilla.com/D96717
parent 9ea0149b

gfx/qcms/src/transform_util.rs

+2 −3
Original line number Diff line number Diff line
@@ -57,7 +57,7 @@ pub fn clamp_float(mut a: f32) -> f32 { 
    However, that version will let NaNs pass through which is undesirable

    for most consumers.

    */

    if a as f64 > 1.0f64 {

    if a > 1. {

        return 1.;

    } else if a >= 0. {

        return a;
@@ -249,8 +249,7 @@ pub fn build_input_gamma_table(mut TRC: Option<&curveType>) -> Option<Vec<f32>> 


    return Some(gamma_table);

}

#[no_mangle]

pub unsafe extern "C" fn build_colorant_matrix(mut p: &qcms_profile) -> matrix {

pub fn build_colorant_matrix(mut p: &qcms_profile) -> matrix {

    let mut result: matrix = matrix {

        m: [[0.; 3]; 3],

        invalid: false,