From d16362731992fb3c9f1957adbe1d862830ec6139 Mon Sep 17 00:00:00 2001
From: Giannis Adamopoulos <gadamopoulos@reactos.org>
Date: Thu, 7 Dec 2017 23:24:32 +0200
Subject: [PATCH] [BROWSEUI] Move _ILIsDesktop to a shared header to avoid
 having 3 instances of it

---
 dll/win32/browseui/explorerband.cpp       |  6 ------
 dll/win32/browseui/shellbars/CBandSite.h  |  5 +++++
 dll/win32/browseui/shellbars/CISFBand.cpp | 18 ------------------
 dll/win32/browseui/shellbrowser.cpp       |  5 -----
 4 files changed, 5 insertions(+), 29 deletions(-)

diff --git a/dll/win32/browseui/explorerband.cpp b/dll/win32/browseui/explorerband.cpp
index 2c8cbe8da94..2b398bd2f1d 100644
--- a/dll/win32/browseui/explorerband.cpp
+++ b/dll/win32/browseui/explorerband.cpp
@@ -54,12 +54,6 @@ static BOOL _ILIsSpecialFolder (LPCITEMIDLIST pidl)
         PT_YAGUID == lpPData->type)) || (pidl && pidl->mkid.cb == 0x00)));
 }
 
-static BOOL _ILIsDesktop (LPCITEMIDLIST pidl)
-{
-    return (pidl && pidl->mkid.cb == 0x00);
-}
-
-
 HRESULT GetDisplayName(LPCITEMIDLIST pidlDirectory,TCHAR *szDisplayName,UINT cchMax,DWORD uFlags)
 {
     IShellFolder *pShellFolder = NULL;
diff --git a/dll/win32/browseui/shellbars/CBandSite.h b/dll/win32/browseui/shellbars/CBandSite.h
index f18cedffb88..c515fbcd054 100644
--- a/dll/win32/browseui/shellbars/CBandSite.h
+++ b/dll/win32/browseui/shellbars/CBandSite.h
@@ -21,6 +21,11 @@
 
 #pragma once
 
+extern inline BOOL _ILIsDesktop(LPCITEMIDLIST pidl)
+{
+    return (pidl == NULL || pidl->mkid.cb == 0);
+}
+
 class CBandSiteBase :
     public CComObjectRootEx<CComMultiThreadModelNoCS>,
     public IBandSite,
diff --git a/dll/win32/browseui/shellbars/CISFBand.cpp b/dll/win32/browseui/shellbars/CISFBand.cpp
index 042a10640e9..24426f6ebda 100644
--- a/dll/win32/browseui/shellbars/CISFBand.cpp
+++ b/dll/win32/browseui/shellbars/CISFBand.cpp
@@ -22,24 +22,6 @@ TODO:
     ** Implement responding to theme change
 */
 
-
-// ***Extras***
-/*++
-* @name _ILIsDesktop
-*
-* Checks whether the given PIDL is of Desktop folder or not.
-*
-* @param pidl
-*        PIDL to be checked.
-*
-* @return True if PIDL is of Desktop, otherwise false.
-*
-*--*/
-static BOOL _ILIsDesktop(LPCITEMIDLIST pidl)
-{
-    return (pidl == NULL || pidl->mkid.cb == 0);
-}
-
 //*****************************************************************************************
 // *** CISFBand *** 
 
diff --git a/dll/win32/browseui/shellbrowser.cpp b/dll/win32/browseui/shellbrowser.cpp
index b95a4e331d0..8126cefe3d0 100644
--- a/dll/win32/browseui/shellbrowser.cpp
+++ b/dll/win32/browseui/shellbrowser.cpp
@@ -818,11 +818,6 @@ HRESULT CShellBrowser::BrowseToPIDL(LPCITEMIDLIST pidl, long flags)
     return S_OK;
 }
 
-BOOL WINAPI _ILIsDesktop(LPCITEMIDLIST pidl)
-{
-    return (pidl == NULL || pidl->mkid.cb == 0);
-}
-
 BOOL WINAPI _ILIsPidlSimple(LPCITEMIDLIST pidl)
 {
     LPCITEMIDLIST                           pidlnext;
-- 
2.17.1