1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
|
From e4478df9389894b6498ffe252310cdecd1e58904 Mon Sep 17 00:00:00 2001
From: Vicente Bergas <vicencb@gmail.com>
Date: Sat, 3 Feb 2018 19:02:38 +0100
Subject: [PATCH] Fix signed vs. unsigned math on ARM
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
On ARM systems, `char` is not signed, but unsigned, and therefore code
like qMin(-11, ...) doesn't really do a correct thing.
Change-Id: I1e03570c8a809cd6d8c699b3bc138a6bd05ce53c
Signed-off-by: Jan Kundrát <jkt@kde.org>
---
src/Gui/Spinner.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/Gui/Spinner.h b/src/Gui/Spinner.h
index b8b06b85..2f60ce45 100644
--- a/src/Gui/Spinner.h
+++ b/src/Gui/Spinner.h
@@ -82,8 +82,8 @@ private slots:
void updateAncestors();
void updateGeometry();
private:
- uchar m_step;
- char m_fadeStep;
+ int m_step;
+ int m_fadeStep;
int m_timer;
QTimer *m_startTimer;
QList<QWidget*> m_ancestors;
--
GitLab
From 2202821fad977923d2f69c044ad09b3dc46c804a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Kundr=C3=A1t?= <jkt@kde.org>
Date: Fri, 24 Aug 2018 13:44:34 +0200
Subject: [PATCH] Fix comparison of integer expressions of different signedness
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
src/Imap/Model/ModelTest/modeltest.cpp: In member function ‘void ModelTest::data()’:
src/Imap/Model/ModelTest/modeltest.cpp:422:27: error: comparison of integer expressions of different signedness: ‘int’ and ‘QFlags<Qt::AlignmentFlag>::Int’ {aka ‘unsigned int’} [-Werror=sign-compare]
Q_ASSERT(alignment == (alignment & (Qt::AlignHorizontal_Mask | Qt::AlignVertical_Mask)));
~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Change-Id: I7007e1c0cba20b31d76f6426548ad4fb43480017
---
src/Imap/Model/ModelTest/modeltest.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/Imap/Model/ModelTest/modeltest.cpp b/src/Imap/Model/ModelTest/modeltest.cpp
index 24046a24..d2c8fcac 100644
--- a/src/Imap/Model/ModelTest/modeltest.cpp
+++ b/src/Imap/Model/ModelTest/modeltest.cpp
@@ -418,8 +418,8 @@ void ModelTest::data()
// Check that the alignment is one we know about
QVariant textAlignmentVariant = model->data(model->index(0, 0), Qt::TextAlignmentRole);
if (textAlignmentVariant.isValid()) {
- int alignment = textAlignmentVariant.toInt();
- Q_ASSERT(alignment == (alignment & (Qt::AlignHorizontal_Mask | Qt::AlignVertical_Mask)));
+ unsigned int alignment = textAlignmentVariant.toUInt();
+ Q_ASSERT(alignment == (alignment & (Qt::AlignHorizontal_Mask | Qt::AlignVertical_Mask)));
}
// General Purpose roles that should return a QColor
--
GitLab
|