RESOLVED FIXED 185556
[GTK] Silence GCC 8 warnings 
https://bugs.webkit.org/show_bug.cgi?id=185556
Summary [GTK] Silence GCC 8 warnings
Michael Catanzaro
Reported 2018-05-11 11:42:27 PDT
Silence GCC 8 warnings. Having already done this for WPE, there's not much more left to do. Notably, Adrian, you were wrong about the C style casts. The double cast is needed there too.
Attachments
Patch (9.02 KB, patch)
2018-05-11 11:48 PDT, Michael Catanzaro 		zan: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Catanzaro
Comment 1 2018-05-11 11:48:03 PDT
Created attachment 340209 [details] Patch
Adrian Perez
Comment 2 2018-05-11 14:47:30 PDT
(In reply to Michael Catanzaro from comment #0) > Silence GCC 8 warnings. Having already done this for WPE, there's not much > more left to do. > > Notably, Adrian, you were wrong about the C style casts. The double cast is > needed there too. Well, one has to be proven wrong now and than, and that's fine. Double-cast it is, then! FWIW, I quickly tested the patch and works here, so it seems fine to me.
Michael Catanzaro
Comment 3 2018-05-21 08:29:28 PDT
Ping reviewers
Michael Catanzaro
Comment 4 2018-05-23 10:50:51 PDT
Committed r232118: <https://trac.webkit.org/changeset/232118>
Note You need to log in before you can comment on or make changes to this bug.