From f53797f9ef491069b7b2cbbabbd9a59eb12aabaa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20D=C3=A9chelotte?= Date: Mon, 7 Oct 2013 23:49:28 +0200 Subject: [PATCH] Remove dead code for a confirmation dialog that we'll probably never want to show Setting a clip as autoattracting will disable an autoattracting drawer, and reciprocally. Although not immediately obvious, it should be fairly easy to figure out, and is totally reversible. So we'll probably never opt to show that confirmation dialog box. --- src/dock.c | 26 -------------------------- 1 file changed, 26 deletions(-) diff --git a/src/dock.c b/src/dock.c index 29091e3e..1e0405cc 100644 --- a/src/dock.c +++ b/src/dock.c @@ -583,32 +583,6 @@ static void toggleAutoAttractCallback(WMenu *menu, WMenuEntry *entry) assert(entry->clientdata != NULL); - /* Setting a clip as autoattracting will disable an autoattracting drawer, - * and reciprocally. Ask for confirmation? */ -#ifdef CONFIRM_AUTOATTRACT_CHANGE - if (dock->type == WM_CLIP && scr->attracting_drawer) { - if (wMessageDialog(scr, _("Workspace Clip"), - _("The drawer that auto-attracted icons so far won't do so anymore!"), - _("OK"), _("Cancel"), NULL) != WAPRDefault) - return; - } else if (dock->type == WM_DRAWER) { - /* check if at least one clip already auto attracts icons */ - int i, ask_for_confirmation = False; - for (i = 0; i < w_global.workspace.count; i++) { - if (w_global.workspace.array[i]->clip->attract_icons) { - ask_for_confirmation = True; - break; - } - } - if (ask_for_confirmation) { - if (wMessageDialog(scr, _("Drawer"), - _("Any clips that auto-attracted icons so far won't do so anymore!"), - _("OK"), _("Cancel"), NULL) != WAPRDefault) - return; - } - } -#endif - dock->attract_icons = !dock->attract_icons; entry->flags.indicator_on = dock->attract_icons;