Bug 53292 - [GTK] Fix compilation warning
Summary: [GTK] Fix compilation warning
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-28 02:39 PST by Alejandro G. Castro
Modified: 2011-01-28 06:36 PST (History)
4 users (show)

See Also:


Attachments
Proposed patch (1.24 KB, patch)
2011-01-28 02:55 PST, Alejandro G. Castro
xan.lopez: review-
Details | Formatted Diff | Diff
Proposed patch (1.24 KB, patch)
2011-01-28 04:14 PST, Alejandro G. Castro
xan.lopez: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alejandro G. Castro 2011-01-28 02:39:48 PST
After r75955 we need to cast the number of pages:

../../../Source/WebKit/gtk/webkit/webkitwebframe.cpp: En la función ‘void draw_page_callback(GtkPrintOperation*, GtkPrintContext*, gint, void*)’:
../../../Source/WebKit/gtk/webkit/webkitwebframe.cpp:771:44: aviso: comparación entre expresiones enteras signed y unsigned
Comment 1 Alejandro G. Castro 2011-01-28 02:55:06 PST
Created attachment 80434 [details]
Proposed patch
Comment 2 WebKit Review Bot 2011-01-28 03:58:56 PST
Attachment 80434 [details] did not build on gtk:
Build output: http://queues.webkit.org/results/7646102
Comment 3 Xan Lopez 2011-01-28 04:08:50 PST
Comment on attachment 80434 [details]
Proposed patch

You are missing the opening '(' it seems.
Comment 4 Alejandro G. Castro 2011-01-28 04:14:55 PST
Created attachment 80437 [details]
Proposed patch

Uploaded the wrong patch sorry.
Comment 5 Xan Lopez 2011-01-28 04:41:46 PST
Comment on attachment 80437 [details]
Proposed patch

r=me
Comment 6 Alejandro G. Castro 2011-01-28 06:36:06 PST
Landed http://trac.webkit.org/changeset/76940