Nux

Merge lp:~3v1n0/nux/check-dnd_area-trusty into lp:nux/trusty

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 846
Merged at revision: 849
Proposed branch: lp:~3v1n0/nux/check-dnd_area-trusty
Merge into: lp:nux/trusty
Diff against target: 14 lines (+2/-3)
1 file modified
Nux/WindowCompositor.cpp (+2/-3)
To merge this branch: bzr merge lp:~3v1n0/nux/check-dnd_area-trusty
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Review via email: mp+237153@code.launchpad.net

Commit message

WindowCompositor: don't try to use an null dnd_area pointer

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

yay

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Nux/WindowCompositor.cpp'
--- Nux/WindowCompositor.cpp 2013-10-30 23:22:34 +0000
+++ Nux/WindowCompositor.cpp 2014-10-03 23:10:53 +0000
@@ -283,9 +283,8 @@
283 }283 }
284 else if (event.type == NUX_DND_DROP)284 else if (event.type == NUX_DND_DROP)
285 {285 {
286 InputArea* current_dnd_area = GetDnDArea();286 if (_dnd_area && _dnd_area->GetAbsoluteGeometry().IsPointInside(event.x - event.x_root, event.y - event.y_root))
287 if (current_dnd_area->GetAbsoluteGeometry().IsPointInside(event.x - event.x_root, event.y - event.y_root))287 _dnd_area->HandleDndDrop(event);
288 current_dnd_area->HandleDndDrop(event);
289 }288 }
290 }289 }
291290

Subscribers

People subscribed via source and target branches