Bug 692294 - Android mupdf.c has two bad function calls
Summary: Android mupdf.c has two bad function calls
Status: RESOLVED FIXED
Alias: None
Product: MuPDF
Classification: Unclassified
Component: apps (show other bugs)
Version: unspecified
Hardware: Other other
: P4 major
Assignee: Robin Watts
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-21 15:04 UTC by Andrew Wyld
Modified: 2011-07-04 14:01 UTC (History)
1 user (show)

See Also:
Customer:
Word Size: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Wyld 2011-06-21 15:04:55 UTC
mupdf.c will not build as written.  It contains two function calls with the wrong number of variables: fz_new_pixmap_with_data has six but expects four, and fz_execute_display_list has three but expects four.

SPECULATIVE CAUSES

fz_new_pixmap_with_data's third and fourth parameters seem to be a width and a height, and the bad call seems to be passing x and y position and then width and height.

fz_execute_display_list expects a bounding box as its fourth parameter, and an object called bbox is present by the bad call.  It may be enough to insert this as the fourth parameter.
Comment 1 Robin Watts 2011-07-04 14:01:47 UTC
Fixed now. Thanks.