seeseemelk/cool-retro-term
1
0
Fork 0
Code Issues 449 Pull Requests Actions Packages Projects Releases 8 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)
Copy Link
Copy Source

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)
Copy Link
Copy Source

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)
Copy Link
Copy Source

Very well. Thank you for your patch!

Very well. Thank you for your patch!
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bounty
bug
duplicate
enhancement
help wanted
invalid
priority
high
priority
low
question
wontfix
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
seeseemelk (Sebastiaan de Schaetzen)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: seeseemelk/cool-retro-term#101