Nux

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

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 855
Merged at revision: 855
Proposed branch: lp:~3v1n0/nux/check-dnd_area
Merge into: lp:nux
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
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Review via email: mp+237150@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 :

LGTM

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 2014-07-30 00:23:32 +0000
+++ Nux/WindowCompositor.cpp 2014-10-03 22:55:18 +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