Merge lp://qastaging/~cimi/notify-osd/no-deprecated-functions into lp://qastaging/~canonical-dx-team/notify-osd/precise

Proposed by Andrea Cimitan
Status: Needs review
Proposed branch: lp://qastaging/~cimi/notify-osd/no-deprecated-functions
Merge into: lp://qastaging/~canonical-dx-team/notify-osd/precise
Diff against target: 136 lines (+36/-16)
5 files modified
egg/egg-timeout-pool.c (+7/-8)
src/bubble.c (+8/-1)
src/defaults.c (+5/-5)
tests/test-grow-bubble.c (+8/-1)
tests/test-scroll-text.c (+8/-1)
To merge this branch: bzr merge lp://qastaging/~cimi/notify-osd/no-deprecated-functions
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
jenkins (community) continuous-integration Approve
Mirco Müller (community) Needs Fixing
Review via email: mp+100412@code.qastaging.launchpad.net

Description of the change

Use the proper not-deprecated functions.

Please make sure egg_timeout_pool_get_ticks is returning the difference with the required scale, since I didn't pay much attention in the right conversion.

To post a comment you must log in.
Revision history for this message
Mirco Müller (macslow) wrote :

Your branch is breaking the timing calculations. Very obvious e.g. in the much too short throbbing animation of the volume over- and undershoot animations.

review: Needs Fixing
Revision history for this message
Andrea Cimitan (cimi) wrote :

Exactly as I said in the description: egg_timeout_pool_get_ticks needs the right scale, could you please help me on that? I will multiply the value returned by the function.

Revision history for this message
Mirco Müller (macslow) wrote :

Dash corners first :) So either this evening or tomorrow.

Revision history for this message
Andrea Cimitan (cimi) wrote :

Indeed, while the color branch has to come in 12.04, this could be added to trunk for 12.10 branch

Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:457
http://s-jenkins:8080/job/notify-osd-ci/1/

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Unmerged revisions

457. By Andrea Cimitan

Don't use deprecated functions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches