Open Bug 688978 Opened 13 years ago Updated 2 years ago

cairo_surface_wrapper_flush function incomplete

Categories

(Core :: Graphics, defect)

Product:
Core
Component:
Graphics
Platform:
x86
All
Type:
defect

Tracking

()

Status:
UNCONFIRMED

People

(Reporter: mark, Unassigned)

Details

Compiling Firefox from source on Windows I noticed a warning thrown by MSVC during the PGO linking phase:
c:\mozdev\src\gfx\cairo\cairo\src\cairo-surface-wrapper.c(720) : warning C4715: '_cairo_surface_wrapper_flush' : not all control paths return a value

The function in question is:
cairo_status_t
_cairo_surface_wrapper_flush (cairo_surface_wrapper_t *wrapper)
{
    if (wrapper->target->backend->flush) {
	return wrapper->target->backend->flush(wrapper->target);
    }
}

This seems to be incomplete because when the if statement is false, the function doesn't return anything, leaving the state undetermined -- was this intended? What will be done with the undetermined state? Is this a potential cause for instability?
Summary: cairo_surface_wraper_flush function incomplete → cairo_surface_wrapper_flush function incomplete
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.