Merge lp:~michihenning/thumbnailer/fix-warnings into lp:thumbnailer/devel
- fix-warnings
- Merge into devel
Status: | Merged |
---|---|
Approved by: | Michi Henning |
Approved revision: | 366 |
Merged at revision: | 367 |
Proposed branch: | lp:~michihenning/thumbnailer/fix-warnings |
Merge into: | lp:thumbnailer/devel |
Diff against target: |
567 lines (+107/-55) 25 files modified
doc/libthumbnailer-qt/examples/qt_example_test.cpp (+4/-0) include/internal/image.h (+1/-1) src/image.cpp (+1/-1) tests/art_extractor/art_extractor_test.cpp (+1/-1) tests/backoff_adjuster/backoff_adjuster_test.cpp (+1/-1) tests/check_access/check_access_test.cpp (+1/-1) tests/dbus/dbus_test.cpp (+5/-5) tests/download/download_test.cpp (+1/-1) tests/file_io/file_io_test.cpp (+2/-2) tests/file_lock/file_lock_test.cpp (+1/-1) tests/gmock_nowarn.h (+24/-0) tests/gobj_ptr/gobj_ptr_test.cpp (+9/-9) tests/gtest_nowarn.h (+24/-0) tests/image-provider/image-provider_test.cpp (+1/-1) tests/image/image_test.cpp (+12/-12) tests/libthumbnailer-qt/libthumbnailer-qt_test.cpp (+1/-1) tests/recovery/MockCache.h (+1/-1) tests/safe_strerror/safe_strerror_test.cpp (+1/-1) tests/settings/settings_test.cpp (+1/-1) tests/slow-vs-thumb/slow-vs-thumb_test.cpp (+1/-1) tests/stress/stress_test.cpp (+1/-1) tests/thumbnailer-admin/thumbnailer-admin_test.cpp (+10/-10) tests/thumbnailer/thumbnailer_test.cpp (+1/-1) tests/version/version_test.cpp (+1/-1) tests/vs-thumb/vs-thumb_test.cpp (+1/-1) |
To merge this branch: | bzr merge lp:~michihenning/thumbnailer/fix-warnings |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
unity-api-1-bot | continuous-integration | Approve | |
James Henstridge | Approve | ||
Review via email: mp+314889@code.launchpad.net |
Commit message
Fixed a bunch of compiler warnings on zesty.
Description of the change
Fixed a bunch of compiler warnings on zesty.
unity-api-1-bot (unity-api-1-bot) wrote : | # |
unity-api-1-bot (unity-api-1-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
unity-api-1-bot (unity-api-1-bot) : | # |
Preview Diff
1 | === modified file 'doc/libthumbnailer-qt/examples/qt_example_test.cpp' |
2 | --- doc/libthumbnailer-qt/examples/qt_example_test.cpp 2016-09-23 01:39:55 +0000 |
3 | +++ doc/libthumbnailer-qt/examples/qt_example_test.cpp 2017-01-17 04:15:24 +0000 |
4 | @@ -115,7 +115,11 @@ |
5 | #include <QTemporaryDir> |
6 | |
7 | #include <boost/filesystem.hpp> |
8 | + |
9 | +#pragma GCC diagnostic push |
10 | +#pragma GCC diagnostic ignored "-Wctor-dtor-privacy" |
11 | #include <gtest/gtest.h> |
12 | +#pragma GCC diagnostic pop |
13 | |
14 | #include <memory> |
15 | |
16 | |
17 | === modified file 'include/internal/image.h' |
18 | --- include/internal/image.h 2016-03-02 04:10:45 +0000 |
19 | +++ include/internal/image.h 2017-01-17 04:15:24 +0000 |
20 | @@ -61,7 +61,7 @@ |
21 | |
22 | // Return the pixel value at the (x,y) coordinates as an integer: |
23 | // r << 24 | g << 16 | b << 8 | a |
24 | - int pixel(int x, int y) const; |
25 | + uint32_t pixel(int x, int y) const; |
26 | |
27 | // Return a scaled version of the image that fits within the given |
28 | // requested size. |
29 | |
30 | === modified file 'src/image.cpp' |
31 | --- src/image.cpp 2016-02-29 07:04:54 +0000 |
32 | +++ src/image.cpp 2017-01-17 04:15:24 +0000 |
33 | @@ -430,7 +430,7 @@ |
34 | return h; |
35 | } |
36 | |
37 | -int Image::pixel(int x, int y) const |
38 | +uint32_t Image::pixel(int x, int y) const |
39 | { |
40 | assert(gdk_pixbuf_get_colorspace(pixbuf_.get()) == GDK_COLORSPACE_RGB); |
41 | assert(gdk_pixbuf_get_bits_per_sample(pixbuf_.get()) == 8); |
42 | |
43 | === modified file 'tests/art_extractor/art_extractor_test.cpp' |
44 | --- tests/art_extractor/art_extractor_test.cpp 2016-02-09 04:09:37 +0000 |
45 | +++ tests/art_extractor/art_extractor_test.cpp 2017-01-17 04:15:24 +0000 |
46 | @@ -23,7 +23,7 @@ |
47 | #include <boost/algorithm/string.hpp> |
48 | |
49 | #include <testsetup.h> |
50 | -#include <gtest/gtest.h> |
51 | +#include <gtest_nowarn.h> |
52 | |
53 | #define AIFF_FILE TESTDATADIR "/testsong.aiff" |
54 | #define FLAC_FILE TESTDATADIR "/testsong.flac" |
55 | |
56 | === modified file 'tests/backoff_adjuster/backoff_adjuster_test.cpp' |
57 | --- tests/backoff_adjuster/backoff_adjuster_test.cpp 2016-01-04 06:09:41 +0000 |
58 | +++ tests/backoff_adjuster/backoff_adjuster_test.cpp 2017-01-17 04:15:24 +0000 |
59 | @@ -18,7 +18,7 @@ |
60 | |
61 | #include <internal/backoff_adjuster.h> |
62 | |
63 | -#include <gtest/gtest.h> |
64 | +#include <gtest_nowarn.h> |
65 | |
66 | #include <thread> |
67 | |
68 | |
69 | === modified file 'tests/check_access/check_access_test.cpp' |
70 | --- tests/check_access/check_access_test.cpp 2015-06-24 01:08:06 +0000 |
71 | +++ tests/check_access/check_access_test.cpp 2017-01-17 04:15:24 +0000 |
72 | @@ -18,7 +18,7 @@ |
73 | |
74 | #include <internal/check_access.h> |
75 | |
76 | -#include <gtest/gtest.h> |
77 | +#include <gtest_nowarn.h> |
78 | #include <testsetup.h> |
79 | |
80 | #include <cstdio> |
81 | |
82 | === modified file 'tests/dbus/dbus_test.cpp' |
83 | --- tests/dbus/dbus_test.cpp 2016-09-26 00:30:42 +0000 |
84 | +++ tests/dbus/dbus_test.cpp 2017-01-17 04:15:24 +0000 |
85 | @@ -25,7 +25,7 @@ |
86 | #include "utils/env_var_guard.h" |
87 | |
88 | #include <boost/algorithm/string/predicate.hpp> |
89 | -#include <gtest/gtest.h> |
90 | +#include <gtest_nowarn.h> |
91 | #include <QProcess> |
92 | #include <QSignalSpy> |
93 | #include <QTemporaryDir> |
94 | @@ -423,7 +423,7 @@ |
95 | { |
96 | CacheStats s = reply.value().full_size_stats; |
97 | EXPECT_EQ(temp_dir() + "/cache/unity-thumbnailer/images", s.cache_path.toStdString()); |
98 | - EXPECT_EQ(1, s.policy); |
99 | + EXPECT_EQ(1u, s.policy); |
100 | EXPECT_EQ(0, s.size); |
101 | EXPECT_EQ(0, s.size_in_bytes); |
102 | EXPECT_NE(0, s.max_size_in_bytes); |
103 | @@ -444,21 +444,21 @@ |
104 | auto list = s.histogram; |
105 | for (auto c : list) |
106 | { |
107 | - EXPECT_EQ(0, c); |
108 | + EXPECT_EQ(0u, c); |
109 | } |
110 | } |
111 | |
112 | { |
113 | CacheStats s = reply.value().thumbnail_stats; |
114 | EXPECT_EQ(temp_dir() + "/cache/unity-thumbnailer/thumbnails", s.cache_path.toStdString()); |
115 | - EXPECT_EQ(1, s.policy); |
116 | + EXPECT_EQ(1u, s.policy); |
117 | EXPECT_EQ(0, s.size); |
118 | } |
119 | |
120 | { |
121 | CacheStats s = reply.value().failure_stats; |
122 | EXPECT_EQ(temp_dir() + "/cache/unity-thumbnailer/failures", s.cache_path.toStdString()); |
123 | - EXPECT_EQ(0, s.policy); |
124 | + EXPECT_EQ(0u, s.policy); |
125 | EXPECT_EQ(0, s.size); |
126 | } |
127 | |
128 | |
129 | === modified file 'tests/download/download_test.cpp' |
130 | --- tests/download/download_test.cpp 2015-12-28 05:05:23 +0000 |
131 | +++ tests/download/download_test.cpp 2017-01-17 04:15:24 +0000 |
132 | @@ -22,7 +22,7 @@ |
133 | #include "utils/artserver.h" |
134 | #include <testsetup.h> |
135 | |
136 | -#include <gtest/gtest.h> |
137 | +#include <gtest_nowarn.h> |
138 | |
139 | #include <QSignalSpy> |
140 | #include <QUrl> |
141 | |
142 | === modified file 'tests/file_io/file_io_test.cpp' |
143 | --- tests/file_io/file_io_test.cpp 2015-11-30 06:55:46 +0000 |
144 | +++ tests/file_io/file_io_test.cpp 2017-01-17 04:15:24 +0000 |
145 | @@ -19,7 +19,7 @@ |
146 | #include <internal/file_io.h> |
147 | |
148 | #include <boost/algorithm/string.hpp> |
149 | -#include <gtest/gtest.h> |
150 | +#include <gtest_nowarn.h> |
151 | #include <testsetup.h> |
152 | |
153 | #include <fcntl.h> |
154 | @@ -34,7 +34,7 @@ |
155 | ASSERT_NE(-1, stat(in_file.c_str(), &st)); |
156 | |
157 | string data = read_file(in_file); |
158 | - ASSERT_EQ(st.st_size, data.size()); |
159 | + ASSERT_EQ(unsigned(st.st_size), data.size()); |
160 | |
161 | string out_file = TESTBINDIR "/testimage.jpg"; |
162 | write_file(out_file, data); |
163 | |
164 | === modified file 'tests/file_lock/file_lock_test.cpp' |
165 | --- tests/file_lock/file_lock_test.cpp 2016-01-09 04:25:19 +0000 |
166 | +++ tests/file_lock/file_lock_test.cpp 2017-01-17 04:15:24 +0000 |
167 | @@ -20,7 +20,7 @@ |
168 | |
169 | #include <unity/UnityExceptions.h> |
170 | |
171 | -#include <gtest/gtest.h> |
172 | +#include <gtest_nowarn.h> |
173 | |
174 | using namespace std; |
175 | using namespace unity::thumbnailer::internal; |
176 | |
177 | === added file 'tests/gmock_nowarn.h' |
178 | --- tests/gmock_nowarn.h 1970-01-01 00:00:00 +0000 |
179 | +++ tests/gmock_nowarn.h 2017-01-17 04:15:24 +0000 |
180 | @@ -0,0 +1,24 @@ |
181 | +/* |
182 | + * Copyright (C) 2017 Canonical Ltd |
183 | + * |
184 | + * This program is free software: you can redistribute it and/or modify |
185 | + * it under the terms of the GNU Lesser General Public License version 3 as |
186 | + * published by the Free Software Foundation. |
187 | + * |
188 | + * This program is distributed in the hope that it will be useful, |
189 | + * but WITHOUT ANY WARRANTY; without even the implied warranty of |
190 | + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
191 | + * GNU Lesser General Public License for more details. |
192 | + * |
193 | + * You should have received a copy of the GNU Lesser General Public License |
194 | + * along with this program. If not, see <http://www.gnu.org/licenses/>. |
195 | + * |
196 | + * Authors: Michi Henning <michi.henning@canonical.com> |
197 | + */ |
198 | + |
199 | +#pragma once |
200 | + |
201 | +#pragma GCC diagnostic push |
202 | +#pragma GCC diagnostic ignored "-Wctor-dtor-privacy" |
203 | +#include <gmock/gmock.h> |
204 | +#pragma GCC diagnostic pop |
205 | |
206 | === modified file 'tests/gobj_ptr/gobj_ptr_test.cpp' |
207 | --- tests/gobj_ptr/gobj_ptr_test.cpp 2015-11-13 04:36:08 +0000 |
208 | +++ tests/gobj_ptr/gobj_ptr_test.cpp 2017-01-17 04:15:24 +0000 |
209 | @@ -22,7 +22,7 @@ |
210 | |
211 | #include <internal/gobj_memory.h> |
212 | #include <glib-object.h> |
213 | -#include <gtest/gtest.h> |
214 | +#include <gtest_nowarn.h> |
215 | |
216 | using namespace std; |
217 | using namespace unity::thumbnailer::internal; |
218 | @@ -87,17 +87,17 @@ |
219 | TEST(Gobj_ptr, refcount) |
220 | { |
221 | GObject* o = G_OBJECT(g_object_new(G_TYPE_OBJECT, nullptr)); |
222 | - EXPECT_EQ(1, o->ref_count); |
223 | + EXPECT_EQ(1u, o->ref_count); |
224 | g_object_ref(o); |
225 | |
226 | { |
227 | - EXPECT_EQ(2, o->ref_count); |
228 | + EXPECT_EQ(2u, o->ref_count); |
229 | gobj_ptr<GObject> u(o); |
230 | - EXPECT_EQ(2, o->ref_count); |
231 | + EXPECT_EQ(2u, o->ref_count); |
232 | // Now it dies and refcount is reduced. |
233 | } |
234 | |
235 | - EXPECT_EQ(1, o->ref_count); |
236 | + EXPECT_EQ(1u, o->ref_count); |
237 | g_object_unref(o); |
238 | } |
239 | |
240 | @@ -105,14 +105,14 @@ |
241 | { |
242 | GObject* o = G_OBJECT(g_object_new(G_TYPE_OBJECT, nullptr)); |
243 | gobj_ptr<GObject> u(o); |
244 | - EXPECT_EQ(1, u->ref_count); |
245 | + EXPECT_EQ(1u, u->ref_count); |
246 | gobj_ptr<GObject> u2(u); |
247 | - EXPECT_EQ(2, u->ref_count); |
248 | + EXPECT_EQ(2u, u->ref_count); |
249 | gobj_ptr<GObject> u3 = u2; |
250 | - EXPECT_EQ(3, u->ref_count); |
251 | + EXPECT_EQ(3u, u->ref_count); |
252 | u3.reset(); |
253 | u2.reset(); |
254 | - EXPECT_EQ(1, u->ref_count); |
255 | + EXPECT_EQ(1u, u->ref_count); |
256 | } |
257 | |
258 | TEST(Gobj_ptr, swap) |
259 | |
260 | === added file 'tests/gtest_nowarn.h' |
261 | --- tests/gtest_nowarn.h 1970-01-01 00:00:00 +0000 |
262 | +++ tests/gtest_nowarn.h 2017-01-17 04:15:24 +0000 |
263 | @@ -0,0 +1,24 @@ |
264 | +/* |
265 | + * Copyright (C) 2017 Canonical Ltd |
266 | + * |
267 | + * This program is free software: you can redistribute it and/or modify |
268 | + * it under the terms of the GNU Lesser General Public License version 3 as |
269 | + * published by the Free Software Foundation. |
270 | + * |
271 | + * This program is distributed in the hope that it will be useful, |
272 | + * but WITHOUT ANY WARRANTY; without even the implied warranty of |
273 | + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
274 | + * GNU Lesser General Public License for more details. |
275 | + * |
276 | + * You should have received a copy of the GNU Lesser General Public License |
277 | + * along with this program. If not, see <http://www.gnu.org/licenses/>. |
278 | + * |
279 | + * Authors: Michi Henning <michi.henning@canonical.com> |
280 | + */ |
281 | + |
282 | +#pragma once |
283 | + |
284 | +#pragma GCC diagnostic push |
285 | +#pragma GCC diagnostic ignored "-Wctor-dtor-privacy" |
286 | +#include <gtest/gtest.h> |
287 | +#pragma GCC diagnostic pop |
288 | |
289 | === modified file 'tests/image-provider/image-provider_test.cpp' |
290 | --- tests/image-provider/image-provider_test.cpp 2016-09-26 00:30:42 +0000 |
291 | +++ tests/image-provider/image-provider_test.cpp 2017-01-17 04:15:24 +0000 |
292 | @@ -27,7 +27,7 @@ |
293 | #include "utils/dbusserver.h" |
294 | #include "utils/testutils.h" |
295 | |
296 | -#include <gtest/gtest.h> |
297 | +#include <gtest_nowarn.h> |
298 | #include <QGuiApplication> |
299 | #include <QImage> |
300 | #include <QQuickImageProvider> |
301 | |
302 | === modified file 'tests/image/image_test.cpp' |
303 | --- tests/image/image_test.cpp 2016-02-29 07:04:54 +0000 |
304 | +++ tests/image/image_test.cpp 2017-01-17 04:15:24 +0000 |
305 | @@ -19,7 +19,7 @@ |
306 | #include <internal/image.h> |
307 | |
308 | #include <boost/algorithm/string.hpp> |
309 | -#include <gtest/gtest.h> |
310 | +#include <gtest_nowarn.h> |
311 | #include <sys/types.h> |
312 | #include <fcntl.h> |
313 | |
314 | @@ -53,8 +53,8 @@ |
315 | EXPECT_EQ(480, i.height()); |
316 | EXPECT_EQ(0xFE0000FF, i.pixel(0, 0)); |
317 | EXPECT_EQ(0xFFFF00FF, i.pixel(639, 0)); |
318 | - EXPECT_EQ(0x00FF01FF, i.pixel(639, 479)); |
319 | - EXPECT_EQ(0x0000FEFF, i.pixel(0, 479)); |
320 | + EXPECT_EQ(0x00FF01FFu, i.pixel(639, 479)); |
321 | + EXPECT_EQ(0x0000FEFFu, i.pixel(0, 479)); |
322 | EXPECT_FALSE(i.has_alpha()); |
323 | |
324 | // Move constructor |
325 | @@ -251,8 +251,8 @@ |
326 | EXPECT_EQ(150, img.height()); |
327 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
328 | EXPECT_EQ(0xFFFF00FF, img.pixel(199, 0)); |
329 | - EXPECT_EQ(0x00FF01FF, img.pixel(199, 149)); |
330 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 149)); |
331 | + EXPECT_EQ(0x00FF01FFu, img.pixel(199, 149)); |
332 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 149)); |
333 | } |
334 | } |
335 | |
336 | @@ -267,8 +267,8 @@ |
337 | EXPECT_EQ(480, img.height()); |
338 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
339 | EXPECT_EQ(0xFFFF00FF, img.pixel(639, 0)); |
340 | - EXPECT_EQ(0x00FF01FF, img.pixel(639, 479)); |
341 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 479)); |
342 | + EXPECT_EQ(0x00FF01FFu, img.pixel(639, 479)); |
343 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 479)); |
344 | |
345 | // Scaled version |
346 | img = Image(data, QSize(320, 240)); |
347 | @@ -276,8 +276,8 @@ |
348 | EXPECT_EQ(240, img.height()); |
349 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
350 | EXPECT_EQ(0xFFFF00FF, img.pixel(319, 0)); |
351 | - EXPECT_EQ(0x00FF01FF, img.pixel(319, 239)); |
352 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 239)); |
353 | + EXPECT_EQ(0x00FF01FFu, img.pixel(319, 239)); |
354 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 239)); |
355 | |
356 | // This version will be produced from the thumbnail, which has |
357 | // been tinted to distinguish it from the original. |
358 | @@ -448,7 +448,7 @@ |
359 | Image img(fd.get(), QSize(400, 400)); |
360 | EXPECT_EQ(200, img.width()); |
361 | EXPECT_EQ(200, img.height()); |
362 | - EXPECT_EQ(0x0, img.pixel(0, 0)); |
363 | + EXPECT_EQ(0x0u, img.pixel(0, 0)); |
364 | EXPECT_EQ(0xFF0000FF, img.pixel(100, 100)); |
365 | EXPECT_TRUE(img.has_alpha()); |
366 | } |
367 | @@ -461,7 +461,7 @@ |
368 | Image img(fd.get(), QSize()); |
369 | EXPECT_EQ(200, img.width()); |
370 | EXPECT_EQ(200, img.height()); |
371 | - EXPECT_EQ(0x0, img.pixel(0, 0)); |
372 | + EXPECT_EQ(0x0u, img.pixel(0, 0)); |
373 | EXPECT_EQ(0xFF0000FF, img.pixel(100, 100)); |
374 | EXPECT_TRUE(img.has_alpha()); |
375 | } |
376 | @@ -474,7 +474,7 @@ |
377 | Image img(fd.get(), QSize(400, 400)); |
378 | EXPECT_EQ(200, img.width()); |
379 | EXPECT_EQ(200, img.height()); |
380 | - EXPECT_EQ(0x0, img.pixel(0, 0)); |
381 | + EXPECT_EQ(0x0u, img.pixel(0, 0)); |
382 | EXPECT_EQ(0xFF0000FF, img.pixel(100, 100)); |
383 | EXPECT_TRUE(img.has_alpha()); |
384 | } |
385 | |
386 | === modified file 'tests/libthumbnailer-qt/libthumbnailer-qt_test.cpp' |
387 | --- tests/libthumbnailer-qt/libthumbnailer-qt_test.cpp 2016-09-26 00:30:42 +0000 |
388 | +++ tests/libthumbnailer-qt/libthumbnailer-qt_test.cpp 2017-01-17 04:15:24 +0000 |
389 | @@ -43,7 +43,7 @@ |
390 | |
391 | #include <boost/algorithm/string/predicate.hpp> |
392 | #include <boost/filesystem.hpp> |
393 | -#include <gtest/gtest.h> |
394 | +#include <gtest_nowarn.h> |
395 | |
396 | #include <QSignalSpy> |
397 | #include <QTemporaryDir> |
398 | |
399 | === modified file 'tests/recovery/MockCache.h' |
400 | --- tests/recovery/MockCache.h 2015-09-01 08:05:02 +0000 |
401 | +++ tests/recovery/MockCache.h 2017-01-17 04:15:24 +0000 |
402 | @@ -22,7 +22,7 @@ |
403 | #include <core/optional.h> |
404 | #include <core/persistent_cache_stats.h> |
405 | |
406 | -#include <gmock/gmock.h> |
407 | +#include <gmock_nowarn.h> |
408 | |
409 | #include <chrono> |
410 | #include <system_error> |
411 | |
412 | === modified file 'tests/safe_strerror/safe_strerror_test.cpp' |
413 | --- tests/safe_strerror/safe_strerror_test.cpp 2015-11-13 04:36:08 +0000 |
414 | +++ tests/safe_strerror/safe_strerror_test.cpp 2017-01-17 04:15:24 +0000 |
415 | @@ -17,7 +17,7 @@ |
416 | */ |
417 | |
418 | #include <internal/safe_strerror.h> |
419 | -#include <gtest/gtest.h> |
420 | +#include <gtest_nowarn.h> |
421 | |
422 | using namespace unity::thumbnailer::internal; |
423 | |
424 | |
425 | === modified file 'tests/settings/settings_test.cpp' |
426 | --- tests/settings/settings_test.cpp 2016-01-11 07:33:57 +0000 |
427 | +++ tests/settings/settings_test.cpp 2017-01-17 04:15:24 +0000 |
428 | @@ -27,7 +27,7 @@ |
429 | #pragma GCC diagnostic ignored "-Wcast-qual" |
430 | #include <gio/gio.h> |
431 | #pragma GCC diagnostic pop |
432 | -#include <gtest/gtest.h> |
433 | +#include <gtest_nowarn.h> |
434 | #include <QTemporaryDir> |
435 | |
436 | #include <unistd.h> |
437 | |
438 | === modified file 'tests/slow-vs-thumb/slow-vs-thumb_test.cpp' |
439 | --- tests/slow-vs-thumb/slow-vs-thumb_test.cpp 2016-09-26 00:50:50 +0000 |
440 | +++ tests/slow-vs-thumb/slow-vs-thumb_test.cpp 2017-01-17 04:15:24 +0000 |
441 | @@ -22,7 +22,7 @@ |
442 | #include <testsetup.h> |
443 | |
444 | #include <boost/filesystem.hpp> |
445 | -#include <gtest/gtest.h> |
446 | +#include <gtest_nowarn.h> |
447 | #include <QSignalSpy> |
448 | #include <QTemporaryDir> |
449 | #include <unity/UnityExceptions.h> |
450 | |
451 | === modified file 'tests/stress/stress_test.cpp' |
452 | --- tests/stress/stress_test.cpp 2016-09-26 00:30:42 +0000 |
453 | +++ tests/stress/stress_test.cpp 2017-01-17 04:15:24 +0000 |
454 | @@ -40,7 +40,7 @@ |
455 | #pragma GCC diagnostic pop |
456 | |
457 | #include <boost/filesystem.hpp> |
458 | -#include <gtest/gtest.h> |
459 | +#include <gtest_nowarn.h> |
460 | #include <QSignalSpy> |
461 | #include <QTemporaryDir> |
462 | |
463 | |
464 | === modified file 'tests/thumbnailer-admin/thumbnailer-admin_test.cpp' |
465 | --- tests/thumbnailer-admin/thumbnailer-admin_test.cpp 2016-02-29 09:07:16 +0000 |
466 | +++ tests/thumbnailer-admin/thumbnailer-admin_test.cpp 2017-01-17 04:15:24 +0000 |
467 | @@ -25,7 +25,7 @@ |
468 | #include <testsetup.h> |
469 | |
470 | #include <boost/algorithm/string/predicate.hpp> |
471 | -#include <gtest/gtest.h> |
472 | +#include <gtest_nowarn.h> |
473 | |
474 | using namespace std; |
475 | using namespace boost; |
476 | @@ -434,8 +434,8 @@ |
477 | EXPECT_EQ(480, img.height()); |
478 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
479 | EXPECT_EQ(0xFFFF00FF, img.pixel(639, 0)); |
480 | - EXPECT_EQ(0x00FF01FF, img.pixel(639, 479)); |
481 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 479)); |
482 | + EXPECT_EQ(0x00FF01FFu, img.pixel(639, 479)); |
483 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 479)); |
484 | } |
485 | |
486 | TEST_F(AdminTest, get_large_thumbnail) |
487 | @@ -452,8 +452,8 @@ |
488 | EXPECT_EQ(240, img.height()); |
489 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
490 | EXPECT_EQ(0xFFFF00FF, img.pixel(319, 0)); |
491 | - EXPECT_EQ(0x00FF01FF, img.pixel(319, 239)); |
492 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 239)); |
493 | + EXPECT_EQ(0x00FF01FFu, img.pixel(319, 239)); |
494 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 239)); |
495 | } |
496 | |
497 | TEST_F(AdminTest, get_small_thumbnail_square) |
498 | @@ -539,7 +539,7 @@ |
499 | Image img(data); |
500 | EXPECT_EQ(200, img.width()); |
501 | EXPECT_EQ(200, img.height()); |
502 | - EXPECT_EQ(0, img.pixel(0, 0)); |
503 | + EXPECT_EQ(0u, img.pixel(0, 0)); |
504 | } |
505 | |
506 | TEST_F(AdminTest, get_png_no_alpha) |
507 | @@ -572,8 +572,8 @@ |
508 | EXPECT_EQ(480, img.height()); |
509 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
510 | EXPECT_EQ(0xFFFF00FF, img.pixel(639, 0)); |
511 | - EXPECT_EQ(0x00FF01FF, img.pixel(639, 479)); |
512 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 479)); |
513 | + EXPECT_EQ(0x00FF01FFu, img.pixel(639, 479)); |
514 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 479)); |
515 | } |
516 | |
517 | TEST_F(AdminTest, get_with_relative_input_path) |
518 | @@ -591,8 +591,8 @@ |
519 | EXPECT_EQ(480, img.height()); |
520 | EXPECT_EQ(0xFE0000FF, img.pixel(0, 0)); |
521 | EXPECT_EQ(0xFFFF00FF, img.pixel(639, 0)); |
522 | - EXPECT_EQ(0x00FF01FF, img.pixel(639, 479)); |
523 | - EXPECT_EQ(0x0000FEFF, img.pixel(0, 479)); |
524 | + EXPECT_EQ(0x00FF01FFu, img.pixel(639, 479)); |
525 | + EXPECT_EQ(0x0000FEFFu, img.pixel(0, 479)); |
526 | } |
527 | |
528 | TEST_F(AdminTest, empty_input_path) |
529 | |
530 | === modified file 'tests/thumbnailer/thumbnailer_test.cpp' |
531 | --- tests/thumbnailer/thumbnailer_test.cpp 2016-11-11 06:43:10 +0000 |
532 | +++ tests/thumbnailer/thumbnailer_test.cpp 2017-01-17 04:15:24 +0000 |
533 | @@ -34,7 +34,7 @@ |
534 | #pragma GCC diagnostic ignored "-Wcast-qual" |
535 | #include <gio/gio.h> |
536 | #pragma GCC diagnostic pop |
537 | -#include <gtest/gtest.h> |
538 | +#include <gtest_nowarn.h> |
539 | #include <QCoreApplication> |
540 | #include <QSignalSpy> |
541 | #include <QTemporaryDir> |
542 | |
543 | === modified file 'tests/version/version_test.cpp' |
544 | --- tests/version/version_test.cpp 2016-02-25 06:05:42 +0000 |
545 | +++ tests/version/version_test.cpp 2017-01-17 04:15:24 +0000 |
546 | @@ -19,7 +19,7 @@ |
547 | #include <internal/version.h> |
548 | |
549 | #include <testsetup.h> |
550 | -#include <gtest/gtest.h> |
551 | +#include <gtest_nowarn.h> |
552 | |
553 | using namespace std; |
554 | using namespace unity::thumbnailer::internal; |
555 | |
556 | === modified file 'tests/vs-thumb/vs-thumb_test.cpp' |
557 | --- tests/vs-thumb/vs-thumb_test.cpp 2016-03-21 02:08:39 +0000 |
558 | +++ tests/vs-thumb/vs-thumb_test.cpp 2017-01-17 04:15:24 +0000 |
559 | @@ -40,7 +40,7 @@ |
560 | #pragma clang diagnostic pop |
561 | #endif |
562 | #pragma GCC diagnostic pop |
563 | -#include <gtest/gtest.h> |
564 | +#include <gtest_nowarn.h> |
565 | #include <QUrl> |
566 | |
567 | #include <cstdio> |
PASSED: Continuous integration, rev:366 /jenkins. canonical. com/unity- api-1/job/ lp-thumbnailer- ci/13/ /jenkins. canonical. com/unity- api-1/job/ build/1415 /jenkins. canonical. com/unity- api-1/job/ build-0- fetch/1422 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1200/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= zesty/1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= zesty/1200/ artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1200/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= zesty/1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= zesty/1200/ artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1200/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= zesty/1200 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= zesty/1200/ artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/unity- api-1/job/ lp-thumbnailer- ci/13/rebuild
https:/