Skip to content
This repository has been archived by the owner on Jan 2, 2023. It is now read-only.

Fix warnings #5075

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix warnings #5075

wants to merge 2 commits into from

Conversation

bogolt
Copy link

@bogolt bogolt commented Aug 26, 2021

Hi, while trying to build project i found several warnings, some of them C++ related, other Qt.
I tried to fix few of them, all fixes seem very small and should not affect real code.

First one is to avoid possible null pointer dereference, that will never happen in real life, since function always receives variable... well at least compiler is happier this way.
Another is about using const interator instead of non-const.

Few of them concern Qt functions deprecation.
Another one is about making connect() function receive options in normalized form.

Dmitrii Sernii added 2 commits August 26, 2021 23:23
fix possible null pointer dereference
make contrustor variable initialization order
correspond to variable order
use constant begin/end functions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant