Fix ftbfs on Fedora 34 #5

Merged
MicheleC merged 1 commits from ftbfs/fedora34 into master 1 year ago
Francois commented 1 year ago
Collaborator

/tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp: In constructor 'TdeSudo::TdeSudo(TQWidget*, const char*, const TQString&, const TQString&, bool)':
/tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:208:36: error: ordered comparison of pointer with integer zero ('char*' and 'int')
208 | while (fgets(buf, 1024, f) > 0)
| ^
/tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:271:43: warning: comparison of integer expressions of different signedness: 'int' and 'TQValueList::size_type' {aka 'long unsigned int'} [-Wsign-compare]
271 | for (int i = 0; i < commandSplit.count(); i++)
| ^
~
/tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:289:59: warning: comparison of integer expressions of different signedness: 'int' and 'TQValueList::size_type' {aka 'long unsigned int'} [-Wsign-compare]
289 | for (int j = 0; j < argsSplit.count(); j++)
| ^~~~~~~~~~~~~~~~~

/tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp: In constructor 'TdeSudo::TdeSudo(TQWidget*, const char*, const TQString&, const TQString&, bool)': /tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:208:36: error: ordered comparison of pointer with integer zero ('char*' and 'int') 208 | while (fgets(buf, 1024, f) > 0) | ~~~~~~~~~~~~~~~~~~~~^~~ /tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:271:43: warning: comparison of integer expressions of different signedness: 'int' and 'TQValueList<TQString>::size_type' {aka 'long unsigned int'} [-Wsign-compare] 271 | for (int i = 0; i < commandSplit.count(); i++) | ~~^~~~~~~~~~~~~~~~~~~~~~ /tmp/BUILD.fc34.x86_64/trinity-tdesudo-14.0.10/tdesudo/tdesudo.cpp:289:59: warning: comparison of integer expressions of different signedness: 'int' and 'TQValueList<TQString>::size_type' {aka 'long unsigned int'} [-Wsign-compare] 289 | for (int j = 0; j < argsSplit.count(); j++) | ~~^~~~~~~~~~~~~~~~~~~
Francois added 1 commit 1 year ago
1136b8150f Fix ftbfs on Fedora 34
MicheleC merged commit 1136b8150f into master 1 year ago
MicheleC deleted branch ftbfs/fedora34 1 year ago
Owner

Looks good, thanks!

Looks good, thanks!
MicheleC added this to the R14.0.11 release milestone 1 year ago
The pull request has been merged as 1136b8150f.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.