seeseemelk/cool-retro-term
1
0
Fork 0
Code Issues449 Pull Requests12 Actions Packages Projects Releases8 Wiki Activity

Fix initialization oreder in TerminalDisplay #101

Merged
JIghtuse merged 1 commits from master into master 2014-08-30 00:59:25 +02:00
Conversation 2 Commits 1 Files Changed 1 +4 -4
JIghtuse commented 2014-08-29 20:37:08 +02:00 (Migrated from github.com)

Data members need to be initialized in the order they were declared in
the class definition. This commit fixes compiler warnings.

Data members need to be initialized in the order they were declared in the class definition. This commit fixes compiler warnings.
JIghtuse commented 2014-08-29 20:38:43 +02:00 (Migrated from github.com)

I also tried to build project with -Weffc++ flag and it shows quite a few warnings. I'll try to fix them too, if you don't mind.
UPD. Oh wait, it is parts of KDE/Qt who throws warnings..

I also tried to build project with -Weffc++ flag and it shows quite a few warnings. I'll try to fix them too, if you don't mind. UPD. Oh wait, it is parts of KDE/Qt who throws warnings..
Swordfish90 commented 2014-08-30 00:59:27 +02:00 (Migrated from github.com)

Very well. Thank you for your patch!

Very well. Thank you for your patch!
Sign in to join this conversation.
No Reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: seeseemelk/cool-retro-term#101
No description provided.
Delete Branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?