Closed Bug 388342 Opened 18 years ago Closed 18 years ago

imglib warning when loading jpeg images - WARNING: NS_ENSURE_TRUE(index < static_cast<PRUint32>(mFrames.Count()))

Categories

(Core :: Graphics: ImageLib, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: tor, Unassigned)

Details

Attachments

(1 file)

The patch that added APNG support added a NS_ENSURE_ARG to imgContainer::GetFrameAt(). Unfortunately the JPEG decoder triggers that because it always tries to get frame 0 (for mixed-multipart?), so using a debug build for dogfooding can get chatty at times with all the warnings that line emits. This patch allows requesting in a container that doesn't yet contain a frame, and makes sure GetFrameAt always sets the return properly.
Attachment #272541 - Flags: review?(pavlov)
Attachment #272541 - Flags: review?(pavlov) → review+
Checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: