Merge lp:~osomon/webbrowser-app/fix-gcc5-ftbfs into lp:webbrowser-app

Proposed by Olivier Tilloy
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 1103
Merged at revision: 1104
Proposed branch: lp:~osomon/webbrowser-app/fix-gcc5-ftbfs
Merge into: lp:webbrowser-app
Diff against target: 21 lines (+2/-2)
1 file modified
tests/unittests/favicon-fetcher/tst_FaviconFetcherTests.cpp (+2/-2)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/fix-gcc5-ftbfs
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
PS Jenkins bot continuous-integration Approve
Alexandre Abreu (community) Approve
Review via email: mp+265991@code.launchpad.net

Commit message

Fix narrowing conversion that makes the build fail with GCC 5.

To post a comment you must log in.
Revision history for this message
Alexandre Abreu (abreu-alexandre) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Looks good to me too.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/unittests/favicon-fetcher/tst_FaviconFetcherTests.cpp'
2--- tests/unittests/favicon-fetcher/tst_FaviconFetcherTests.cpp 2015-05-26 21:42:54 +0000
3+++ tests/unittests/favicon-fetcher/tst_FaviconFetcherTests.cpp 2015-07-27 15:35:08 +0000
4@@ -34,7 +34,7 @@
5 // local
6 #include "favicon-fetcher.h"
7
8-const char icon_data[] = {
9+const unsigned char icon_data[] = {
10 0x00, 0x00, 0x01, 0x00, 0x01, 0x00, 0x01, 0x01, 0x02, 0x00, 0x01, 0x00,
11 0x01, 0x00, 0x38, 0x00, 0x00, 0x00, 0x16, 0x00, 0x00, 0x00, 0x28, 0x00,
12 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, 0x02, 0x00, 0x00, 0x00, 0x01, 0x00,
13@@ -98,7 +98,7 @@
14 response << "HTTP/1.0 200 OK\r\n"
15 << "Content-Length: " << icon_data_size << "\r\n"
16 << "Content-Type: image/x-icon\r\n\r\n"
17- << QString::fromLocal8Bit(icon_data, icon_data_size) << "\n";
18+ << QString::fromLocal8Bit((const char*) icon_data, icon_data_size) << "\n";
19 } else if (redirection.exactMatch(path)) {
20 int n = redirection.cap(1).toInt();
21 response << "HTTP/1.0 303 See Other\r\n"

Subscribers

People subscribed via source and target branches

to status/vote changes: