Closed Bug 1694680 Opened 3 years ago Closed 3 years ago

Fix qcms_data_create_rgb_with_gamma.

Categories

(Core :: Graphics: Color Management, defect)

defect

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: jrmuizel, Assigned: jrmuizel)

References

Details

Attachments

(1 file)

This fixes a number of problems:

  1. The check around get_rgb_colorants was inverted. This caused us to
    only continue if the colorants were wrong.

  2. get_rgb_colorants can just return the Matrix instead of taking
    a reference to it.

  3. The OOBs checks in write_u32 and write_u16 had their conditions
    inverted.

  4. No tests

This fixes a number of problems:

  1. The check around get_rgb_colorants was inverted. This caused us to
    only continue if the colorants were wrong.

  2. get_rgb_colorants can just return the Matrix instead of taking
    a reference to it.

  3. No tests

Pushed by jmuizelaar@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa296c490720
Fix qcms_data_create_rgb_with_gamma.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
Regressions: 1696819
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: