mirror of
https://github.com/gryf/wmaker.git
synced 2025-12-29 09:52:29 +01:00
WPrefs: Removed unused argument to function 'handleDragOver'
The x coordinate is not needed by the function because menu are a vertical stacking on items, which means only y coordinate matters. This patch removes the x parameter to fix a compilation warning. Signed-off-by: Christophe CURIS <christophe.curis@free.fr>
This commit is contained in:
committed by
Carlos R. Mafra
parent
e4664c6492
commit
f5f367841b
@@ -1001,7 +1001,7 @@ static WEditMenu *findMenuInWindow(Display * dpy, Window toplevel, int x, int y)
|
|||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void handleDragOver(WEditMenu * menu, WMView * view, WEditMenuItem * item, int x, int y)
|
static void handleDragOver(WEditMenu *menu, WMView *view, WEditMenuItem *item, int y)
|
||||||
{
|
{
|
||||||
WMScreen *scr = W_VIEW_SCREEN(menu->view);
|
WMScreen *scr = W_VIEW_SCREEN(menu->view);
|
||||||
Window blaw;
|
Window blaw;
|
||||||
@@ -1019,7 +1019,7 @@ static void handleDragOver(WEditMenu * menu, WMView * view, WEditMenuItem * item
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void handleItemDrop(WEditMenu * menu, WEditMenuItem * item, int x, int y)
|
static void handleItemDrop(WEditMenu *menu, WEditMenuItem *item, int y)
|
||||||
{
|
{
|
||||||
WMScreen *scr = W_VIEW_SCREEN(menu->view);
|
WMScreen *scr = W_VIEW_SCREEN(menu->view);
|
||||||
Window blaw;
|
Window blaw;
|
||||||
@@ -1225,7 +1225,7 @@ static void dragItem(WEditMenu * menu, WEditMenuItem * item, Bool copy)
|
|||||||
dmenu = findMenuInWindow(dpy, win, x, y);
|
dmenu = findMenuInWindow(dpy, win, x, y);
|
||||||
|
|
||||||
if (dmenu) {
|
if (dmenu) {
|
||||||
handleDragOver(dmenu, dview, dritem, x - dx, y - dy);
|
handleDragOver(dmenu, dview, dritem, y - dy);
|
||||||
if (!enteredMenu) {
|
if (!enteredMenu) {
|
||||||
enteredMenu = True;
|
enteredMenu = True;
|
||||||
XRecolorCursor(dpy, scr->defaultCursor, &green, &back);
|
XRecolorCursor(dpy, scr->defaultCursor, &green, &back);
|
||||||
@@ -1271,7 +1271,7 @@ static void dragItem(WEditMenu * menu, WEditMenuItem * item, Bool copy)
|
|||||||
|
|
||||||
if (!menu->flags.isFactory && !copy) {
|
if (!menu->flags.isFactory && !copy) {
|
||||||
WRemoveEditMenuItem(menu, dritem);
|
WRemoveEditMenuItem(menu, dritem);
|
||||||
handleItemDrop(dmenu ? dmenu : menu, dritem, orix, oriy);
|
handleItemDrop(dmenu ? dmenu : menu, dritem, oriy);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
WRemoveEditMenuItem(menu, dritem);
|
WRemoveEditMenuItem(menu, dritem);
|
||||||
@@ -1283,7 +1283,7 @@ static void dragItem(WEditMenu * menu, WEditMenuItem * item, Bool copy)
|
|||||||
(*menu->delegate->itemCloned) (menu->delegate, menu, item, dritem);
|
(*menu->delegate->itemCloned) (menu->delegate, menu, item, dritem);
|
||||||
}
|
}
|
||||||
|
|
||||||
handleItemDrop(dmenu, dritem, x - dy, y - dy);
|
handleItemDrop(dmenu, dritem, y - dy);
|
||||||
|
|
||||||
if (item->submenu && (menu->flags.isFactory || copy)) {
|
if (item->submenu && (menu->flags.isFactory || copy)) {
|
||||||
WEditMenu *submenu;
|
WEditMenu *submenu;
|
||||||
|
|||||||
Reference in New Issue
Block a user