Fix warnings in CompTimer
Bug #893998 reported by
Sam Spilsbury
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz Core |
Fix Released
|
Undecided
|
Sam Spilsbury | ||
compiz (Ubuntu) |
Fix Released
|
Undecided
|
Sam Spilsbury |
Bug Description
CompTimer and related classes currently generate warnings
because the GLib timeout is a signed integer and that's being compared to an unsigned
timeDiff integer from the CompTimer class.
Related branches
lp://qastaging/~smspillaz/compiz-core/fix-timer-warnings-893998
- Daniel van Vugt: Approve
- Tim Penhey (community): Approve
-
Diff: 17 lines (+3/-3)1 file modifiedsrc/timer/src/timer.cpp (+3/-3)
Changed in compiz-core: | |
assignee: | nobody → Sam Spilsbury (smspillaz) |
milestone: | none → 0.9.6 |
status: | New → In Progress |
Changed in compiz (Ubuntu): | |
assignee: | nobody → Sam Spilsbury (smspillaz) |
status: | New → In Progress |
Changed in compiz-core: | |
status: | In Progress → Fix Committed |
Changed in compiz-core: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Maybe try adding this gcc option to prevent similar regressions:
-Werror
Make all warnings into errors.