Ros-dev March 2011

ros-dev@reactos.org

34 participants
47 discussions

by perencia

Hi!, my name is Pedro, and i'm a 5th grade student of computer science in Universitat Autonoma de Barcelona ( Spain ). I'd like to apply in this gsoc, i've always wanted to be part of an open-source project, i think it can be a really profitable and exciting experience to share ideas and discuss them with other motivated coders. Now as i'm finishing my last year i've found myself attracted by low-level questions. I've been aware of ReactOS for some time now - couple of years ago I downloaded the code and submitted a patch for the build system - actually, very silly patch :). About the projects proposed, some of them seem a bit heavy for me. At first the ACPI Hal cought my attention, since i'm doing my final year project en the energy-performance tradeoff in multi-cores, but it seems very daunting, i have no experience in kernel development nor drivers. Although is in Java where i'm really experienced, I'm confident with c and c++, and i'm very interested in COM, I've done some of it in DirectX and i really enjoyed that, so i thought the MMC Console seems more affordable. Anyway, i'd like to get some advice, if that's possible, about where i could direct myself, which project my profile may fit better or which has greater priority, and what can i do to start getting in contact to the project and make a better apply proposal. I hope this would be a beginning of a great and profitable experience for me and the community :)

13 years, 6 months

Re: [ros-dev] [ros-diffs] [cfinck] 51118: Create a branch for Neeraj Yadav's work on the Audio Mixer

by Ged Murphy

This seems a little premature. I assume Neeraj is aware that there's no guarantee that the audio mixer will be one of the selected GSoC projects? I'm by no means suggesting that we would do this, but good business sense says that if someone is willing to work on a component outside of GSoC, isn't it counter-productive for the project to use one of their slots up? Anyway, we'll see what we have on the April 8 deadline. I wish Neeraj luck in both his application and his forthcoming work in reactos. Ged. -----Original Message----- From: ros-diffs-bounces(a)reactos.org [mailto:ros-diffs-bounces@reactos.org] On Behalf Of cfinck(a)svn.reactos.org Sent: 22 March 2011 12:46 To: ros-diffs(a)reactos.org Subject: [ros-diffs] [cfinck] 51118: Create a branch for Neeraj Yadav's work on the Audio Mixer Author: cfinck Date: Tue Mar 22 12:45:40 2011 New Revision: 51118 URL: http://svn.reactos.org/svn/reactos?rev=51118&view=rev Log: Create a branch for Neeraj Yadav's work on the Audio Mixer Added: branches/nyadav-audio-branch/ - copied from r51117, trunk/

13 years, 6 months

Introduction

by Neeraj Yadav

Hello, My name is Neeraj Yadav. I am an undergraduate student of Indian Institute of Technology, Delhi studying Textile Engineering.Besides my regular studies I have interest in Multimedia, Networking, Network-Programming and Security.I have a decent knowledge of C and C++ programming.I am familiar with git and svn.I am familiar with Linux, windows and Symbian APIs.I have experience of UI design with Qt. ReactOS is coming as my new interest and luckily this year ReactOS has been selected as a mentoring organisation.I find myself appropriate for SSH server implementation and Audio Mixer Project.

13 years, 6 months

Re: [ros-dev] [ros-diffs] [rharabien] 51108: Fix ProbeForRead. It wasn't ever checking if memory can be accessed. Thanks to big-endian it wasn't breaking MmUserProbeAddress as well. Code is now nearly the same as in ProbeFo...

by Timo Kreuzer

Windows doesn't do any access checks in ProbeForRead, it only checks the range and alignment. The MmUserProbeAddress access is used to raise an exception with the appropriate parameters. So the old version was correct (except for the misleading comment maybe) Am 21.03.2011 15:43, schrieb rharabien(a)svn.reactos.org: > Author: rharabien > Date: Mon Mar 21 14:43:56 2011 > New Revision: 51108 > > URL: http://svn.reactos.org/svn/reactos?rev=51108&view=rev > Log: > Fix ProbeForRead. It wasn't ever checking if memory can be accessed. Thanks to big-endian it wasn't breaking MmUserProbeAddress as well. Code is now nearly the same as in ProbeForWrite. It shouldn't break anything. If it does, it's not bug in this code. :) > > Modified: > trunk/reactos/ntoskrnl/ex/exintrin.c > > Modified: trunk/reactos/ntoskrnl/ex/exintrin.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/exintrin.c?rev… > ============================================================================== > --- trunk/reactos/ntoskrnl/ex/exintrin.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/ex/exintrin.c [iso-8859-1] Mon Mar 21 14:43:56 2011 > @@ -103,6 +103,8 @@ > IN SIZE_T Length, > IN ULONG Alignment) > { > + ULONG_PTR Last, Current = (ULONG_PTR)Address; > + CHAR Temp; > PAGED_CODE(); > > /* Only probe if we have a valid length */ > @@ -115,18 +117,31 @@ > (Alignment == 8) || > (Alignment == 16)); > > - /* Check for correct alignment */ > - if (((ULONG_PTR)Address& (Alignment - 1)) != 0) > + /* Check the alignment */ > + if ((Current& (Alignment - 1)) != 0) > { > /* Incorrect alignment */ > ExRaiseDatatypeMisalignment(); > } > - else if (((ULONG_PTR)Address + Length)< (ULONG_PTR)Address || > - ((ULONG_PTR)Address + Length)> (ULONG_PTR)MmUserProbeAddress) > + > + /* Get the end address */ > + Last = Current + Length - 1; > + if ((Last< Current) || (Last>= (ULONG_PTR)MmUserProbeAddress)) > + { > + /* Raise an access violation */ > + ExRaiseAccessViolation(); > + } > + > + /* Round down to the last page */ > + Last = PAGE_ROUND_DOWN(Last) + PAGE_SIZE; > + do > { > /* Attempt a read */ > - *(volatile CHAR* const)MmUserProbeAddress = 0; > - } > + Temp = *(volatile CHAR*)Current; > + > + /* Go to the next address */ > + Current = PAGE_ROUND_DOWN(Current) + PAGE_SIZE; > + } while (Current != Last); > } > } > > > >

13 years, 6 months

Server down

by Pierre Schweitzer

Hi, both doxygen.reactos.org & iso.reactos.org are down (which results in daily builds isos not being uploaded). Anyone able to fix them? Regards, P. Schweitzer

13 years, 7 months

HAL multi-targeted build

by Pierre Schweitzer

Hi, currently working on HAL and HAL targets (ACPI, MP, UP and so on), I discovered a quite bad issue in our build process. Just look at the function HalInitSystem@8 (halinit.c), depending on the build target we don't add the same code (look at CONFIG_SMP, which is only defined for HAL MP). But, currently, as we are building using HAL library for reducing code redudancy, this halinit.c file is only built ONCE. So, build targets aren't taken into account (MP, ACPI). So, two choices, or we break that model, and forget about lib, and just keep base .rbuild (hal.rbuild, halacpi.rbuild, and so forth) with all files defined in those. Or with find something smarter better such as rebuilding the lib with the new flags for each build? Or finding a way to include .rbuild files in other rbuild files? I'm not really sure about the way to proceed, that's why I "release" that question to the whole ML. Furthermore, could CMAKE be any use here? To facilitate the process? Regards, P. Schweitzer

13 years, 7 months

'Security Controls' GSoC Idea

by Guillaume Touron

Hello RoS developers, I introduce myself. I'm Guillaume Touron, I'm a French engineering student at National School of Computer Science and Applied Mathematics with specialization in information systems and security. Apart from my studies I am interested in OS development, Windows internals and application security (vulnerabilities, reverse engineering, rootkits...). I read a lot about how Windows internals work (kernel data structures...) and Windows security. I have also some experience in NT driver development and kernel debugger (WinDBG). For many months now, I study RoS source code, and especially ntoskrnl source. This includes object manager, i/o manager, security layer, traps/exceptions management... I checkouted source code and installed RoS build environment and I'm ready to begin development. I also took a look at Se* kernel exported APIs, and tried to understand how security checks were performed. That's why I'm interested in RoS GSoC Idea 'Security Controls'. Indeed, implementing this feature would be a good start to add multiuser experience in RoS which is currently missing, and to manage efficiently users and groups. I think I know what is needed to complete this project successfully and I'm ready to write a detailed presentation/proposal to explain what I understand and what would be my goals. I also did GSoC 2010 last year for Honeynet Project on network security area, but I'm more interested in kernel development. Finally, few questions : What should I do to propose in a formal way my proposal to RoS community ? Is it an obligation to have already send some patches or can we begin development as a new contributor ? Who can I contact to speak about technical details ? Thank you very much.

13 years, 7 months

Re: [ros-dev] [ros-diffs] [rharabien] 51103: Don't destroy ImageList used by ListView if control was created without LVS_SHAREIMAGELISTS flag Fixes crashes when closing RegEdit, RApps and hardware tab in mouse settings.

by Gabriel ilardi

Welcome to the jungle! Nice to see you committing. Good luck and have fun! > Date: Mon, 21 Mar 2011 14:07:30 +0000 > To: ros-diffs(a)reactos.org > From: rharabien(a)svn.reactos.org > Subject: [ros-diffs] [rharabien] 51103: Don't destroy ImageList used by ListView if control was created without LVS_SHAREIMAGELISTS flag Fixes crashes when closing RegEdit, RApps and hardware tab in mouse settings. > > Author: rharabien > Date: Mon Mar 21 14:07:29 2011 > New Revision: 51103 > > URL: http://svn.reactos.org/svn/reactos?rev=51103&view=rev > Log: > Don't destroy ImageList used by ListView if control was created without LVS_SHAREIMAGELISTS flag > Fixes crashes when closing RegEdit, RApps and hardware tab in mouse settings. > > Modified: > trunk/reactos/base/applications/rapps/winmain.c > trunk/reactos/base/applications/regedit/childwnd.c > trunk/reactos/dll/cpl/input/settings.c > > Modified: trunk/reactos/base/applications/rapps/winmain.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/wi… > ============================================================================== > --- trunk/reactos/base/applications/rapps/winmain.c [iso-8859-1] (original) > +++ trunk/reactos/base/applications/rapps/winmain.c [iso-8859-1] Mon Mar 21 14:07:29 2011 > @@ -10,7 +10,6 @@ > > HWND hMainWnd; > HINSTANCE hInst; > -HIMAGELIST hImageListView = NULL; > HIMAGELIST hImageTreeView = NULL; > INT SelectedEnumType = ENUM_ALL_COMPONENTS; > SETTINGS_INFO SettingsInfo; > @@ -168,8 +167,8 @@ > { > WCHAR szBuffer1[MAX_STR_LEN], szBuffer2[MAX_STR_LEN]; > HICON hIcon; > - > - if (hImageListView) ImageList_Destroy(hImageListView); > + HIMAGELIST hImageListView; > + > (VOID) ListView_DeleteAllItems(hListView); > > /* Create image list */ > @@ -208,7 +207,11 @@ > } > > /* Set image list for ListView */ > - (VOID) ListView_SetImageList(hListView, hImageListView, LVSIL_SMALL); > + hImageListView = ListView_SetImageList(hListView, hImageListView, LVSIL_SMALL); > + > + /* Destroy old image list */ > + if (hImageListView) > + ImageList_Destroy(hImageListView); > > SelectedEnumType = EnumType; > > @@ -725,7 +728,6 @@ > FreeAvailableAppList(); > if (IS_INSTALLED_ENUM(SelectedEnumType)) > FreeInstalledAppList(); > - if (hImageListView) ImageList_Destroy(hImageListView); > if (hImageTreeView) ImageList_Destroy(hImageTreeView); > > PostQuitMessage(0); > > Modified: trunk/reactos/base/applications/regedit/childwnd.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/regedit/… > ============================================================================== > --- trunk/reactos/base/applications/regedit/childwnd.c [iso-8859-1] (original) > +++ trunk/reactos/base/applications/regedit/childwnd.c [iso-8859-1] Mon Mar 21 14:07:29 2011 > @@ -65,7 +65,7 @@ > ReleaseDC(hWnd, hdc); > } > > -static void ResizeWnd(ChildWnd* pChildWnd, int cx, int cy) > +static void ResizeWnd(int cx, int cy) > { > HDWP hdwp = BeginDeferWindowPos(3); > RECT rt, rs, rb; > @@ -77,12 +77,12 @@ > GetWindowRect(hStatusBar, &rs); > cy = rs.bottom - rs.top; > } > - GetWindowRect(pChildWnd->hAddressBtnWnd, &rb); > - cx = pChildWnd->nSplitPos + SPLIT_WIDTH/2; > - DeferWindowPos(hdwp, pChildWnd->hAddressBarWnd, 0, rt.left, rt.top, rt.right-rt.left - tHeight-2, tHeight, SWP_NOZORDER|SWP_NOACTIVATE); > - DeferWindowPos(hdwp, pChildWnd->hAddressBtnWnd, 0, rt.right - tHeight, rt.top, tHeight, tHeight, SWP_NOZORDER|SWP_NOACTIVATE); > - DeferWindowPos(hdwp, pChildWnd->hTreeWnd, 0, rt.left, rt.top + tHeight+2, pChildWnd->nSplitPos-SPLIT_WIDTH/2-rt.left, rt.bottom-rt.top-cy, SWP_NOZORDER|SWP_NOACTIVATE); > - DeferWindowPos(hdwp, pChildWnd->hListWnd, 0, rt.left+cx, rt.top + tHeight+2, rt.right-cx, rt.bottom-rt.top-cy, SWP_NOZORDER|SWP_NOACTIVATE); > + GetWindowRect(g_pChildWnd->hAddressBtnWnd, &rb); > + cx = g_pChildWnd->nSplitPos + SPLIT_WIDTH/2; > + DeferWindowPos(hdwp, g_pChildWnd->hAddressBarWnd, 0, rt.left, rt.top, rt.right-rt.left - tHeight-2, tHeight, SWP_NOZORDER|SWP_NOACTIVATE); > + DeferWindowPos(hdwp, g_pChildWnd->hAddressBtnWnd, 0, rt.right - tHeight, rt.top, tHeight, tHeight, SWP_NOZORDER|SWP_NOACTIVATE); > + DeferWindowPos(hdwp, g_pChildWnd->hTreeWnd, 0, rt.left, rt.top + tHeight+2, g_pChildWnd->nSplitPos-SPLIT_WIDTH/2-rt.left, rt.bottom-rt.top-cy, SWP_NOZORDER|SWP_NOACTIVATE); > + DeferWindowPos(hdwp, g_pChildWnd->hListWnd, 0, rt.left+cx, rt.top + tHeight+2, rt.right-cx, rt.bottom-rt.top-cy, SWP_NOZORDER|SWP_NOACTIVATE); > EndDeferWindowPos(hdwp); > } > > @@ -111,7 +111,7 @@ > last_split = -1; > GetClientRect(hWnd, &rt); > g_pChildWnd->nSplitPos = x; > - ResizeWnd(g_pChildWnd, rt.right, rt.bottom); > + ResizeWnd(rt.right, rt.bottom); > ReleaseCapture(); > } > > @@ -125,7 +125,6 @@ > > static BOOL _CmdWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) > { > - ChildWnd* pChildWnd = g_pChildWnd; > HTREEITEM hSelection; > HKEY hRootKey; > LPCTSTR keyPath, s; > @@ -143,18 +142,18 @@ > /* TODO */ > break; > case ID_TREE_EXPANDBRANCH: > - (void)TreeView_Expand(pChildWnd->hTreeWnd, TreeView_GetSelection(pChildWnd->hTreeWnd), TVE_EXPAND); > + (void)TreeView_Expand(g_pChildWnd->hTreeWnd, TreeView_GetSelection(g_pChildWnd->hTreeWnd), TVE_EXPAND); > break; > case ID_TREE_COLLAPSEBRANCH: > - (void)TreeView_Expand(pChildWnd->hTreeWnd, TreeView_GetSelection(pChildWnd->hTreeWnd), TVE_COLLAPSE); > + (void)TreeView_Expand(g_pChildWnd->hTreeWnd, TreeView_GetSelection(g_pChildWnd->hTreeWnd), TVE_COLLAPSE); > break; > case ID_TREE_RENAME: > - SetFocus(pChildWnd->hTreeWnd); > - (void)TreeView_EditLabel(pChildWnd->hTreeWnd, TreeView_GetSelection(pChildWnd->hTreeWnd)); > + SetFocus(g_pChildWnd->hTreeWnd); > + (void)TreeView_EditLabel(g_pChildWnd->hTreeWnd, TreeView_GetSelection(g_pChildWnd->hTreeWnd)); > break; > case ID_TREE_DELETE: > - hSelection = TreeView_GetSelection(pChildWnd->hTreeWnd); > - keyPath = GetItemPath(pChildWnd->hTreeWnd, hSelection, &hRootKey); > + hSelection = TreeView_GetSelection(g_pChildWnd->hTreeWnd); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, hSelection, &hRootKey); > > if (keyPath == 0 || *keyPath == 0) > { > @@ -164,18 +163,18 @@ > DeleteNode(g_pChildWnd->hTreeWnd, 0); > break; > case ID_TREE_EXPORT: > - ExportRegistryFile(pChildWnd->hTreeWnd); > + ExportRegistryFile(g_pChildWnd->hTreeWnd); > break; > case ID_EDIT_FIND: > FindDialog(hWnd); > break; > case ID_EDIT_COPYKEYNAME: > - hSelection = TreeView_GetSelection(pChildWnd->hTreeWnd); > - keyPath = GetItemPath(pChildWnd->hTreeWnd, hSelection, &hRootKey); > + hSelection = TreeView_GetSelection(g_pChildWnd->hTreeWnd); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, hSelection, &hRootKey); > CopyKeyName(hWnd, hRootKey, keyPath); > break; > case ID_EDIT_NEW_KEY: > - CreateNewKey(pChildWnd->hTreeWnd, TreeView_GetSelection(pChildWnd->hTreeWnd)); > + CreateNewKey(g_pChildWnd->hTreeWnd, TreeView_GetSelection(g_pChildWnd->hTreeWnd)); > break; > case ID_EDIT_NEW_STRINGVALUE: > case ID_EDIT_NEW_BINARYVALUE: > @@ -183,8 +182,8 @@ > SendMessage(hFrameWnd, WM_COMMAND, wParam, lParam); > break; > case ID_SWITCH_PANELS: > - pChildWnd->nFocusPanel = !pChildWnd->nFocusPanel; > - SetFocus(pChildWnd->nFocusPanel? pChildWnd->hListWnd: pChildWnd->hTreeWnd); > + g_pChildWnd->nFocusPanel = !g_pChildWnd->nFocusPanel; > + SetFocus(g_pChildWnd->nFocusPanel? g_pChildWnd->hListWnd: g_pChildWnd->hTreeWnd); > break; > default: > if ((wID >= ID_TREE_SUGGESTION_MIN) && (wID <= ID_TREE_SUGGESTION_MAX)) > @@ -196,7 +195,7 @@ > s += _tcslen(s) + 1; > wID--; > } > - SelectNode(pChildWnd->hTreeWnd, s); > + SelectNode(g_pChildWnd->hTreeWnd, s); > break; > } > return FALSE; > @@ -317,7 +316,6 @@ > LRESULT CALLBACK ChildWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) > { > BOOL Result; > - ChildWnd* pChildWnd = g_pChildWnd; > > switch (message) > { > @@ -329,45 +327,45 @@ > /* load "My Computer" string */ > LoadString(hInst, IDS_MY_COMPUTER, buffer, sizeof(buffer)/sizeof(TCHAR)); > > - g_pChildWnd = pChildWnd = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(ChildWnd)); > - > - if (!pChildWnd) return 0; > - _tcsncpy(pChildWnd->szPath, buffer, MAX_PATH); > - pChildWnd->nSplitPos = 250; > - pChildWnd->hWnd = hWnd; > - pChildWnd->hAddressBarWnd = CreateWindowEx(WS_EX_CLIENTEDGE, _T("Edit"), NULL, WS_CHILD | WS_VISIBLE | WS_CHILDWINDOW | WS_TABSTOP, > + g_pChildWnd = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(ChildWnd)); > + if (!g_pChildWnd) return 0; > + > + _tcsncpy(g_pChildWnd->szPath, buffer, MAX_PATH); > + g_pChildWnd->nSplitPos = 250; > + g_pChildWnd->hWnd = hWnd; > + g_pChildWnd->hAddressBarWnd = CreateWindowEx(WS_EX_CLIENTEDGE, _T("Edit"), NULL, WS_CHILD | WS_VISIBLE | WS_CHILDWINDOW | WS_TABSTOP, > CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, > hWnd, (HMENU)0, hInst, 0); > - pChildWnd->hAddressBtnWnd = CreateWindowEx(WS_EX_CLIENTEDGE, _T("Button"), _T("Â»"), WS_CHILD | WS_VISIBLE | WS_CHILDWINDOW | WS_TABSTOP | BS_DEFPUSHBUTTON, > + g_pChildWnd->hAddressBtnWnd = CreateWindowEx(WS_EX_CLIENTEDGE, _T("Button"), _T("Â»"), WS_CHILD | WS_VISIBLE | WS_CHILDWINDOW | WS_TABSTOP | BS_DEFPUSHBUTTON, > CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, > hWnd, (HMENU)0, hInst, 0); > - pChildWnd->hTreeWnd = CreateTreeView(hWnd, pChildWnd->szPath, (HMENU) TREE_WINDOW); > - pChildWnd->hListWnd = CreateListView(hWnd, (HMENU) LIST_WINDOW/*, pChildWnd->szPath*/); > - SetFocus(pChildWnd->hTreeWnd); > + g_pChildWnd->hTreeWnd = CreateTreeView(hWnd, g_pChildWnd->szPath, (HMENU) TREE_WINDOW); > + g_pChildWnd->hListWnd = CreateListView(hWnd, (HMENU) LIST_WINDOW/*, g_pChildWnd->szPath*/); > + SetFocus(g_pChildWnd->hTreeWnd); > > /* set the address bar and button font */ > - if ((pChildWnd->hAddressBarWnd) && (pChildWnd->hAddressBtnWnd)) > + if ((g_pChildWnd->hAddressBarWnd) && (g_pChildWnd->hAddressBtnWnd)) > { > hFont = (HFONT)GetStockObject(DEFAULT_GUI_FONT); > - SendMessage(pChildWnd->hAddressBarWnd, > + SendMessage(g_pChildWnd->hAddressBarWnd, > WM_SETFONT, > (WPARAM)hFont, > 0); > - SendMessage(pChildWnd->hAddressBtnWnd, > + SendMessage(g_pChildWnd->hAddressBtnWnd, > WM_SETFONT, > (WPARAM)hFont, > 0); > } > /* Subclass the AddressBar */ > - oldproc = (WNDPROC)(LONG_PTR)GetWindowLongPtr(pChildWnd->hAddressBarWnd, GWL_WNDPROC); > - SetWindowLongPtr(pChildWnd->hAddressBarWnd, GWL_USERDATA, (DWORD_PTR)oldproc); > - SetWindowLongPtr(pChildWnd->hAddressBarWnd, GWL_WNDPROC, (DWORD_PTR)AddressBarProc); > + oldproc = (WNDPROC)(LONG_PTR)GetWindowLongPtr(g_pChildWnd->hAddressBarWnd, GWL_WNDPROC); > + SetWindowLongPtr(g_pChildWnd->hAddressBarWnd, GWL_USERDATA, (DWORD_PTR)oldproc); > + SetWindowLongPtr(g_pChildWnd->hAddressBarWnd, GWL_WNDPROC, (DWORD_PTR)AddressBarProc); > break; > } > case WM_COMMAND: > if(HIWORD(wParam) == BN_CLICKED) > { > - PostMessage(pChildWnd->hAddressBarWnd, WM_KEYUP, VK_RETURN, 0); > + PostMessage(g_pChildWnd->hAddressBarWnd, WM_KEYUP, VK_RETURN, 0); > } > else if (!_CmdWndProc(hWnd, message, wParam, lParam)) > { > @@ -383,7 +381,7 @@ > POINT pt; > GetCursorPos(&pt); > ScreenToClient(hWnd, &pt); > - if (pt.x>=pChildWnd->nSplitPos-SPLIT_WIDTH/2 && pt.x<pChildWnd->nSplitPos+SPLIT_WIDTH/2+1) > + if (pt.x>=g_pChildWnd->nSplitPos-SPLIT_WIDTH/2 && pt.x<g_pChildWnd->nSplitPos+SPLIT_WIDTH/2+1) > { > SetCursor(LoadCursor(0, IDC_SIZEWE)); > return TRUE; > @@ -392,10 +390,10 @@ > goto def; > case WM_DESTROY: > DestroyTreeView(); > - DestroyListView(pChildWnd->hListWnd); > + DestroyListView(g_pChildWnd->hListWnd); > DestroyMainMenu(); > - HeapFree(GetProcessHeap(), 0, pChildWnd); > - pChildWnd = NULL; > + HeapFree(GetProcessHeap(), 0, g_pChildWnd); > + g_pChildWnd = NULL; > PostQuitMessage(0); > break; > case WM_LBUTTONDOWN: > @@ -403,9 +401,9 @@ > RECT rt; > int x = (short)LOWORD(lParam); > GetClientRect(hWnd, &rt); > - if (x>=pChildWnd->nSplitPos-SPLIT_WIDTH/2 && x<pChildWnd->nSplitPos+SPLIT_WIDTH/2+1) > - { > - last_split = pChildWnd->nSplitPos; > + if (x>=g_pChildWnd->nSplitPos-SPLIT_WIDTH/2 && x<g_pChildWnd->nSplitPos+SPLIT_WIDTH/2+1) > + { > + last_split = g_pChildWnd->nSplitPos; > draw_splitbar(hWnd, last_split); > SetCapture(hWnd); > } > @@ -432,7 +430,7 @@ > RECT rt; > draw_splitbar(hWnd, last_split); > GetClientRect(hWnd, &rt); > - ResizeWnd(pChildWnd, rt.right, rt.bottom); > + ResizeWnd(rt.right, rt.bottom); > last_split = -1; > ReleaseCapture(); > SetCursor(LoadCursor(0, IDC_ARROW)); > @@ -476,9 +474,9 @@ > break; > > case WM_SETFOCUS: > - if (pChildWnd != NULL) > - { > - SetFocus(pChildWnd->nFocusPanel? pChildWnd->hListWnd: pChildWnd->hTreeWnd); > + if (g_pChildWnd != NULL) > + { > + SetFocus(g_pChildWnd->nFocusPanel? g_pChildWnd->hListWnd: g_pChildWnd->hTreeWnd); > } > break; > > @@ -486,22 +484,22 @@ > break; > > case WM_NOTIFY: > - if ((int)wParam == TREE_WINDOW) > + if ((int)wParam == TREE_WINDOW && g_pChildWnd != NULL) > { > switch (((LPNMHDR)lParam)->code) > { > case TVN_ITEMEXPANDING: > - return !OnTreeExpanding(pChildWnd->hTreeWnd, (NMTREEVIEW*)lParam); > + return !OnTreeExpanding(g_pChildWnd->hTreeWnd, (NMTREEVIEW*)lParam); > case TVN_SELCHANGED: > { > LPCTSTR keyPath, rootName; > LPTSTR fullPath; > HKEY hRootKey; > > - keyPath = GetItemPath(pChildWnd->hTreeWnd, ((NMTREEVIEW*)lParam)->itemNew.hItem, &hRootKey); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, ((NMTREEVIEW*)lParam)->itemNew.hItem, &hRootKey); > if (keyPath) > { > - RefreshListView(pChildWnd->hListWnd, hRootKey, keyPath); > + RefreshListView(g_pChildWnd->hListWnd, hRootKey, keyPath); > rootName = get_root_key_name(hRootKey); > fullPath = HeapAlloc(GetProcessHeap(), 0, (_tcslen(rootName) + 1 + _tcslen(keyPath) + 1) * sizeof(TCHAR)); > if (fullPath) > @@ -512,7 +510,7 @@ > else > fullPath = _tcscpy(fullPath, rootName); > SendMessage(hStatusBar, SB_SETTEXT, 0, (LPARAM)fullPath); > - SendMessage(pChildWnd->hAddressBarWnd, WM_SETTEXT, 0, (LPARAM)fullPath); > + SendMessage(g_pChildWnd->hAddressBarWnd, WM_SETTEXT, 0, (LPARAM)fullPath); > HeapFree(GetProcessHeap(), 0, fullPath); > /* disable hive manipulation items temporarily (enable only if necessary) */ > EnableMenuItem(GetSubMenu(hMenuFrame,0), ID_REGISTRY_LOADHIVE, MF_BYCOMMAND | MF_GRAYED); > @@ -547,15 +545,15 @@ > } > break; > case NM_SETFOCUS: > - pChildWnd->nFocusPanel = 0; > + g_pChildWnd->nFocusPanel = 0; > break; > case TVN_BEGINLABELEDIT: > { > LPNMTVDISPINFO ptvdi; > /* cancel label edit for rootkeys */ > ptvdi = (LPNMTVDISPINFO) lParam; > - if (!TreeView_GetParent(pChildWnd->hTreeWnd, ptvdi->item.hItem) || > - !TreeView_GetParent(pChildWnd->hTreeWnd, TreeView_GetParent(pChildWnd->hTreeWnd, ptvdi->item.hItem))) > + if (!TreeView_GetParent(g_pChildWnd->hTreeWnd, ptvdi->item.hItem) || > + !TreeView_GetParent(g_pChildWnd->hTreeWnd, TreeView_GetParent(g_pChildWnd->hTreeWnd, ptvdi->item.hItem))) > return TRUE; > break; > } > @@ -571,14 +569,14 @@ > ptvdi = (LPNMTVDISPINFO) lParam; > if (ptvdi->item.pszText) > { > - keyPath = GetItemPath(pChildWnd->hTreeWnd, TreeView_GetParent(pChildWnd->hTreeWnd, ptvdi->item.hItem), &hRootKey); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, TreeView_GetParent(g_pChildWnd->hTreeWnd, ptvdi->item.hItem), &hRootKey); > _sntprintf(szBuffer, sizeof(szBuffer) / sizeof(szBuffer[0]), _T("%s\\%s"), keyPath, ptvdi->item.pszText); > - keyPath = GetItemPath(pChildWnd->hTreeWnd, ptvdi->item.hItem, &hRootKey); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, ptvdi->item.hItem, &hRootKey); > if (RegOpenKeyEx(hRootKey, szBuffer, 0, KEY_READ, &hKey) == ERROR_SUCCESS) > { > lResult = FALSE; > RegCloseKey(hKey); > - (void)TreeView_EditLabel(pChildWnd->hTreeWnd, ptvdi->item.hItem); > + (void)TreeView_EditLabel(g_pChildWnd->hTreeWnd, ptvdi->item.hItem); > } > else > { > @@ -594,15 +592,15 @@ > } > else > { > - if ((int)wParam == LIST_WINDOW) > + if ((int)wParam == LIST_WINDOW && g_pChildWnd != NULL) > { > switch (((LPNMHDR)lParam)->code) > { > case NM_SETFOCUS: > - pChildWnd->nFocusPanel = 1; > + g_pChildWnd->nFocusPanel = 1; > break; > default: > - if(!ListWndNotifyProc(pChildWnd->hListWnd, wParam, lParam, &Result)) > + if(!ListWndNotifyProc(g_pChildWnd->hListWnd, wParam, lParam, &Result)) > { > goto def; > } > @@ -616,27 +614,27 @@ > case WM_CONTEXTMENU: > { > POINT pt; > - if((HWND)wParam == pChildWnd->hListWnd) > + if((HWND)wParam == g_pChildWnd->hListWnd) > { > int i, cnt; > BOOL IsDefault; > pt.x = (short) LOWORD(lParam); > pt.y = (short) HIWORD(lParam); > - cnt = ListView_GetSelectedCount(pChildWnd->hListWnd); > - i = ListView_GetNextItem(pChildWnd->hListWnd, -1, LVNI_FOCUSED | LVNI_SELECTED); > + cnt = ListView_GetSelectedCount(g_pChildWnd->hListWnd); > + i = ListView_GetNextItem(g_pChildWnd->hListWnd, -1, LVNI_FOCUSED | LVNI_SELECTED); > if (pt.x == -1 && pt.y == -1) > { > RECT rc; > if (i != -1) > { > rc.left = LVIR_BOUNDS; > - SendMessage(pChildWnd->hListWnd, LVM_GETITEMRECT, i, (LPARAM) &rc); > + SendMessage(g_pChildWnd->hListWnd, LVM_GETITEMRECT, i, (LPARAM) &rc); > pt.x = rc.left + 8; > pt.y = rc.top + 8; > } > else > pt.x = pt.y = 0; > - ClientToScreen(pChildWnd->hListWnd, &pt); > + ClientToScreen(g_pChildWnd->hListWnd, &pt); > } > if(i == -1) > { > @@ -646,7 +644,7 @@ > { > HMENU mnu = GetSubMenu(hPopupMenus, PM_MODIFYVALUE); > SetMenuDefaultItem(mnu, ID_EDIT_MODIFY, MF_BYCOMMAND); > - IsDefault = IsDefaultValue(pChildWnd->hListWnd, i); > + IsDefault = IsDefaultValue(g_pChildWnd->hListWnd, i); > if(cnt == 1) > EnableMenuItem(mnu, ID_EDIT_RENAME, MF_BYCOMMAND | (IsDefault ? MF_DISABLED | MF_GRAYED : MF_ENABLED)); > else > @@ -657,7 +655,7 @@ > TrackPopupMenu(mnu, TPM_RIGHTBUTTON, pt.x, pt.y, 0, hFrameWnd, NULL); > } > } > - else if ((HWND)wParam == pChildWnd->hTreeWnd) > + else if ((HWND)wParam == g_pChildWnd->hTreeWnd) > { > TVHITTESTINFO hti; > HMENU hContextMenu; > @@ -677,13 +675,13 @@ > if (pt.x == -1 && pt.y == -1) > { > RECT rc; > - hti.hItem = TreeView_GetSelection(pChildWnd->hTreeWnd); > + hti.hItem = TreeView_GetSelection(g_pChildWnd->hTreeWnd); > if (hti.hItem != NULL) > { > - TreeView_GetItemRect(pChildWnd->hTreeWnd, hti.hItem, &rc, TRUE); > + TreeView_GetItemRect(g_pChildWnd->hTreeWnd, hti.hItem, &rc, TRUE); > pt.x = rc.left + 8; > pt.y = rc.top + 8; > - ClientToScreen(pChildWnd->hTreeWnd, &pt); > + ClientToScreen(g_pChildWnd->hTreeWnd, &pt); > hti.flags = TVHT_ONITEM; > } > else > @@ -693,19 +691,19 @@ > { > hti.pt.x = pt.x; > hti.pt.y = pt.y; > - ScreenToClient(pChildWnd->hTreeWnd, &hti.pt); > - (void)TreeView_HitTest(pChildWnd->hTreeWnd, &hti); > + ScreenToClient(g_pChildWnd->hTreeWnd, &hti.pt); > + (void)TreeView_HitTest(g_pChildWnd->hTreeWnd, &hti); > } > > if (hti.flags & TVHT_ONITEM) > { > hContextMenu = GetSubMenu(hPopupMenus, PM_TREECONTEXT); > - (void)TreeView_SelectItem(pChildWnd->hTreeWnd, hti.hItem); > + (void)TreeView_SelectItem(g_pChildWnd->hTreeWnd, hti.hItem); > > memset(&item, 0, sizeof(item)); > item.mask = TVIF_STATE | TVIF_CHILDREN; > item.hItem = hti.hItem; > - (void)TreeView_GetItem(pChildWnd->hTreeWnd, &item); > + (void)TreeView_GetItem(g_pChildWnd->hTreeWnd, &item); > > /* Set the Expand/Collapse menu item appropriately */ > LoadString(hInst, (item.state & TVIS_EXPANDED) ? IDS_COLLAPSE : IDS_EXPAND, buffer, sizeof(buffer) / sizeof(buffer[0])); > @@ -734,7 +732,7 @@ > } > > /* Come up with suggestions */ > - keyPath = GetItemPath(pChildWnd->hTreeWnd, NULL, &hRootKey); > + keyPath = GetItemPath(g_pChildWnd->hTreeWnd, NULL, &hRootKey); > SuggestKeys(hRootKey, keyPath, Suggestions, sizeof(Suggestions) / sizeof(Suggestions[0])); > if (Suggestions[0]) > { > @@ -758,16 +756,16 @@ > s += _tcslen(s) + 1; > } > } > - TrackPopupMenu(hContextMenu, TPM_RIGHTBUTTON, pt.x, pt.y, 0, pChildWnd->hWnd, NULL); > + TrackPopupMenu(hContextMenu, TPM_RIGHTBUTTON, pt.x, pt.y, 0, g_pChildWnd->hWnd, NULL); > } > } > break; > } > > case WM_SIZE: > - if (wParam != SIZE_MINIMIZED && pChildWnd != NULL) > - { > - ResizeWnd(pChildWnd, LOWORD(lParam), HIWORD(lParam)); > + if (wParam != SIZE_MINIMIZED && g_pChildWnd != NULL) > + { > + ResizeWnd(LOWORD(lParam), HIWORD(lParam)); > } > /* fall through */ > default: > > Modified: trunk/reactos/dll/cpl/input/settings.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/cpl/input/settings.c?r… > ============================================================================== > --- trunk/reactos/dll/cpl/input/settings.c [iso-8859-1] (original) > +++ trunk/reactos/dll/cpl/input/settings.c [iso-8859-1] Mon Mar 21 14:07:29 2011 > @@ -14,7 +14,6 @@ > #include "input.h" > > static HWND MainDlgWnd; > -static HIMAGELIST hImgList; > // for SaveInputLang() > static INT OldLayoutNum; > > @@ -299,7 +298,7 @@ > } > > static VOID > -InitLangList(HWND hWnd) > +InitLangList(HWND hWnd, HIMAGELIST hImgList) > { > HKEY hKey, hSubKey; > TCHAR szBuf[MAX_PATH], szPreload[CCH_LAYOUT_ID + 1], szSub[CCH_LAYOUT_ID + 1]; > @@ -370,11 +369,19 @@ > VOID > UpdateLayoutsList(VOID) > { > - (VOID) ImageList_Destroy(hImgList); > + HIMAGELIST hImgList; > + > + /* Clear the list */ > (VOID) ListView_DeleteAllItems(GetDlgItem(MainDlgWnd, IDC_KEYLAYOUT_LIST)); > + > + /* Crate new list */ > hImgList = ImageList_Create(16, 16, ILC_COLOR8 | ILC_MASK, 0, 1); > - InitLangList(MainDlgWnd); > - (VOID) ListView_SetImageList(GetDlgItem(MainDlgWnd, IDC_KEYLAYOUT_LIST), hImgList, LVSIL_SMALL); > + InitLangList(MainDlgWnd, hImgList); > + hImgList = ListView_SetImageList(GetDlgItem(MainDlgWnd, IDC_KEYLAYOUT_LIST), hImgList, LVSIL_SMALL); > + > + /* Destroy old image list */ > + if(hImgList) > + (VOID) ImageList_Destroy(hImgList); > } > > typedef struct _REG_KB_ENTRY_ > @@ -728,12 +735,14 @@ > { > case WM_INITDIALOG: > { > + HIMAGELIST hImgList; > + > MainDlgWnd = hwndDlg; > AddListColumn(hwndDlg); > (VOID) ListView_SetExtendedListViewStyle(GetDlgItem(MainDlgWnd, IDC_KEYLAYOUT_LIST), > LVS_EX_FULLROWSELECT); > hImgList = ImageList_Create(16, 16, ILC_COLOR8 | ILC_MASK, 0, 1); > - InitLangList(hwndDlg); > + InitLangList(hwndDlg, hImgList); > (VOID) ListView_SetImageList(GetDlgItem(MainDlgWnd, IDC_KEYLAYOUT_LIST), hImgList, LVSIL_SMALL); > } > break; > @@ -781,7 +790,6 @@ > } > break; > case WM_DESTROY: > - (VOID) ImageList_Destroy(hImgList); > break; > } > > >

13 years, 7 months

Theming

by Andrew Green

I have been doing some investigation on theming. I have been looking into comctl32. One of the issues is that windows version 5 of the control has not theming and uses SxS so that applications that use version 6 will get themed controls. Currently comctl only has one version and applies theming regardless of the version the application requests. I have spoken on the Wine mailing list and the only person that replied did not want to use SxS. I assume to make it easier to maintain(though i don't understand why common parts cant share the same files and build 2 dlls). I was wondering what React OS developers think is the best choice. The second issue is that if comctl32 version 6 is loaded in windows it takes control of components that are usually in user32. Someone said this was called ( window class redirection). I can't really find any information on it. Though isn't this just registering the class except from comctl32? or does this cause a race condition(with the last one one loaded taking control)? is this is how "window class redirection" is different? Also the last thing is. Should sub classing be used? You should be able to load another window proc method even from user32 and call it when it is suitable from new control. I would imagine this could cause issues when the default procedure can't handle something as the new control would. Any advise would be much appreciated.

13 years, 7 months

Uninitialized static variable in function

by Alexander Potashev

Hi, In the commit r51083 (http://git.reactos.org/?p=reactos.git;a=commitdiff;h=83ba2eecbf0ff43ec3a178…), please, initialize "i" to 0. Non-initialized variable store garbage, not zeroes. -- Alexander Potashev

13 years, 7 months

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev March 2011