unity-2d

Merge lp://qastaging/~unity-2d-team/unity-2d/remove-unused-code-shell-launcher-animation into lp://qastaging/~unity-2d-team/unity-2d/unity-2d-shell

  1. remove-unused-code-shell-launcher-animation
  2. Merge into unity-2d-shell
Proposed by Lohith D Shivamurthy
Status: Merged
Approved by: Michał Sawicz
Approved revision: 926
Merged at revision: 924
Proposed branch: lp://qastaging/~unity-2d-team/unity-2d/remove-unused-code-shell-launcher-animation
Merge into: lp://qastaging/~unity-2d-team/unity-2d/unity-2d-shell
Diff against target: 497 lines (+0/-391)
9 files modified
libunity-2d-private/src/CMakeLists.txt (+0/-2)
libunity-2d-private/src/edgehitdetector.cpp (+0/-65)
libunity-2d-private/src/edgehitdetector.h (+0/-44)
libunity-2d-private/src/mousearea.cpp (+0/-117)
libunity-2d-private/src/mousearea.h (+0/-60)
libunity-2d-private/src/unity2dpanel.cpp (+0/-43)
libunity-2d-private/src/unity2dpanel.h (+0/-5)
libunity-2d-private/tests/CMakeLists.txt (+0/-7)
libunity-2d-private/tests/mouseareademo.cpp (+0/-48)
To merge this branch: bzr merge lp://qastaging/~unity-2d-team/unity-2d/remove-unused-code-shell-launcher-animation
Reviewer Review Type Date Requested Status
Michał Sawicz Approve
Review via email: mp+89843@code.qastaging.launchpad.net

Description of the change

[shell] Remove unused files mousearea.cpp/h, edgehitdetector.cpp/h & mouseareademo.cpp.
And also clean unity-2d-shell.cpp/h to remove launcher animation code which is no more used.

To post a comment you must log in.
Revision history for this message
Michał Sawicz (saviq) wrote :

* looks like Unity2dPanel::resizeEvent can be removed altogether

review: Needs Fixing
Revision history for this message
Lohith D Shivamurthy (dyams) wrote :

Yes sure

lp://qastaging/~unity-2d-team/unity-2d/remove-unused-code-shell-launcher-animation updated
926. By Lohith D Shivamurthy

[shell] remove unused Unity2dPanel::resizeEvent too

Revision history for this message
Michał Sawicz (saviq) :
review: Approve

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

to all changes: