19 Nov
2013
19 Nov
'13
7:05 p.m.
Author: gadamopoulos
Date: Tue Nov 19 19:05:17 2013
New Revision: 61048
URL: http://svn.reactos.org/svn/reactos?rev=61048&view=rev
Log:
[shell32]
- Append class members with m_ in CDefView class
Modified:
trunk/reactos/dll/win32/shell32/shlview.cpp
Modified: trunk/reactos/dll/win32/shell32/shlview.cpp
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/shlview.…
==============================================================================
--- trunk/reactos/dll/win32/shell32/shlview.cpp [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/shell32/shlview.cpp [iso-8859-1] Tue Nov 19 19:05:17 2013
@@ -76,31 +76,31 @@
public IServiceProvider
{
private:
- CComPtr<IShellFolder> pSFParent;
- CComPtr<IShellFolder2> pSF2Parent;
- CComPtr<IShellBrowser> pShellBrowser;
- CComPtr<ICommDlgBrowser> pCommDlgBrowser;
- HWND hWndList; /* ListView control */
- HWND hWndParent;
- FOLDERSETTINGS FolderSettings;
- HMENU hMenu;
- UINT uState;
- UINT cidl;
- LPITEMIDLIST *apidl;
- LISTVIEW_SORT_INFO ListViewSortInfo;
- ULONG hNotify; /* change notification
handle */
- HACCEL hAccel;
- DWORD dwAspects;
- DWORD dwAdvf;
- CComPtr<IAdviseSink> pAdvSink;
+ CComPtr<IShellFolder> m_pSFParent;
+ CComPtr<IShellFolder2> m_pSF2Parent;
+ CComPtr<IShellBrowser> m_pShellBrowser;
+ CComPtr<ICommDlgBrowser> m_pCommDlgBrowser;
+ HWND m_hWndList; /* ListView control */
+ HWND m_hWndParent;
+ FOLDERSETTINGS m_FolderSettings;
+ HMENU m_hMenu;
+ UINT m_uState;
+ UINT m_cidl;
+ LPITEMIDLIST *m_apidl;
+ LISTVIEW_SORT_INFO m_sortInfo;
+ ULONG m_hNotify; /* change notification handle */
+ HACCEL m_hAccel;
+ DWORD m_dwAspects;
+ DWORD m_dwAdvf;
+ CComPtr<IAdviseSink> m_pAdvSink;
// for drag and drop
- CComPtr<IDropTarget> pCurDropTarget; /* The sub-item,
which is currently dragged over */
- CComPtr<IDataObject> pCurDataObject; /* The dragged
data-object */
- LONG iDragOverItem; /* Dragged over
item's index, iff pCurDropTarget != NULL */
- UINT cScrollDelay; /* Send a WM_*SCROLL msg
every 250 ms during drag-scroll */
- POINT ptLastMousePos; /* Mouse position at
last DragOver call */
+ CComPtr<IDropTarget> m_pCurDropTarget; /* The sub-item, which
is currently dragged over */
+ CComPtr<IDataObject> m_pCurDataObject; /* The dragged
data-object */
+ LONG m_iDragOverItem; /* Dragged over item's
index, iff m_pCurDropTarget != NULL */
+ UINT m_cScrollDelay; /* Send a WM_*SCROLL msg every
250 ms during drag-scroll */
+ POINT m_ptLastMousePos; /* Mouse position at last
DragOver call */
//
- CComPtr<IContextMenu> pCM;
+ CComPtr<IContextMenu> m_pCM;
public:
CDefView();
~CDefView();
@@ -319,38 +319,38 @@
CDefView::CDefView()
{
- hWndList = NULL;
- hWndParent = NULL;
- FolderSettings.fFlags = 0;
- FolderSettings.ViewMode = 0;
- hMenu = NULL;
- uState = 0;
- cidl = 0;
- apidl = NULL;
- ListViewSortInfo.bIsAscending = FALSE;
- ListViewSortInfo.nHeaderID = 0;
- ListViewSortInfo.nLastHeaderID = 0;
- hNotify = 0;
- hAccel = NULL;
- dwAspects = 0;
- dwAdvf = 0;
- iDragOverItem = 0;
- cScrollDelay = 0;
- ptLastMousePos.x = 0;
- ptLastMousePos.y = 0;
+ m_hWndList = NULL;
+ m_hWndParent = NULL;
+ m_FolderSettings.fFlags = 0;
+ m_FolderSettings.ViewMode = 0;
+ m_hMenu = NULL;
+ m_uState = 0;
+ m_cidl = 0;
+ m_apidl = NULL;
+ m_sortInfo.bIsAscending = FALSE;
+ m_sortInfo.nHeaderID = 0;
+ m_sortInfo.nLastHeaderID = 0;
+ m_hNotify = 0;
+ m_hAccel = NULL;
+ m_dwAspects = 0;
+ m_dwAdvf = 0;
+ m_iDragOverItem = 0;
+ m_cScrollDelay = 0;
+ m_ptLastMousePos.x = 0;
+ m_ptLastMousePos.y = 0;
}
CDefView::~CDefView()
{
TRACE(" destroying IShellView(%p)\n", this);
- SHFree(apidl);
+ SHFree(m_apidl);
}
HRESULT WINAPI CDefView::Initialize(IShellFolder *shellFolder)
{
- pSFParent = shellFolder;
- shellFolder->QueryInterface(IID_PPV_ARG(IShellFolder2, &pSF2Parent));
+ m_pSFParent = shellFolder;
+ shellFolder->QueryInterface(IID_PPV_ARG(IShellFolder2, &m_pSF2Parent));
return S_OK;
}
@@ -363,10 +363,10 @@
{
HRESULT ret = S_OK;
- if (pCommDlgBrowser.p != NULL)
+ if (m_pCommDlgBrowser.p != NULL)
{
TRACE("ICommDlgBrowser::IncludeObject pidl=%p\n", pidl);
- ret = pCommDlgBrowser->IncludeObject((IShellView *)this, pidl);
+ ret = m_pCommDlgBrowser->IncludeObject((IShellView *)this, pidl);
TRACE("--0x%08x\n", ret);
}
@@ -377,10 +377,10 @@
{
HRESULT ret = S_FALSE;
- if (pCommDlgBrowser.p != NULL)
+ if (m_pCommDlgBrowser.p != NULL)
{
TRACE("ICommDlgBrowser::OnDefaultCommand\n");
- ret = pCommDlgBrowser->OnDefaultCommand((IShellView *)this);
+ ret = m_pCommDlgBrowser->OnDefaultCommand((IShellView *)this);
TRACE("-- returns %08x\n", ret);
}
@@ -391,10 +391,10 @@
{
HRESULT ret = S_FALSE;
- if (pCommDlgBrowser.p != NULL)
+ if (m_pCommDlgBrowser.p != NULL)
{
TRACE("ICommDlgBrowser::OnStateChange flags=%x\n", uFlags);
- ret = pCommDlgBrowser->OnStateChange((IShellView *)this, uFlags);
+ ret = m_pCommDlgBrowser->OnStateChange((IShellView *)this, uFlags);
TRACE("--\n");
}
@@ -412,15 +412,15 @@
TRACE("\n");
- if (pCommDlgBrowser != NULL)
- {
- pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_CHECKBUTTON,
- FCIDM_TB_SMALLICON, (FolderSettings.ViewMode ==
FVM_LIST) ? TRUE : FALSE, &result);
- pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_CHECKBUTTON,
- FCIDM_TB_REPORTVIEW, (FolderSettings.ViewMode ==
FVM_DETAILS) ? TRUE : FALSE, &result);
- pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_ENABLEBUTTON,
+ if (m_pCommDlgBrowser != NULL)
+ {
+ m_pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_CHECKBUTTON,
+ FCIDM_TB_SMALLICON, (m_FolderSettings.ViewMode ==
FVM_LIST) ? TRUE : FALSE, &result);
+ m_pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_CHECKBUTTON,
+ FCIDM_TB_REPORTVIEW, (m_FolderSettings.ViewMode ==
FVM_DETAILS) ? TRUE : FALSE, &result);
+ m_pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_ENABLEBUTTON,
FCIDM_TB_SMALLICON, TRUE, &result);
- pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_ENABLEBUTTON,
+ m_pShellBrowser->SendControlMsg(FCW_TOOLBAR, TB_ENABLEBUTTON,
FCIDM_TB_REPORTVIEW, TRUE, &result);
}
}
@@ -438,8 +438,8 @@
TRACE("(%p)\n", this);
- tmpstyle = ::GetWindowLongPtrW(hWndList, GWL_STYLE);
- ::SetWindowLongPtrW(hWndList, GWL_STYLE, dwAdd | (tmpstyle & ~dwRemove));
+ tmpstyle = ::GetWindowLongPtrW(m_hWndList, GWL_STYLE);
+ ::SetWindowLongPtrW(m_hWndList, GWL_STYLE, dwAdd | (tmpstyle & ~dwRemove));
}
/**********************************************************
@@ -456,12 +456,12 @@
LVS_SHAREIMAGELISTS | LVS_EDITLABELS | LVS_AUTOARRANGE;
dwExStyle = WS_EX_CLIENTEDGE;
- if (FolderSettings.fFlags & FWF_DESKTOP)
+ if (m_FolderSettings.fFlags & FWF_DESKTOP)
dwStyle |= LVS_ALIGNLEFT;
else
dwStyle |= LVS_ALIGNTOP;
- switch (FolderSettings.ViewMode)
+ switch (m_FolderSettings.ViewMode)
{
case FVM_ICON:
dwStyle |= LVS_ICON;
@@ -484,34 +484,34 @@
break;
}
- if (FolderSettings.fFlags & FWF_AUTOARRANGE)
+ if (m_FolderSettings.fFlags & FWF_AUTOARRANGE)
dwStyle |= LVS_AUTOARRANGE;
- if (FolderSettings.fFlags & FWF_DESKTOP)
- FolderSettings.fFlags |= FWF_NOCLIENTEDGE | FWF_NOSCROLL;
-
- if (FolderSettings.fFlags & FWF_SINGLESEL)
+ if (m_FolderSettings.fFlags & FWF_DESKTOP)
+ m_FolderSettings.fFlags |= FWF_NOCLIENTEDGE | FWF_NOSCROLL;
+
+ if (m_FolderSettings.fFlags & FWF_SINGLESEL)
dwStyle |= LVS_SINGLESEL;
- if (FolderSettings.fFlags & FWF_NOCLIENTEDGE)
+ if (m_FolderSettings.fFlags & FWF_NOCLIENTEDGE)
dwExStyle &= ~WS_EX_CLIENTEDGE;
- hWndList = CreateWindowExW( dwExStyle,
- WC_LISTVIEWW,
- NULL,
- dwStyle,
- 0, 0, 0, 0,
- m_hWnd,
- (HMENU)ID_LISTVIEW,
- shell32_hInstance,
- NULL);
-
- if (!hWndList)
+ m_hWndList = CreateWindowExW( dwExStyle,
+ WC_LISTVIEWW,
+ NULL,
+ dwStyle,
+ 0, 0, 0, 0,
+ m_hWnd,
+ (HMENU)ID_LISTVIEW,
+ shell32_hInstance,
+ NULL);
+
+ if (!m_hWndList)
return FALSE;
- ListViewSortInfo.bIsAscending = TRUE;
- ListViewSortInfo.nHeaderID = -1;
- ListViewSortInfo.nLastHeaderID = -1;
+ m_sortInfo.bIsAscending = TRUE;
+ m_sortInfo.nHeaderID = -1;
+ m_sortInfo.nLastHeaderID = -1;
UpdateListColors();
@@ -521,7 +521,7 @@
void CDefView::UpdateListColors()
{
- if (FolderSettings.fFlags & FWF_DESKTOP)
+ if (m_FolderSettings.fFlags & FWF_DESKTOP)
{
/* Check if drop shadows option is enabled */
BOOL bDropShadow = FALSE;
@@ -532,21 +532,21 @@
L"ListviewShadow", RRF_RT_DWORD, NULL, &bDropShadow,
&cbDropShadow);
if (bDropShadow && SystemParametersInfoW(SPI_GETDESKWALLPAPER,
_countof(wszBuf), wszBuf, 0) && wszBuf[0])
{
- SendMessageW(hWndList, LVM_SETTEXTBKCOLOR, 0, CLR_NONE);
- SendMessageW(hWndList, LVM_SETBKCOLOR, 0, CLR_NONE);
- SendMessageW(hWndList, LVM_SETTEXTCOLOR, 0, RGB(255, 255, 255));
- SendMessageW(hWndList, LVM_SETEXTENDEDLISTVIEWSTYLE,
LVS_EX_TRANSPARENTSHADOWTEXT, LVS_EX_TRANSPARENTSHADOWTEXT);
+ SendMessageW(m_hWndList, LVM_SETTEXTBKCOLOR, 0, CLR_NONE);
+ SendMessageW(m_hWndList, LVM_SETBKCOLOR, 0, CLR_NONE);
+ SendMessageW(m_hWndList, LVM_SETTEXTCOLOR, 0, RGB(255, 255, 255));
+ SendMessageW(m_hWndList, LVM_SETEXTENDEDLISTVIEWSTYLE,
LVS_EX_TRANSPARENTSHADOWTEXT, LVS_EX_TRANSPARENTSHADOWTEXT);
}
else
{
COLORREF crDesktop = GetSysColor(COLOR_DESKTOP);
- SendMessageW(hWndList, LVM_SETTEXTBKCOLOR, 0, crDesktop);
- SendMessageW(hWndList, LVM_SETBKCOLOR, 0, crDesktop);
+ SendMessageW(m_hWndList, LVM_SETTEXTBKCOLOR, 0, crDesktop);
+ SendMessageW(m_hWndList, LVM_SETBKCOLOR, 0, crDesktop);
if (GetRValue(crDesktop) + GetGValue(crDesktop) + GetBValue(crDesktop) >
128 * 3)
- SendMessageW(hWndList, LVM_SETTEXTCOLOR, 0, RGB(0, 0, 0));
+ SendMessageW(m_hWndList, LVM_SETTEXTCOLOR, 0, RGB(0, 0, 0));
else
- SendMessageW(hWndList, LVM_SETTEXTCOLOR, 0, RGB(255, 255, 255));
- SendMessageW(hWndList, LVM_SETEXTENDEDLISTVIEWSTYLE,
LVS_EX_TRANSPARENTSHADOWTEXT, 0);
+ SendMessageW(m_hWndList, LVM_SETTEXTCOLOR, 0, RGB(255, 255, 255));
+ SendMessageW(m_hWndList, LVM_SETEXTENDEDLISTVIEWSTYLE,
LVS_EX_TRANSPARENTSHADOWTEXT, 0);
}
}
}
@@ -564,22 +564,22 @@
TRACE("%p\n", this);
- SendMessageW(hWndList, LVM_DELETEALLITEMS, 0, 0);
+ SendMessageW(m_hWndList, LVM_DELETEALLITEMS, 0, 0);
lvColumn.mask = LVCF_FMT | LVCF_WIDTH | LVCF_TEXT;
lvColumn.pszText = szTemp;
- if (pSF2Parent)
+ if (m_pSF2Parent)
{
for (int i = 0; 1; i++)
{
- if (FAILED(pSF2Parent->GetDetailsOf(NULL, i, &sd)))
+ if (FAILED(m_pSF2Parent->GetDetailsOf(NULL, i, &sd)))
break;
lvColumn.fmt = sd.fmt;
lvColumn.cx = sd.cxChar * 8; /* chars->pixel */
StrRetToStrNW( szTemp, 50, &sd.str, NULL);
- SendMessageW(hWndList, LVM_INSERTCOLUMNW, i, (LPARAM) &lvColumn);
+ SendMessageW(m_hWndList, LVM_INSERTCOLUMNW, i, (LPARAM) &lvColumn);
}
}
else
@@ -587,8 +587,8 @@
FIXME("no SF2\n");
}
- SendMessageW(hWndList, LVM_SETIMAGELIST, LVSIL_SMALL, (LPARAM)ShellSmallIconList);
- SendMessageW(hWndList, LVM_SETIMAGELIST, LVSIL_NORMAL, (LPARAM)ShellBigIconList);
+ SendMessageW(m_hWndList, LVM_SETIMAGELIST, LVSIL_SMALL, (LPARAM)ShellSmallIconList);
+ SendMessageW(m_hWndList, LVM_SETIMAGELIST, LVSIL_NORMAL, (LPARAM)ShellBigIconList);
return TRUE;
}
@@ -723,11 +723,11 @@
lvItem.mask = LVIF_PARAM;
for (lvItem.iItem = 0;
- SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
lvItem.iItem++)
{
LPITEMIDLIST currentpidl = (LPITEMIDLIST) lvItem.lParam;
- HRESULT hr = pSFParent->CompareIDs(0, pidl, currentpidl);
+ HRESULT hr = m_pSFParent->CompareIDs(0, pidl, currentpidl);
if (SUCCEEDED(hr) && !HRESULT_CODE(hr))
{
@@ -747,13 +747,13 @@
TRACE("(%p)(pidl=%p)\n", this, pidl);
lvItem.mask = LVIF_TEXT | LVIF_IMAGE | LVIF_PARAM; /*set the mask*/
- lvItem.iItem = ListView_GetItemCount(hWndList); /*add the item to the end of the
list*/
+ lvItem.iItem = ListView_GetItemCount(m_hWndList); /*add the item to the end of the
list*/
lvItem.iSubItem = 0;
lvItem.lParam = (LPARAM) ILClone(ILFindLastID(pidl)); /*set the
item's data*/
lvItem.pszText = LPSTR_TEXTCALLBACKW; /*get text on a callback basis*/
lvItem.iImage = I_IMAGECALLBACK; /*get the image on a callback basis*/
- if (SendMessageW(hWndList, LVM_INSERTITEMW, 0, (LPARAM)&lvItem) == -1)
+ if (SendMessageW(m_hWndList, LVM_INSERTITEMW, 0, (LPARAM)&lvItem) == -1)
return FALSE;
else
return TRUE;
@@ -770,7 +770,7 @@
nIndex = LV_FindItemByPidl(ILFindLastID(pidl));
- return (-1 == ListView_DeleteItem(hWndList, nIndex)) ? FALSE : TRUE;
+ return (-1 == ListView_DeleteItem(m_hWndList, nIndex)) ? FALSE : TRUE;
}
/**********************************************************
@@ -789,15 +789,15 @@
{
lvItem.mask = LVIF_PARAM; /* only the pidl */
lvItem.iItem = nItem;
- SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
SHFree((LPITEMIDLIST)lvItem.lParam);
lvItem.mask = LVIF_PARAM|LVIF_IMAGE;
lvItem.iItem = nItem;
lvItem.lParam = (LPARAM) ILClone(ILFindLastID(pidlNew)); /* set the item's
data */
- lvItem.iImage = SHMapPIDLToSystemImageListIndex(pSFParent, pidlNew, 0);
- SendMessageW(hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
- SendMessageW(hWndList, LVM_UPDATE, nItem, 0);
+ lvItem.iImage = SHMapPIDLToSystemImageListIndex(m_pSFParent, pidlNew, 0);
+ SendMessageW(m_hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_UPDATE, nItem, 0);
return TRUE; /* FIXME: better handling */
}
@@ -834,7 +834,7 @@
TRACE("%p\n", this);
/* get the itemlist from the shfolder*/
- hRes = pSFParent->EnumObjects(m_hWnd, SHCONTF_NONFOLDERS | SHCONTF_FOLDERS,
&pEnumIDList);
+ hRes = m_pSFParent->EnumObjects(m_hWnd, SHCONTF_NONFOLDERS | SHCONTF_FOLDERS,
&pEnumIDList);
if (hRes != S_OK)
{
if (hRes == S_FALSE)
@@ -859,15 +859,15 @@
}
/* sort the array */
- DPA_Sort(hdpa, CompareItems, (LPARAM)pSFParent.p);
+ DPA_Sort(hdpa, CompareItems, (LPARAM)m_pSFParent.p);
/*turn the listview's redrawing off*/
- SendMessageA(hWndList, WM_SETREDRAW, FALSE, 0);
+ SendMessageA(m_hWndList, WM_SETREDRAW, FALSE, 0);
DPA_DestroyCallback( hdpa, fill_list, (void *)this);
/*turn the listview's redrawing back on and force it to draw*/
- SendMessageA(hWndList, WM_SETREDRAW, TRUE, 0);
+ SendMessageA(m_hWndList, WM_SETREDRAW, TRUE, 0);
pEnumIDList->Release(); /* destroy the list*/
@@ -876,27 +876,27 @@
LRESULT CDefView::OnShowWindow(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL
&bHandled)
{
- ::UpdateWindow(hWndList);
+ ::UpdateWindow(m_hWndList);
bHandled = FALSE;
return 0;
}
LRESULT CDefView::OnGetDlgCode(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL
&bHandled)
{
- return SendMessageW(hWndList, uMsg, 0, 0);
+ return SendMessageW(m_hWndList, uMsg, 0, 0);
}
LRESULT CDefView::OnDestroy(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled)
{
RevokeDragDrop(m_hWnd);
- SHChangeNotifyDeregister(hNotify);
+ SHChangeNotifyDeregister(m_hNotify);
bHandled = FALSE;
return 0;
}
LRESULT CDefView::OnEraseBackground(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL
&bHandled)
{
- if (FolderSettings.fFlags & (FWF_DESKTOP | FWF_TRANSPARENT))
+ if (m_FolderSettings.fFlags & (FWF_DESKTOP | FWF_TRANSPARENT))
return SendMessageW(GetParent(), WM_ERASEBKGND, wParam, lParam); /* redirect to
parent */
bHandled = FALSE;
@@ -909,12 +909,12 @@
UpdateListColors();
/* Forward WM_SYSCOLORCHANGE to common controls */
- return SendMessageW(hWndList, uMsg, 0, 0);
+ return SendMessageW(m_hWndList, uMsg, 0, 0);
}
LRESULT CDefView::OnGetShellBrowser(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL
&bHandled)
{
- return (LRESULT)pShellBrowser.p;
+ return (LRESULT)m_pShellBrowser.p;
}
/**********************************************************
@@ -940,16 +940,16 @@
RegisterDragDrop(m_hWnd, pdt);
/* register for receiving notifications */
- pSFParent->QueryInterface(IID_PPV_ARG(IPersistFolder2, &ppf2));
+ m_pSFParent->QueryInterface(IID_PPV_ARG(IPersistFolder2, &ppf2));
if (ppf2)
{
ppf2->GetCurFolder((LPITEMIDLIST*)&ntreg.pidl);
ntreg.fRecursive = TRUE;
- hNotify = SHChangeNotifyRegister(m_hWnd, SHCNF_IDLIST, SHCNE_ALLEVENTS,
SHV_CHANGE_NOTIFY, 1, &ntreg);
+ m_hNotify = SHChangeNotifyRegister(m_hWnd, SHCNF_IDLIST, SHCNE_ALLEVENTS,
SHV_CHANGE_NOTIFY, 1, &ntreg);
SHFree((LPITEMIDLIST)ntreg.pidl);
}
- hAccel = LoadAcceleratorsA(shell32_hInstance, MAKEINTRESOURCEA( IDA_SHELLVIEW));
+ m_hAccel = LoadAcceleratorsA(shell32_hInstance, MAKEINTRESOURCEA( IDA_SHELLVIEW));
return S_OK;
}
@@ -1048,7 +1048,7 @@
/**********************************************************
* ShellView_GetSelections()
*
-* - fills the this->apidl list with the selected objects
+* - fills the m_apidl list with the selected objects
*
* RETURNS
* number of selected items
@@ -1058,16 +1058,16 @@
LVITEMW lvItem;
UINT i = 0;
- SHFree(apidl);
-
- cidl = ListView_GetSelectedCount(hWndList);
- apidl = (LPITEMIDLIST*)SHAlloc(cidl * sizeof(LPITEMIDLIST));
-
- TRACE("selected=%i\n", cidl);
-
- if (apidl)
- {
- TRACE("-- Items selected =%u\n", cidl);
+ SHFree(m_apidl);
+
+ m_cidl = ListView_GetSelectedCount(m_hWndList);
+ m_apidl = (LPITEMIDLIST*)SHAlloc(m_cidl * sizeof(LPITEMIDLIST));
+
+ TRACE("selected=%i\n", m_cidl);
+
+ if (m_apidl)
+ {
+ TRACE("-- Items selected =%u\n", m_cidl);
lvItem.mask = LVIF_STATE | LVIF_PARAM;
lvItem.stateMask = LVIS_SELECTED;
@@ -1075,13 +1075,13 @@
lvItem.iSubItem = 0;
lvItem.state = 0;
- while(SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM)&lvItem) && (i
< cidl))
+ while(SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM)&lvItem) &&
(i < m_cidl))
{
if(lvItem.state & LVIS_SELECTED)
{
- apidl[i] = (LPITEMIDLIST)lvItem.lParam;
+ m_apidl[i] = (LPITEMIDLIST)lvItem.lParam;
i++;
- if (i == cidl)
+ if (i == m_cidl)
break;
TRACE("-- selected Item found\n");
}
@@ -1089,7 +1089,7 @@
}
}
- return cidl;
+ return m_cidl;
}
/**********************************************************
@@ -1102,8 +1102,8 @@
UINT uCommand;
HRESULT hResult;
- cidl = ListView_GetSelectedCount(hWndList);
- if (cidl == 0)
+ m_cidl = ListView_GetSelectedCount(m_hWndList);
+ if (m_cidl == 0)
return S_OK;
hResult = OnDefaultCommand();
@@ -1114,15 +1114,15 @@
if (!hMenu)
return E_FAIL;
- hResult = GetItemObject( SVGIO_SELECTION, IID_PPV_ARG(IContextMenu, &pCM));
+ hResult = GetItemObject( SVGIO_SELECTION, IID_PPV_ARG(IContextMenu, &m_pCM));
if (FAILED(hResult))
goto cleanup;
- hResult = IUnknown_SetSite(pCM, (IShellView *)this);
+ hResult = IUnknown_SetSite(m_pCM, (IShellView *)this);
//if (FAILED( hResult))
// goto cleanup;
- hResult = pCM->QueryContextMenu(hMenu, 0, 0x20, 0x7fff, CMF_DEFAULTONLY);
+ hResult = m_pCM->QueryContextMenu(hMenu, 0, 0x20, 0x7fff, CMF_DEFAULTONLY);
if (FAILED(hResult))
goto cleanup;
@@ -1138,17 +1138,17 @@
cmi.lpVerb = (LPCSTR)MAKEINTRESOURCEA(uCommand);
cmi.hwnd = m_hWnd;
- hResult = pCM->InvokeCommand(&cmi);
+ hResult = m_pCM->InvokeCommand(&cmi);
cleanup:
if (hMenu)
DestroyMenu(hMenu);
- if (pCM)
- {
- IUnknown_SetSite(pCM, NULL);
- pCM.Release();
+ if (m_pCM)
+ {
+ IUnknown_SetSite(m_pCM, NULL);
+ m_pCM.Release();
}
return hResult;
@@ -1167,7 +1167,7 @@
HRESULT hResult;
// for some reason I haven't figured out, we sometimes recurse into this method
- if (pCM != NULL)
+ if (m_pCM != NULL)
return 0;
x = LOWORD(lParam);
@@ -1179,21 +1179,21 @@
if (!hMenu)
return E_FAIL;
- cidl = ListView_GetSelectedCount(hWndList);
-
- hResult = GetItemObject( cidl ? SVGIO_SELECTION : SVGIO_BACKGROUND,
IID_PPV_ARG(IContextMenu, &pCM));
+ m_cidl = ListView_GetSelectedCount(m_hWndList);
+
+ hResult = GetItemObject( m_cidl ? SVGIO_SELECTION : SVGIO_BACKGROUND,
IID_PPV_ARG(IContextMenu, &m_pCM));
if (FAILED( hResult))
goto cleanup;
- hResult = IUnknown_SetSite(pCM, (IShellView *)this);
+ hResult = IUnknown_SetSite(m_pCM, (IShellView *)this);
//if (FAILED( hResult))
// goto cleanup;
- hResult = pCM->QueryContextMenu(hMenu, 0, FCIDM_SHVIEWFIRST, FCIDM_SHVIEWLAST,
CMF_NORMAL);
+ hResult = m_pCM->QueryContextMenu(hMenu, 0, FCIDM_SHVIEWFIRST, FCIDM_SHVIEWLAST,
CMF_NORMAL);
if (FAILED( hResult))
goto cleanup;
- if (FolderSettings.fFlags & FWF_DESKTOP)
+ if (m_FolderSettings.fFlags & FWF_DESKTOP)
SetMenuDefaultItem(hMenu, FCIDM_SHVIEW_OPEN, MF_BYCOMMAND);
uCommand = TrackPopupMenu(hMenu,
@@ -1209,14 +1209,14 @@
cmi.cbSize = sizeof(cmi);
cmi.lpVerb = (LPCSTR)MAKEINTRESOURCEA(uCommand);
cmi.hwnd = m_hWnd;
- pCM->InvokeCommand(&cmi);
+ m_pCM->InvokeCommand(&cmi);
cleanup:
- if (pCM)
- {
- IUnknown_SetSite(pCM, NULL);
- pCM.Release();
+ if (m_pCM)
+ {
+ IUnknown_SetSite(m_pCM, NULL);
+ m_pCM.Release();
}
if (hMenu)
@@ -1235,15 +1235,15 @@
if (!hMenu)
return 0;
- hResult = GetItemObject( bUseSelection ? SVGIO_SELECTION : SVGIO_BACKGROUND,
IID_PPV_ARG(IContextMenu, &pCM));
+ hResult = GetItemObject( bUseSelection ? SVGIO_SELECTION : SVGIO_BACKGROUND,
IID_PPV_ARG(IContextMenu, &m_pCM));
if (FAILED( hResult))
goto cleanup;
- hResult = IUnknown_SetSite(pCM, (IShellView *)this);
+ hResult = IUnknown_SetSite(m_pCM, (IShellView *)this);
//if (FAILED( hResult))
// goto cleanup;
- hResult = pCM->QueryContextMenu(hMenu, 0, FCIDM_SHVIEWFIRST, FCIDM_SHVIEWLAST,
CMF_NORMAL);
+ hResult = m_pCM->QueryContextMenu(hMenu, 0, FCIDM_SHVIEWFIRST, FCIDM_SHVIEWLAST,
CMF_NORMAL);
if (FAILED( hResult))
goto cleanup;
@@ -1251,14 +1251,14 @@
cmi.cbSize = sizeof(cmi);
cmi.lpVerb = (LPCSTR)MAKEINTRESOURCEA(uCommand);
cmi.hwnd = m_hWnd;
- pCM->InvokeCommand(&cmi);
+ m_pCM->InvokeCommand(&cmi);
cleanup:
- if (pCM)
- {
- IUnknown_SetSite(pCM, NULL);
- pCM.Release();
+ if (m_pCM)
+ {
+ IUnknown_SetSite(m_pCM, NULL);
+ m_pCM.Release();
}
if (hMenu)
@@ -1285,9 +1285,9 @@
TRACE("%p width=%u height=%u\n", this, wWidth, wHeight);
/*resize the ListView to fit our window*/
- if (hWndList)
- {
- ::MoveWindow(hWndList, 0, 0, wWidth, wHeight, TRUE);
+ if (m_hWndList)
+ {
+ ::MoveWindow(m_hWndList, 0, 0, wWidth, wHeight, TRUE);
}
return 0;
@@ -1303,17 +1303,17 @@
{
TRACE("%p\n", this);
- if (uState != SVUIA_DEACTIVATE)
- {
- if (hMenu)
+ if (m_uState != SVUIA_DEACTIVATE)
+ {
+ if (m_hMenu)
{
- pShellBrowser->SetMenuSB(0, 0, 0);
- pShellBrowser->RemoveMenusSB(hMenu);
- DestroyMenu(hMenu);
- hMenu = 0;
+ m_pShellBrowser->SetMenuSB(0, 0, 0);
+ m_pShellBrowser->RemoveMenusSB(m_hMenu);
+ DestroyMenu(m_hMenu);
+ m_hMenu = 0;
}
- uState = SVUIA_DEACTIVATE;
+ m_uState = SVUIA_DEACTIVATE;
}
}
@@ -1326,7 +1326,7 @@
TRACE("%p uState=%x\n", this, uState);
/*don't do anything if the state isn't really changing */
- if (this->uState == uState)
+ if (m_uState == uState)
{
return;
}
@@ -1337,11 +1337,11 @@
if(uState != SVUIA_DEACTIVATE)
{
/*merge the menus */
- hMenu = CreateMenu();
-
- if(hMenu)
+ m_hMenu = CreateMenu();
+
+ if(m_hMenu)
{
- pShellBrowser->InsertMenusSB(hMenu, &omw);
+ m_pShellBrowser->InsertMenusSB(m_hMenu, &omw);
TRACE("-- after fnInsertMenusSB\n");
/*build the top level menu get the menu item's text*/
@@ -1358,7 +1358,7 @@
/*insert our menu into the menu bar*/
if (mii.hSubMenu)
{
- InsertMenuItemA(hMenu, FCIDM_MENU_HELP, FALSE, &mii);
+ InsertMenuItemA(m_hMenu, FCIDM_MENU_HELP, FALSE, &mii);
}
/*get the view menu so we can merge with it*/
@@ -1366,7 +1366,7 @@
mii.cbSize = sizeof(mii);
mii.fMask = MIIM_SUBMENU;
- if (GetMenuItemInfoA(hMenu, FCIDM_MENU_VIEW, FALSE, &mii))
+ if (GetMenuItemInfoA(m_hMenu, FCIDM_MENU_VIEW, FALSE, &mii))
{
MergeViewMenu(mii.hSubMenu);
}
@@ -1379,17 +1379,17 @@
mii.cbSize = sizeof(mii);
mii.fMask = MIIM_SUBMENU;
- if (GetMenuItemInfoA(hMenu, FCIDM_MENU_FILE, FALSE, &mii))
+ if (GetMenuItemInfoA(m_hMenu, FCIDM_MENU_FILE, FALSE, &mii))
{
MergeFileMenu(mii.hSubMenu);
}
}
TRACE("-- before fnSetMenuSB\n");
- pShellBrowser->SetMenuSB(hMenu, 0, m_hWnd);
+ m_pShellBrowser->SetMenuSB(m_hMenu, 0, m_hWnd);
}
}
- this->uState = uState;
+ m_uState = uState;
TRACE("--\n");
}
@@ -1414,11 +1414,11 @@
should always be done before merging menus (OnActivate merges the
menus) if one of our windows has the focus.*/
- pShellBrowser->OnViewWindowActive((IShellView *)this);
+ m_pShellBrowser->OnViewWindowActive((IShellView *)this);
DoActivate(SVUIA_ACTIVATE_FOCUS);
/* Set the focus to the listview */
- ::SetFocus(hWndList);
+ ::SetFocus(m_hWndList);
/* Notify the ICommDlgBrowser interface */
OnStateChange(CDBOSC_SETFOCUS);
@@ -1461,25 +1461,25 @@
switch (dwCmdID)
{
case FCIDM_SHVIEW_SMALLICON:
- FolderSettings.ViewMode = FVM_SMALLICON;
+ m_FolderSettings.ViewMode = FVM_SMALLICON;
SetStyle (LVS_SMALLICON, LVS_TYPEMASK);
CheckToolbar();
break;
case FCIDM_SHVIEW_BIGICON:
- FolderSettings.ViewMode = FVM_ICON;
+ m_FolderSettings.ViewMode = FVM_ICON;
SetStyle (LVS_ICON, LVS_TYPEMASK);
CheckToolbar();
break;
case FCIDM_SHVIEW_LISTVIEW:
- FolderSettings.ViewMode = FVM_LIST;
+ m_FolderSettings.ViewMode = FVM_LIST;
SetStyle (LVS_LIST, LVS_TYPEMASK);
CheckToolbar();
break;
case FCIDM_SHVIEW_REPORTVIEW:
- FolderSettings.ViewMode = FVM_DETAILS;
+ m_FolderSettings.ViewMode = FVM_DETAILS;
SetStyle (LVS_REPORT, LVS_TYPEMASK);
CheckToolbar();
break;
@@ -1489,10 +1489,10 @@
case 0x31:
case 0x32:
case 0x33:
- ListViewSortInfo.nHeaderID = (LPARAM) (dwCmdID - 0x30);
- ListViewSortInfo.bIsAscending = TRUE;
- ListViewSortInfo.nLastHeaderID = ListViewSortInfo.nHeaderID;
- SendMessageA(hWndList, LVM_SORTITEMS, (WPARAM) &ListViewSortInfo,
(LPARAM)ListViewCompareItems);
+ m_sortInfo.nHeaderID = (LPARAM) (dwCmdID - 0x30);
+ m_sortInfo.bIsAscending = TRUE;
+ m_sortInfo.nLastHeaderID = m_sortInfo.nHeaderID;
+ SendMessageA(m_hWndList, LVM_SORTITEMS, (WPARAM) &m_sortInfo,
(LPARAM)ListViewCompareItems);
break;
case FCIDM_SHVIEW_REFRESH:
@@ -1579,8 +1579,8 @@
case HDN_ENDTRACKW:
TRACE("-- HDN_ENDTRACKW %p\n", this);
- /*nColumn1 = ListView_GetColumnWidth(hWndList, 0);
- nColumn2 = ListView_GetColumnWidth(hWndList, 1);*/
+ /*nColumn1 = ListView_GetColumnWidth(m_hWndList, 0);
+ nColumn2 = ListView_GetColumnWidth(m_hWndList, 1);*/
break;
case LVN_DELETEITEM:
@@ -1602,18 +1602,18 @@
break;
case LVN_COLUMNCLICK:
- ListViewSortInfo.nHeaderID = lpnmlv->iSubItem;
- if (ListViewSortInfo.nLastHeaderID == ListViewSortInfo.nHeaderID)
+ m_sortInfo.nHeaderID = lpnmlv->iSubItem;
+ if (m_sortInfo.nLastHeaderID == m_sortInfo.nHeaderID)
{
- ListViewSortInfo.bIsAscending = !ListViewSortInfo.bIsAscending;
+ m_sortInfo.bIsAscending = !m_sortInfo.bIsAscending;
}
else
{
- ListViewSortInfo.bIsAscending = TRUE;
+ m_sortInfo.bIsAscending = TRUE;
}
- ListViewSortInfo.nLastHeaderID = ListViewSortInfo.nHeaderID;
-
- SendMessageW(lpnmlv->hdr.hwndFrom, LVM_SORTITEMS, (WPARAM)
&ListViewSortInfo, (LPARAM)ListViewCompareItems);
+ m_sortInfo.nLastHeaderID = m_sortInfo.nHeaderID;
+
+ SendMessageW(lpnmlv->hdr.hwndFrom, LVM_SORTITEMS, (WPARAM)
&m_sortInfo, (LPARAM)ListViewCompareItems);
break;
case LVN_GETDISPINFOA:
@@ -1623,10 +1623,10 @@
if (lpdi->item.mask & LVIF_TEXT) /* text requested */
{
- if (pSF2Parent)
+ if (m_pSF2Parent)
{
SHELLDETAILS sd;
- if (FAILED(pSF2Parent->GetDetailsOf(pidl, lpdi->item.iSubItem,
&sd)))
+ if (FAILED(m_pSF2Parent->GetDetailsOf(pidl,
lpdi->item.iSubItem, &sd)))
{
FIXME("failed to get details\n");
break;
@@ -1652,7 +1652,7 @@
}
if(lpdi->item.mask & LVIF_IMAGE) /* image requested */
{
- lpdi->item.iImage = SHMapPIDLToSystemImageListIndex(pSFParent, pidl,
0);
+ lpdi->item.iImage = SHMapPIDLToSystemImageListIndex(m_pSFParent, pidl,
0);
}
lpdi->item.mask |= LVIF_DI_SETITEM;
break;
@@ -1672,11 +1672,11 @@
DWORD dwAttributes = SFGAO_CANLINK;
DWORD dwEffect = DROPEFFECT_COPY | DROPEFFECT_MOVE;
- if (SUCCEEDED(pSFParent->GetUIObjectOf(m_hWnd, cidl,
(LPCITEMIDLIST*)apidl, IID_IDataObject, 0, (LPVOID *)&pda)))
+ if (SUCCEEDED(m_pSFParent->GetUIObjectOf(m_hWnd, m_cidl,
(LPCITEMIDLIST*)m_apidl, IID_IDataObject, 0, (LPVOID *)&pda)))
{
IDropSource * pds = (IDropSource *)this; /* own DropSource
interface */
- if (SUCCEEDED(pSFParent->GetAttributesOf(cidl,
(LPCITEMIDLIST*)apidl, &dwAttributes)))
+ if (SUCCEEDED(m_pSFParent->GetAttributesOf(m_cidl,
(LPCITEMIDLIST*)m_apidl, &dwAttributes)))
{
if (dwAttributes & SFGAO_CANLINK)
{
@@ -1701,7 +1701,7 @@
TRACE("-- LVN_BEGINLABELEDITW %p\n", this);
- pSFParent->GetAttributesOf(1, (LPCITEMIDLIST*)&pidl, &dwAttr);
+ m_pSFParent->GetAttributesOf(1, (LPCITEMIDLIST*)&pidl, &dwAttr);
if (SFGAO_CANRENAME & dwAttr)
{
return FALSE;
@@ -1720,18 +1720,18 @@
lvItem.iItem = lpdi->item.iItem;
lvItem.iSubItem = 0;
lvItem.mask = LVIF_PARAM;
- SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
pidl = (LPITEMIDLIST)lpdi->item.lParam;
- hr = pSFParent->SetNameOf(0, pidl, lpdi->item.pszText,
SHGDN_INFOLDER, &pidl);
+ hr = m_pSFParent->SetNameOf(0, pidl, lpdi->item.pszText,
SHGDN_INFOLDER, &pidl);
if (SUCCEEDED(hr) && pidl)
{
lvItem.mask = LVIF_PARAM|LVIF_IMAGE;
lvItem.lParam = (LPARAM)pidl;
- lvItem.iImage = SHMapPIDLToSystemImageListIndex(pSFParent, pidl, 0);
- SendMessageW(hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
- SendMessageW(hWndList, LVM_UPDATE, lpdi->item.iItem, 0);
+ lvItem.iImage = SHMapPIDLToSystemImageListIndex(m_pSFParent, pidl,
0);
+ SendMessageW(m_hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_UPDATE, lpdi->item.iItem, 0);
return TRUE;
}
@@ -1748,7 +1748,7 @@
if (plvKeyDown->wVKey == VK_BACK)
{
LPSHELLBROWSER lpSb;
- if ((lpSb = (LPSHELLBROWSER)SendMessageW(hWndParent,
CWM_GETISHELLBROWSER, 0, 0)))
+ if ((lpSb = (LPSHELLBROWSER)SendMessageW(m_hWndParent,
CWM_GETISHELLBROWSER, 0, 0)))
{
lpSb->BrowseObject(NULL, SBSP_PARENT);
}
@@ -1807,7 +1807,7 @@
*/
LRESULT CDefView::OnCustomItem(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL
&bHandled)
{
- if (!pCM.p)
+ if (!m_pCM.p)
{
/* no menu */
ERR("no menu!!!\n");
@@ -1815,7 +1815,7 @@
}
CComPtr<IContextMenu2> pCM2;
- HRESULT hres = pCM.p->QueryInterface(IID_PPV_ARG(IContextMenu2, &pCM2));
+ HRESULT hres = m_pCM.p->QueryInterface(IID_PPV_ARG(IContextMenu2, &pCM2));
if(FAILED(hres))
return FALSE;
@@ -1872,7 +1872,7 @@
{
if (lpmsg->message >= WM_KEYFIRST && lpmsg->message <=
WM_KEYLAST)
{
- if (::TranslateAcceleratorW(m_hWnd, hAccel, lpmsg) != 0)
+ if (::TranslateAcceleratorW(m_hWnd, m_hAccel, lpmsg) != 0)
return S_OK;
/* FIXME: should call TranslateAcceleratorSB */
@@ -1901,7 +1901,7 @@
TRACE("(%p)->(state=%x) stub\n", this, uState);
/*don't do anything if the state isn't really changing*/
- if (this->uState == uState)
+ if (m_uState == uState)
{
return S_OK;
}
@@ -1915,14 +1915,14 @@
/*
GetFolderPath is not a method of IShellFolder
- IShellFolder_GetFolderPath( pSFParent, szName, sizeof(szName) );
+ IShellFolder_GetFolderPath( m_pSFParent, szName, sizeof(szName) );
*/
/* set the number of parts */
- pShellBrowser->SendControlMsg(FCW_STATUS, SB_SETPARTS, 1, (LPARAM)nPartArray,
&lResult);
+ m_pShellBrowser->SendControlMsg(FCW_STATUS, SB_SETPARTS, 1,
(LPARAM)nPartArray, &lResult);
/* set the text for the parts */
/*
- pShellBrowser->SendControlMsg(FCW_STATUS, SB_SETTEXTA, 0, (LPARAM)szName,
&lResult);
+ m_pShellBrowser->SendControlMsg(FCW_STATUS, SB_SETTEXTA, 0,
(LPARAM)szName, &lResult);
*/
}
@@ -1933,7 +1933,7 @@
{
TRACE("(%p)\n", this);
- SendMessageW(hWndList, LVM_DELETEALLITEMS, 0, 0);
+ SendMessageW(m_hWndList, LVM_DELETEALLITEMS, 0, 0);
FillList();
return S_OK;
@@ -1955,20 +1955,20 @@
return E_UNEXPECTED;
/*set up the member variables*/
- pShellBrowser = psb;
- FolderSettings = *lpfs;
+ m_pShellBrowser = psb;
+ m_FolderSettings = *lpfs;
/*get our parent window*/
- pShellBrowser->GetWindow(&hWndParent);
+ m_pShellBrowser->GetWindow(&m_hWndParent);
/* try to get the ICommDlgBrowserInterface, adds a reference !!! */
- pCommDlgBrowser = NULL;
- if (SUCCEEDED(pShellBrowser->QueryInterface(IID_PPV_ARG(ICommDlgBrowser,
&pCommDlgBrowser))))
+ m_pCommDlgBrowser = NULL;
+ if (SUCCEEDED(m_pShellBrowser->QueryInterface(IID_PPV_ARG(ICommDlgBrowser,
&m_pCommDlgBrowser))))
{
TRACE("-- CommDlgBrowser\n");
}
- Create(hWndParent, prcView, NULL, WS_CHILD | WS_TABSTOP, 0, 0U);
+ Create(m_hWndParent, prcView, NULL, WS_CHILD | WS_TABSTOP, 0, 0U);
if (m_hWnd == NULL)
return E_FAIL;
@@ -1992,14 +1992,14 @@
/*Make absolutely sure all our UI is cleaned up.*/
UIActivate(SVUIA_DEACTIVATE);
- if (hMenu)
- {
- DestroyMenu(hMenu);
+ if (m_hMenu)
+ {
+ DestroyMenu(m_hMenu);
}
DestroyWindow();
- pShellBrowser.Release();
- pCommDlgBrowser.Release();
+ m_pShellBrowser.Release();
+ m_pCommDlgBrowser.Release();
return S_OK;
}
@@ -2007,12 +2007,12 @@
HRESULT WINAPI CDefView::GetCurrentInfo(LPFOLDERSETTINGS lpfs)
{
TRACE("(%p)->(%p) vmode=%x flags=%x\n", this, lpfs,
- FolderSettings.ViewMode, FolderSettings.fFlags);
+ m_FolderSettings.ViewMode, m_FolderSettings.fFlags);
if (!lpfs)
return E_INVALIDARG;
- *lpfs = FolderSettings;
+ *lpfs = m_FolderSettings;
return S_OK;
}
@@ -2043,14 +2043,14 @@
LVITEMW lvItem;
if(uFlags & SVSI_ENSUREVISIBLE)
- SendMessageW(hWndList, LVM_ENSUREVISIBLE, i, 0);
+ SendMessageW(m_hWndList, LVM_ENSUREVISIBLE, i, 0);
lvItem.mask = LVIF_STATE;
lvItem.stateMask = LVIS_SELECTED | LVIS_FOCUSED;
lvItem.iItem = 0;
lvItem.iSubItem = 0;
- while (SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem))
+ while (SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem))
{
if (lvItem.iItem == i)
{
@@ -2068,13 +2068,13 @@
lvItem.state &= ~LVIS_SELECTED;
}
- SendMessageW(hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
+ SendMessageW(m_hWndList, LVM_SETITEMW, 0, (LPARAM) &lvItem);
lvItem.iItem++;
}
if(uFlags & SVSI_EDIT)
- SendMessageW(hWndList, LVM_EDITLABELW, i, 0);
+ SendMessageW(m_hWndList, LVM_EDITLABELW, i, 0);
}
return S_OK;
@@ -2093,8 +2093,8 @@
case SVGIO_BACKGROUND:
if (IsEqualIID(riid, IID_IContextMenu))
{
- //*ppvOut = ISvBgCm_Constructor(pSFParent, FALSE);
- CDefFolderMenu_Create2(NULL, NULL, 0, NULL, pSFParent, NULL, 0, NULL,
(IContextMenu**)ppvOut);
+ //*ppvOut = ISvBgCm_Constructor(m_pSFParent, FALSE);
+ CDefFolderMenu_Create2(NULL, NULL, 0, NULL, m_pSFParent, NULL, 0, NULL,
(IContextMenu**)ppvOut);
if (!ppvOut)
hr = E_OUTOFMEMORY;
else
@@ -2104,7 +2104,7 @@
case SVGIO_SELECTION:
GetSelections();
- hr = pSFParent->GetUIObjectOf(m_hWnd, cidl, (LPCITEMIDLIST*)apidl, riid,
0, ppvOut);
+ hr = m_pSFParent->GetUIObjectOf(m_hWnd, m_cidl, (LPCITEMIDLIST*)m_apidl,
riid, 0, ppvOut);
break;
}
@@ -2120,7 +2120,7 @@
if (!pViewMode)
return E_INVALIDARG;
- *pViewMode = this->FolderSettings.ViewMode;
+ *pViewMode = m_FolderSettings.ViewMode;
return S_OK;
}
@@ -2163,17 +2163,17 @@
/* This will not necessarily be the actual mode set above.
This mimics the behavior of Windows XP. */
- this->FolderSettings.ViewMode = ViewMode;
+ m_FolderSettings.ViewMode = ViewMode;
return S_OK;
}
HRESULT STDMETHODCALLTYPE CDefView::GetFolder(REFIID riid, void **ppv)
{
- if (pSFParent == NULL)
+ if (m_pSFParent == NULL)
return E_FAIL;
- return pSFParent->QueryInterface(riid, ppv);
+ return m_pSFParent->QueryInterface(riid, ppv);
}
HRESULT STDMETHODCALLTYPE CDefView::Item(int iItemIndex, LPITEMIDLIST *ppidl)
@@ -2185,7 +2185,7 @@
item.mask = LVIF_PARAM;
item.iItem = iItemIndex;
- if (SendMessageW(this->hWndList, LVM_GETITEMW, 0, (LPARAM)&item))
+ if (SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM)&item))
{
*ppidl = ILClone((PITEMID_CHILD)item.lParam);
return S_OK;
@@ -2203,7 +2203,7 @@
if (uFlags != SVGIO_ALLVIEW)
FIXME("some flags unsupported, %x\n", uFlags & ~SVGIO_ALLVIEW);
- *pcItems = SendMessageW(this->hWndList, LVM_GETITEMCOUNT, 0, 0);
+ *pcItems = SendMessageW(m_hWndList, LVM_GETITEMCOUNT, 0, 0);
return S_OK;
}
@@ -2217,7 +2217,7 @@
{
TRACE("(%p)->(%p)\n", this, piItem);
- *piItem = SendMessageW(this->hWndList, LVM_GETSELECTIONMARK, 0, 0);
+ *piItem = SendMessageW(m_hWndList, LVM_GETSELECTIONMARK, 0, 0);
return S_OK;
}
@@ -2226,7 +2226,7 @@
{
TRACE("(%p)->(%p)\n", this, piItem);
- *piItem = SendMessageW(this->hWndList, LVM_GETNEXTITEM, -1, LVNI_FOCUSED);
+ *piItem = SendMessageW(m_hWndList, LVM_GETNEXTITEM, -1, LVNI_FOCUSED);
return S_OK;
}
@@ -2240,11 +2240,11 @@
{
TRACE("(%p)->(%p)\n", this, ppt);
- if (NULL == this->hWndList) return S_FALSE;
+ if (NULL == m_hWndList) return S_FALSE;
if (ppt)
{
- const DWORD ret = SendMessageW(this->hWndList, LVM_GETITEMSPACING, 0, 0);
+ const DWORD ret = SendMessageW(m_hWndList, LVM_GETITEMSPACING, 0, 0);
ppt->x = LOWORD(ret);
ppt->y = HIWORD(ret);
@@ -2273,11 +2273,11 @@
lvItem.stateMask = LVIS_SELECTED;
if (dwFlags & SVSI_ENSUREVISIBLE)
- SendMessageW(this->hWndList, LVM_ENSUREVISIBLE, iItem, 0);
+ SendMessageW(m_hWndList, LVM_ENSUREVISIBLE, iItem, 0);
/* all items */
if (dwFlags & SVSI_DESELECTOTHERS)
- SendMessageW(this->hWndList, LVM_SETITEMSTATE, -1, (LPARAM)&lvItem);
+ SendMessageW(m_hWndList, LVM_SETITEMSTATE, -1, (LPARAM)&lvItem);
/* this item */
if (dwFlags & SVSI_SELECT)
@@ -2286,10 +2286,10 @@
if (dwFlags & SVSI_FOCUSED)
lvItem.stateMask |= LVIS_FOCUSED;
- SendMessageW(this->hWndList, LVM_SETITEMSTATE, iItem, (LPARAM)&lvItem);
+ SendMessageW(m_hWndList, LVM_SETITEMSTATE, iItem, (LPARAM)&lvItem);
if (dwFlags & SVSI_EDIT)
- SendMessageW(this->hWndList, LVM_EDITLABELW, iItem, 0);
+ SendMessageW(m_hWndList, LVM_EDITLABELW, iItem, 0);
return S_OK;
}
@@ -2371,56 +2371,56 @@
htinfo.pt.x = pt.x;
htinfo.pt.y = pt.y;
htinfo.flags = LVHT_ONITEM;
- ::ScreenToClient(hWndList, &htinfo.pt);
- lResult = SendMessageW(hWndList, LVM_HITTEST, 0, (LPARAM)&htinfo);
+ ::ScreenToClient(m_hWndList, &htinfo.pt);
+ lResult = SendMessageW(m_hWndList, LVM_HITTEST, 0, (LPARAM)&htinfo);
/* Send WM_*SCROLL messages every 250 ms during drag-scrolling */
- ::GetClientRect(hWndList, &clientRect);
- if (htinfo.pt.x == ptLastMousePos.x && htinfo.pt.y == ptLastMousePos.y
&&
+ ::GetClientRect(m_hWndList, &clientRect);
+ if (htinfo.pt.x == m_ptLastMousePos.x && htinfo.pt.y == m_ptLastMousePos.y
&&
(htinfo.pt.x < SCROLLAREAWIDTH || htinfo.pt.x > clientRect.right -
SCROLLAREAWIDTH ||
htinfo.pt.y < SCROLLAREAWIDTH || htinfo.pt.y > clientRect.bottom -
SCROLLAREAWIDTH ))
{
- cScrollDelay = (cScrollDelay + 1) % 5; /* DragOver is called every 50 ms */
- if (cScrollDelay == 0)
+ m_cScrollDelay = (m_cScrollDelay + 1) % 5; /* DragOver is called every 50 ms */
+ if (m_cScrollDelay == 0)
{
/* Mouse did hover another 250 ms over the scroll-area */
if (htinfo.pt.x < SCROLLAREAWIDTH)
- SendMessageW(hWndList, WM_HSCROLL, SB_LINEUP, 0);
+ SendMessageW(m_hWndList, WM_HSCROLL, SB_LINEUP, 0);
if (htinfo.pt.x > clientRect.right - SCROLLAREAWIDTH)
- SendMessageW(hWndList, WM_HSCROLL, SB_LINEDOWN, 0);
+ SendMessageW(m_hWndList, WM_HSCROLL, SB_LINEDOWN, 0);
if (htinfo.pt.y < SCROLLAREAWIDTH)
- SendMessageW(hWndList, WM_VSCROLL, SB_LINEUP, 0);
+ SendMessageW(m_hWndList, WM_VSCROLL, SB_LINEUP, 0);
if (htinfo.pt.y > clientRect.bottom - SCROLLAREAWIDTH)
- SendMessageW(hWndList, WM_VSCROLL, SB_LINEDOWN, 0);
+ SendMessageW(m_hWndList, WM_VSCROLL, SB_LINEDOWN, 0);
}
}
else
{
- cScrollDelay = 0; /* Reset, if the cursor is not over the listview's
scroll-area */
- }
-
- ptLastMousePos = htinfo.pt;
+ m_cScrollDelay = 0; /* Reset, if the cursor is not over the listview's
scroll-area */
+ }
+
+ m_ptLastMousePos = htinfo.pt;
/* If we are still over the previous sub-item, notify it via DragOver and return. */
- if (pCurDropTarget && lResult == iDragOverItem)
- return pCurDropTarget->DragOver(grfKeyState, pt, pdwEffect);
+ if (m_pCurDropTarget && lResult == m_iDragOverItem)
+ return m_pCurDropTarget->DragOver(grfKeyState, pt, pdwEffect);
/* We've left the previous sub-item, notify it via DragLeave and Release it. */
- if (pCurDropTarget)
- {
- pCurDropTarget->DragLeave();
- pCurDropTarget.Release();
- }
-
- iDragOverItem = lResult;
+ if (m_pCurDropTarget)
+ {
+ m_pCurDropTarget->DragLeave();
+ m_pCurDropTarget.Release();
+ }
+
+ m_iDragOverItem = lResult;
if (lResult == -1)
{
/* We are not above one of the listview's subitems. Bind to the parent
folder's
* DropTarget interface. */
- hr = pSFParent->QueryInterface(IID_PPV_ARG(IDropTarget,&pCurDropTarget));
+ hr =
m_pSFParent->QueryInterface(IID_PPV_ARG(IDropTarget,&m_pCurDropTarget));
}
else
{
@@ -2429,25 +2429,25 @@
lvItem.mask = LVIF_PARAM;
lvItem.iItem = lResult;
lvItem.iSubItem = 0;
- SendMessageW(hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
-
- /* ... and bind pCurDropTarget to the IDropTarget interface of an UIObject of
this object */
- hr = pSFParent->GetUIObjectOf(hWndList, 1,
- (LPCITEMIDLIST*)&lvItem.lParam,
IID_IDropTarget, NULL, (LPVOID*)&pCurDropTarget);
- }
-
- /* If anything failed, pCurDropTarget should be NULL now, which ought to be a save
state. */
+ SendMessageW(m_hWndList, LVM_GETITEMW, 0, (LPARAM) &lvItem);
+
+ /* ... and bind m_pCurDropTarget to the IDropTarget interface of an UIObject of
this object */
+ hr = m_pSFParent->GetUIObjectOf(m_hWndList, 1,
+ (LPCITEMIDLIST*)&lvItem.lParam,
IID_IDropTarget, NULL, (LPVOID*)&m_pCurDropTarget);
+ }
+
+ /* If anything failed, m_pCurDropTarget should be NULL now, which ought to be a save
state. */
if (FAILED(hr))
return hr;
/* Notify the item just entered via DragEnter. */
- return pCurDropTarget->DragEnter(pCurDataObject, grfKeyState, pt, pdwEffect);
+ return m_pCurDropTarget->DragEnter(m_pCurDataObject, grfKeyState, pt, pdwEffect);
}
HRESULT WINAPI CDefView::DragEnter(IDataObject *pDataObject, DWORD grfKeyState, POINTL
pt, DWORD *pdwEffect)
{
/* Get a hold on the data object for later calls to DragEnter on the sub-folders */
- pCurDataObject = pDataObject;
+ m_pCurDataObject = pDataObject;
pDataObject->AddRef();
return drag_notify_subitem(grfKeyState, pt, pdwEffect);
@@ -2460,32 +2460,32 @@
HRESULT WINAPI CDefView::DragLeave()
{
- if (pCurDropTarget)
- {
- pCurDropTarget->DragLeave();
- pCurDropTarget.Release();
- }
-
- if (pCurDataObject != NULL)
- {
- pCurDataObject.Release();
- }
-
- iDragOverItem = 0;
+ if (m_pCurDropTarget)
+ {
+ m_pCurDropTarget->DragLeave();
+ m_pCurDropTarget.Release();
+ }
+
+ if (m_pCurDataObject != NULL)
+ {
+ m_pCurDataObject.Release();
+ }
+
+ m_iDragOverItem = 0;
return S_OK;
}
HRESULT WINAPI CDefView::Drop(IDataObject* pDataObject, DWORD grfKeyState, POINTL pt,
DWORD *pdwEffect)
{
- if (pCurDropTarget)
- {
- pCurDropTarget->Drop(pDataObject, grfKeyState, pt, pdwEffect);
- pCurDropTarget.Release();
- }
-
- pCurDataObject.Release();
- iDragOverItem = 0;
+ if (m_pCurDropTarget)
+ {
+ m_pCurDropTarget->Drop(pDataObject, grfKeyState, pt, pdwEffect);
+ m_pCurDropTarget.Release();
+ }
+
+ m_pCurDataObject.Release();
+ m_iDragOverItem = 0;
return S_OK;
}
@@ -2550,9 +2550,9 @@
FIXME("partial stub: %p %08x %08x %p\n", this, aspects, advf, pAdvSink);
/* FIXME: we set the AdviseSink, but never use it to send any advice */
- this->pAdvSink = pAdvSink;
- dwAspects = aspects;
- dwAdvf = advf;
+ m_pAdvSink = pAdvSink;
+ m_dwAspects = aspects;
+ m_dwAdvf = advf;
return S_OK;
}
@@ -2563,15 +2563,15 @@
if (ppAdvSink)
{
- *ppAdvSink = pAdvSink;
- pAdvSink.p->AddRef();
+ *ppAdvSink = m_pAdvSink;
+ m_pAdvSink.p->AddRef();
}
if (pAspects)
- *pAspects = dwAspects;
+ *pAspects = m_dwAspects;
if (pAdvf)
- *pAdvf = dwAdvf;
+ *pAdvf = m_dwAdvf;
return S_OK;
}
@@ -2579,7 +2579,7 @@
HRESULT STDMETHODCALLTYPE CDefView::QueryService(REFGUID guidService, REFIID riid, void
**ppvObject)
{
if (IsEqualIID(guidService, SID_IShellBrowser))
- return pShellBrowser->QueryInterface(riid, ppvObject);
+ return m_pShellBrowser->QueryInterface(riid, ppvObject);
else if(IsEqualIID(guidService, SID_IFolderView))
return QueryInterface(riid, ppvObject);