From 1cab8406817abd9ad49efa2f339cd0d2ec9ab11c Mon Sep 17 00:00:00 2001 From: David Quintana Date: Fri, 9 Jan 2015 16:32:53 +0000 Subject: [PATCH] Revert files that shouldn't have been included. I blame an ongoing headache for it. svn path=/trunk/; revision=66017 --- reactos/base/shell/explorer/trayntfy.cpp | 35 ++++++++++++---------- reactos/base/shell/rshell/CMenuDeskBar.cpp | 2 -- 2 files changed, 20 insertions(+), 17 deletions(-) diff --git a/reactos/base/shell/explorer/trayntfy.cpp b/reactos/base/shell/explorer/trayntfy.cpp index f4a79f79a57..adc5ae05361 100644 --- a/reactos/base/shell/explorer/trayntfy.cpp +++ b/reactos/base/shell/explorer/trayntfy.cpp @@ -82,7 +82,7 @@ public: return -1; } - BOOL AddButton(IN CONST NOTIFYICONDATA *iconData) + VOID AddButton(IN CONST NOTIFYICONDATA *iconData) { TBBUTTON tbBtn; NOTIFYICONDATA * notifyItem; @@ -91,7 +91,8 @@ public: int index = FindItemByIconData(iconData, ¬ifyItem); if (index >= 0) { - return UpdateButton(iconData); + UpdateButton(iconData); + return; } notifyItem = new NOTIFYICONDATA(); @@ -137,11 +138,9 @@ public: CToolbar::AddButton(&tbBtn); SetButtonSize(ICON_SIZE, ICON_SIZE); - - return TRUE; } - BOOL UpdateButton(IN CONST NOTIFYICONDATA *iconData) + VOID UpdateButton(IN CONST NOTIFYICONDATA *iconData) { NOTIFYICONDATA * notifyItem; TBBUTTONINFO tbbi = { 0 }; @@ -149,7 +148,8 @@ public: int index = FindItemByIconData(iconData, ¬ifyItem); if (index < 0) { - return AddButton(iconData); + AddButton(iconData); + return; } tbbi.cbSize = sizeof(tbbi); @@ -197,17 +197,15 @@ public: /* TODO: support NIF_INFO, NIF_GUID, NIF_REALTIME, NIF_SHOWTIP */ SetButtonInfo(index, &tbbi); - - return TRUE; } - BOOL RemoveButton(IN CONST NOTIFYICONDATA *iconData) + VOID RemoveButton(IN CONST NOTIFYICONDATA *iconData) { NOTIFYICONDATA * notifyItem; int index = FindItemByIconData(iconData, ¬ifyItem); if (index < 0) - return FALSE; + return; DeleteButton(index); @@ -217,8 +215,6 @@ public: } delete notifyItem; - - return TRUE; } VOID GetTooltipText(int index, LPTSTR szTip, DWORD cchTip) @@ -436,11 +432,20 @@ public: switch (data->notify_code) { case NIM_ADD: - return Toolbar.AddButton(iconData); + { + Toolbar.AddButton(iconData); + break; + } case NIM_MODIFY: - return Toolbar.UpdateButton(iconData); + { + Toolbar.UpdateButton(iconData); + break; + } case NIM_DELETE: - return Toolbar.RemoveButton(iconData); + { + Toolbar.RemoveButton(iconData); + break; + } default: TRACE("NotifyMessage received with unknown code %d.\n", data->notify_code); break; diff --git a/reactos/base/shell/rshell/CMenuDeskBar.cpp b/reactos/base/shell/rshell/CMenuDeskBar.cpp index b5e113a5be4..d068283d54c 100644 --- a/reactos/base/shell/rshell/CMenuDeskBar.cpp +++ b/reactos/base/shell/rshell/CMenuDeskBar.cpp @@ -338,8 +338,6 @@ HRESULT STDMETHODCALLTYPE CMenuDeskBar::Popup(POINTL *ppt, RECTL *prcExclude, MP x = ppt->x + rc.left; y = ppt->y + rc.top; break; - case MPPF_TOP: - break; case MPPF_TOP | MPPF_ALIGN_LEFT: x = ppt->x - rc.right; y = ppt->y + rc.top;