Ros-diffs

ros-diffs@reactos.org

81390 discussions

[gdalsnes] 17607: mostly naming changes
by gdalsnes＠svn.reactos.com 02 Sep '05

02 Sep '05

mostly naming changes Modified: branches/win32k rewrite attempt/win32k/eng/window.c Modified: branches/win32k rewrite attempt/win32k/include/accelerator.h Modified: branches/win32k rewrite attempt/win32k/include/cursoricon.h Modified: branches/win32k rewrite attempt/win32k/include/hook.h Modified: branches/win32k rewrite attempt/win32k/include/ntuser.h Modified: branches/win32k rewrite attempt/win32k/include/object.h Modified: branches/win32k rewrite attempt/win32k/include/userfuncs.h Modified: branches/win32k rewrite attempt/win32k/include/win32.h Modified: branches/win32k rewrite attempt/win32k/include/window.h Modified: branches/win32k rewrite attempt/win32k/include/winpos.h Modified: branches/win32k rewrite attempt/win32k/ntuser/accelerator.c Modified: branches/win32k rewrite attempt/win32k/ntuser/callback.c Modified: branches/win32k rewrite attempt/win32k/ntuser/caret.c Modified: branches/win32k rewrite attempt/win32k/ntuser/class.c Modified: branches/win32k rewrite attempt/win32k/ntuser/cursoricon.c Modified: branches/win32k rewrite attempt/win32k/ntuser/desktop.c Modified: branches/win32k rewrite attempt/win32k/ntuser/focus.c Modified: branches/win32k rewrite attempt/win32k/ntuser/hook.c Modified: branches/win32k rewrite attempt/win32k/ntuser/hotkey.c Modified: branches/win32k rewrite attempt/win32k/ntuser/input.c Modified: branches/win32k rewrite attempt/win32k/ntuser/menu.c Modified: branches/win32k rewrite attempt/win32k/ntuser/message.c Modified: branches/win32k rewrite attempt/win32k/ntuser/misc.c Modified: branches/win32k rewrite attempt/win32k/ntuser/monitor.c Modified: branches/win32k rewrite attempt/win32k/ntuser/msgqueue.c Modified: branches/win32k rewrite attempt/win32k/ntuser/ntuser.c Modified: branches/win32k rewrite attempt/win32k/ntuser/object.c Modified: branches/win32k rewrite attempt/win32k/ntuser/painting.c Modified: branches/win32k rewrite attempt/win32k/ntuser/prop.c Modified: branches/win32k rewrite attempt/win32k/ntuser/scrollbar.c Modified: branches/win32k rewrite attempt/win32k/ntuser/timer.c Modified: branches/win32k rewrite attempt/win32k/ntuser/vis.c Modified: branches/win32k rewrite attempt/win32k/ntuser/windc.c Modified: branches/win32k rewrite attempt/win32k/ntuser/window.c Modified: branches/win32k rewrite attempt/win32k/ntuser/winpos.c Modified: branches/win32k rewrite attempt/win32k/ntuser/winsta.c Modified: branches/win32k rewrite attempt/win32k/objects/color.c Modified: branches/win32k rewrite attempt/win32k/objects/dc.c _____ Modified: branches/win32k rewrite attempt/win32k/eng/window.c --- branches/win32k rewrite attempt/win32k/eng/window.c 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/eng/window.c 2005-09-02 20:45:56 UTC (rev 17607) @@ -210,7 +210,8 @@ pso, hwnd, pfn, fl, iPixelFormat); /* Get window object */ - Window = IntGetWindowObject(hwnd); + //FIXME: user lock????? + Window = UserGetWindowObject(hwnd); if (Window == NULL) { return NULL; @@ -262,11 +263,16 @@ { WNDGDI *WndObjInt = ObjToGDI(pwo, WND); PWINDOW_OBJECT Window; + BOOL inUser; DPRINT("EngDeleteWnd: pwo = 0x%x\n", pwo); /* Get window object */ - Window = IntGetWindowObject(WndObjInt->Hwnd); + inUser = UserIsEntered(); + if (!inUser){ + UserEnterShared(); + } + Window = UserGetWindowObject(WndObjInt->Hwnd); if (Window == NULL) { DPRINT1("Warning: Couldnt get window object for WndObjInt->Hwnd!!!\n"); @@ -277,6 +283,9 @@ /* Remove object from window */ RemoveEntryList(&WndObjInt->ListEntry); } + if (!inUser){ + UserLeave(); + } /* Free resources */ IntEngDeleteClipRegion(WndObjInt->ClientClipObj); _____ Modified: branches/win32k rewrite attempt/win32k/include/accelerator.h --- branches/win32k rewrite attempt/win32k/include/accelerator.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/accelerator.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -13,7 +13,6 @@ { /*---------- USER_OBJECT_HDR --------------*/ HACCEL hSelf; /* want typesafe handle */ - LONG refs_placeholder; BYTE flags_placeholder; /*---------- USER_OBJECT_HDR --------------*/ _____ Modified: branches/win32k rewrite attempt/win32k/include/cursoricon.h --- branches/win32k rewrite attempt/win32k/include/cursoricon.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/cursoricon.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -18,7 +18,6 @@ { /*---------- USER_OBJECT_HDR --------------*/ HCURSOR hSelf; /* want typesafe handle */ - LONG refs_placeholder; BYTE flags_placeholder; /*---------- USER_OBJECT_HDR --------------*/ _____ Modified: branches/win32k rewrite attempt/win32k/include/hook.h --- branches/win32k rewrite attempt/win32k/include/hook.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/hook.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -7,7 +7,6 @@ { /*---------- USER_OBJECT_HDR --------------*/ HHOOK hSelf; - LONG refs; BYTE hdrFlags; /*---------- USER_OBJECT_HDR --------------*/ _____ Modified: branches/win32k rewrite attempt/win32k/include/ntuser.h --- branches/win32k rewrite attempt/win32k/include/ntuser.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/ntuser.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -5,18 +5,25 @@ typedef struct _USER_OBJECT_HDR { HANDLE hSelf; - LONG refs; BYTE flags; + } USER_OBJECT_HDR, *PUSER_OBJECT_HDR; typedef struct _USER_REFERENCE_ENTRY { SINGLE_LIST_ENTRY Entry; - PUSER_OBJECT_HDR hdr; + PUSER_OBJECT_HDR* ppHdr; } USER_REFERENCE_ENTRY, *PUSER_REFERENCE_ENTRY; +/* ul = user lock */ +#define ulShared 1 +#define ulExclusive 2 +/* unused for now... */ +//#define ASSERT_LOCK(type) ASSERT(PsGetWin32Thread()->LockType >= type) +#define ASSERT_LOCK(type) + extern char* _file; extern DWORD _line; extern DWORD _locked; @@ -28,35 +35,14 @@ #define CLEANUP /*unreachable*/ ASSERT(FALSE); _cleanup_ #define END_CLEANUP return _ret_; -#if 0 -#define UserEnterShared() { \ - UUserEnterShared(); ASSERT(InterlockedIncrement(&_locked) > 0); \ - } - #define UserEnterExclusive() {\ - UUserEnterExclusive(); ASSERT(InterlockedIncrement(&_locked) > 0); \ - } - -#define UserLeave() { ASSERT(InterlockedDecrement(&_locked) >= 0); \ - UUserLeave(); } - -#endif - - VOID FASTCALL UserStackTrace(); -#define UserEnterShared() \ -{ \ - DPRINT1("try lock, %s, %i (%i)\n",__FILE__,__LINE__, _locked); \ - ASSERT(UserLock.Owner != KeGetCurrentThread()); \ - UUserEnterShared(); \ - ASSERT(InterlockedIncrement(&_locked) == 1 /*> 0*/); \ - DPRINT("got lock, %s, %i (%i)\n",__FILE__,__LINE__, _locked); \ -} +#define UserEnterShared() UserEnterExclusive() #define UserEnterExclusive() \ { \ - /* DPRINT1("try lock, %s, %i (%i)\n",__FILE__,__LINE__, _locked);*/ \ + /* DPRINT1("try xlock, %s, %i (%i)\n",__FILE__,__LINE__, _locked);*/ \ if (UserLock.Owner == KeGetCurrentThread()){ \ DPRINT1("file %s, line %i\n",_file, _line); \ ASSERT(FALSE); \ @@ -83,27 +69,11 @@ -#define GetWnd(hwnd) IntGetWindowObject(hwnd) +#define GetWnd(hwnd) UserGetWindowObject(hwnd) -#if 0 -#define IntLockUserShared() {if(_locked){ DPRINT1("last %s, %i\n",_file,_line);} \ - IIntLockUserShared(); \ - _locked++; _file = __FILE__; _line = __LINE__; \ - } - - #define IntUserEnterExclusive() {if(_locked){ DPRINT1("last %s, %i\n",_file,_line);} \ - IIntUserEnterExclusive(); \ - _locked++; _file = __FILE__; _line = __LINE__; \ - } - - -#define IntUserLeave() { if(!_locked){ DPRINT1("not locked %s, %i\n",__FILE__,__LINE__);} \ - _locked--; IIntUserLeave(); } -#endif - NTSTATUS FASTCALL InitUserImpl(VOID); VOID FASTCALL UninitUser(VOID); VOID FASTCALL UUserEnterShared(VOID); _____ Modified: branches/win32k rewrite attempt/win32k/include/object.h --- branches/win32k rewrite attempt/win32k/include/object.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/object.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -6,6 +6,26 @@ #include <win32k/pen.h> +typedef struct _VOLATILE_OBJECT_ENTRY +{ + PUSER_OBJECT_HDR hdr; + PUSER_HANDLE_ENTRY handle_entry; + WORD recycle_count; + WORD objType; + SINGLE_LIST_ENTRY link; +} VOLATILE_OBJECT_ENTRY, *PVOLATILE_OBJECT_ENTRY; + + +typedef struct _OBJECT_ENTRY +{ + PUSER_OBJECT_HDR hdr; + PUSER_HANDLE_ENTRY handle_entry; + WORD generation; + WORD type; +} OBJECT_ENTRY, *POBJECT_ENTRY; + + + #define USER_OBJ_DESTROYING (0x1) #define USER_OBJ_DESTROYED (0x2) _____ Modified: branches/win32k rewrite attempt/win32k/include/userfuncs.h --- branches/win32k rewrite attempt/win32k/include/userfuncs.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/userfuncs.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -12,7 +12,7 @@ #define UserGetCurrentQueue() (&PsGetWin32Thread()->Queue) #define UserGetForegroundWThread() (QUEUE_2_WTHREAD(gInputDesktop->ActiveQueue)) -#define UserGetForegroundQueue() (gInputDesktop->ActiveQueue) +#define UserGetForegroundQueue() (gInputDesktop ? gInputDesktop->ActiveQueue : NULL) #define UserSetForegroundQueue(q) (gInputDesktop->ActiveQueue = (q)) @@ -33,44 +33,28 @@ /******************** OBJECT.C ***************/ -BOOLEAN FASTCALL UserSetCheckDestroy(PUSER_OBJECT_HDR hdr); - -VOID FASTCALL UserDestroyObject(PUSER_OBJECT_HDR obj); - /******************** HANDLE.C ***************/ extern USER_HANDLE_TABLE gHandleTable; PUSER_HANDLE_ENTRY handle_to_entry(PUSER_HANDLE_TABLE ht, HANDLE handle ); +#define VOLATILE_OBJECTS_LIST SINGLE_LIST_ENTRY __object_list__ = {NULL}; + + +#define UserLinkVolatileObject(obj) UserLinkVolatileObject2(&obj->hdr, &__object_list__, (PVOLATILE_OBJECT_ENTRY)_alloca(sizeof(VOLATILE_OBJECT_ENTRY))) +#define UserValidateVolatileObjects() UserValidateVolatileObjects2(&__object_list__) -#include <malloc.h> +VOID FASTCALL UserLinkVolatileObject2(PUSER_OBJECT_HDR hdr, PSINGLE_LIST_ENTRY list, PVOLATILE_OBJECT_ENTRY e); +BOOLEAN FASTCALL UserValidateVolatileObjects2(PSINGLE_LIST_ENTRY list); +#define UserGetEntry(obj) UserGetEntry2(&(obj)->hdr) +#define UserValidateEntry(obj) UserValidateEntry2(&(obj)) -#define UserReferenceObject(obj) (obj->hdr.refs++) -#define UserDereferenceObject(obj){ \ - if (--obj->hdr.refs == 0 && obj->hdr.flags & USER_OBJ_DESTROYING && !(obj->hdr.flags & USER_OBJ_DESTROYED)){ \ - UserDestroyObject(&obj->hdr); \ - } \ -} +inline OBJECT_ENTRY FASTCALL UserGetEntry2(PUSER_OBJECT_HDR hdr); +inline BOOLEAN FASTCALL UserValidateEntry2(POBJECT_ENTRY e); - -#define UserReferenceObjectCo(obj){ \ - PUSER_REFERENCE_ENTRY ref = (PUSER_REFERENCE_ENTRY)_alloca(sizeof(USER_REFERENCE_ENTRY)); \ - obj->hdr.refs++; \ - ref->hdr = &obj->hdr; \ - PushEntryList(&PsGetWin32Thread()->UserObjReferencesStack, &ref->Entry); \ -} - -#define UserDereferenceObjectCo(obj){ \ - PSINGLE_LIST_ENTRY entry = PopEntryList(&PsGetWin32Thread()->UserObjReferencesStack); \ - PUSER_REFERENCE_ENTRY ref = CONTAINING_RECORD(entry, USER_REFERENCE_ENTRY, Entry); \ - ASSERT(&obj->hdr == ref->hdr); \ - if (--ref->hdr->refs == 0 && obj->hdr.flags & USER_OBJ_DESTROYING && !(obj->hdr.flags & USER_OBJ_DESTROYED)) { \ - UserDestroyObject(ref->hdr); \ - } \ -} VOID UserInitHandleTable(PUSER_HANDLE_TABLE ht, PVOID mem, ULONG bytes); HANDLE UserAllocHandle(PUSER_HANDLE_TABLE ht, PVOID object, USER_OBJECT_TYPE type ); @@ -90,7 +74,7 @@ /************* CALLBACK.C *****************/ LRESULT STDCALL -coUserCallWindowProc(WNDPROC Proc, +co_UserCallWindowProc(WNDPROC Proc, BOOLEAN IsAnsiProc, HWND Wnd, UINT Message, @@ -99,7 +83,7 @@ INT lParamBufferSize); VOID STDCALL -coUserCallSentMessageCallback(SENDASYNCPROC CompletionCallback, +co_UserCallSentMessageCallback(SENDASYNCPROC CompletionCallback, HWND hWnd, UINT Msg, ULONG_PTR CompletionCallbackContext, @@ -107,13 +91,13 @@ HMENU STDCALL -coUserLoadSysMenuTemplate(); +co_UserLoadSysMenuTemplate(); BOOL STDCALL -coUserLoadDefaultCursors(VOID); +co_UserLoadDefaultCursors(VOID); LRESULT STDCALL -coUserCallHookProc(INT HookId, +co_UserCallHookProc(INT HookId, INT Code, WPARAM wParam, LPARAM lParam, @@ -176,7 +160,7 @@ UserGetActiveDesktop(VOID); NTSTATUS FASTCALL -coUserShowDesktop(PDESKTOP_OBJECT Desktop, ULONG Width, ULONG Height); +co_UserShowDesktop(PDESKTOP_OBJECT Desktop, ULONG Width, ULONG Height); NTSTATUS FASTCALL IntHideDesktop(PDESKTOP_OBJECT Desktop); @@ -203,7 +187,7 @@ BOOL IntRegisterShellHookWindow(HWND hWnd); BOOL IntDeRegisterShellHookWindow(HWND hWnd); -VOID IntShellHookNotify(WPARAM Message, LPARAM lParam); +VOID co_UserShellHookNotify(WPARAM Message, LPARAM lParam); #define IntIsActiveDesktop(Desktop) ((Desktop) == (gInputDesktop)) @@ -281,17 +265,17 @@ UserGetThreadFocusWindow(); BOOL FASTCALL -coUserMouseActivateWindow(PWINDOW_OBJECT Window); +co_UserMouseActivateWindow(PWINDOW_OBJECT Window); BOOL FASTCALL -coUserSetForegroundWindow(PWINDOW_OBJECT Window); +co_UserSetForegroundWindow(PWINDOW_OBJECT Window); PWINDOW_OBJECT FASTCALL -coUserSetActiveWindow(PWINDOW_OBJECT Window); +co_UserSetActiveWindow(PWINDOW_OBJECT Window); PWINDOW_OBJECT FASTCALL UserGetForegroundWindow(VOID); PWINDOW_OBJECT FASTCALL -coUserSetFocus(PWINDOW_OBJECT Wnd OPTIONAL); +co_UserSetFocus(PWINDOW_OBJECT Wnd OPTIONAL); /******************** PAINTING.C ********************************/ @@ -299,12 +283,12 @@ VOID FASTCALL IntValidateParent(PWINDOW_OBJECT Child, HRGN ValidRegion); BOOL FASTCALL -coUserRedrawWindow(PWINDOW_OBJECT Wnd, const RECT* UpdateRect, HRGN UpdateRgn, ULONG Flags); +co_UserRedrawWindow(PWINDOW_OBJECT Wnd, const RECT* UpdateRect, HRGN UpdateRgn, ULONG Flags); BOOL FASTCALL IntGetPaintMessage(HWND hWnd, UINT MsgFilterMin, UINT MsgFilterMax, PW32THREAD Thread, MSG *Message, BOOL Remove); -BOOL FASTCALL coUserValidateRgn(PWINDOW_OBJECT hWnd, HRGN hRgn); +BOOL FASTCALL co_UserValidateRgn(PWINDOW_OBJECT hWnd, HRGN hRgn); #define IntLockWindowUpdate(Window) \ @@ -321,7 +305,7 @@ const RECT *lprcClip, HRGN hrgnUpdate, LPRECT lprcUpdate); INT FASTCALL -coUserGetUpdateRgn(PWINDOW_OBJECT Window, HRGN hRgn, BOOL bErase); +co_UserGetUpdateRgn(PWINDOW_OBJECT Window, HRGN hRgn, BOOL bErase); /******************** MESSAGE.C ********************************/ @@ -329,7 +313,7 @@ UserPostMessage(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam); LRESULT FASTCALL -coUserSendMessage(HWND hWnd, +co_UserSendMessage(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam); @@ -337,7 +321,7 @@ LRESULT FASTCALL -coUserSendMessageTimeout(HWND hWnd, +co_UserSendMessageTimeout(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam, @@ -352,12 +336,15 @@ /************************ WINDOW.C *****************************/ +PWINDOW_OBJECT FASTCALL UserGetWindowObject_UpdateHandle2(HWND* h); +#define UserGetWindowObject_UpdateHandle(handle) UserGetWindowObject_UpdateHandle2(&(handle)) + BOOL FASTCALL UserSetSystemMenu(PWINDOW_OBJECT WindowObject, PMENU_OBJECT MenuObject); inline VOID FASTCALL UserFreeWindowObject(PWINDOW_OBJECT Wnd); -PWINDOW_OBJECT FASTCALL IntGetWindowObject(HWND hWnd); +inline PWINDOW_OBJECT FASTCALL UserGetWindowObject(HWND hWnd); PWINDOW_OBJECT FASTCALL UserCreateWindowObject(ULONG bytes); @@ -368,14 +355,119 @@ UserGetWindowLong(PWINDOW_OBJECT Wnd, DWORD Index, BOOL Ansi); LONG FASTCALL -coUserSetWindowLong(PWINDOW_OBJECT Wnd, DWORD Index, LONG NewValue, BOOL Ansi); +co_UserSetWindowLong(PWINDOW_OBJECT Wnd, DWORD Index, LONG NewValue, BOOL Ansi); BOOLEAN FASTCALL -coUserDestroyWindow(PWINDOW_OBJECT Wnd); +co_UserDestroyWindow(PWINDOW_OBJECT Wnd); HWND FASTCALL -GetHwnd(PWINDOW_OBJECT Wnd); +GetHwndSafe(PWINDOW_OBJECT Wnd); +#define HAS_DLGFRAME(Style, ExStyle) \ + (((ExStyle) & WS_EX_DLGMODALFRAME) || \ + (((Style) & WS_DLGFRAME) && (!((Style) & WS_THICKFRAME)))) + +#define HAS_THICKFRAME(Style, ExStyle) \ + (((Style) & WS_THICKFRAME) && \ + (!(((Style) & (WS_DLGFRAME | WS_BORDER)) == WS_DLGFRAME))) + +#define HAS_THINFRAME(Style, ExStyle) \ + (((Style) & WS_BORDER) || (!((Style) & (WS_CHILD | WS_POPUP)))) + +#define IntIsDesktopWindow(WndObj) \ + (WndObj->ParentWnd == NULL) + +#define IntIsBroadcastHwnd(hWnd) \ + (hWnd == HWND_BROADCAST || hWnd == HWND_TOPMOST) + +#if 0 +#define IntWndBelongsToThread(WndObj, W32Thread) \ + (((WndObj->W32Thread->Thread && WndObj->W32Thread->Thread->Tcb.Win32Thread)) && \ + (WndObj->W32Thread->Thread->Tcb.Win32Thread == W32Thread)) +#endif + +#define IntWndBelongsToThread(WndObj, _W32Thread) (QUEUE_2_WTHREAD((WndObj)->Queue) == (_W32Thread)) + + +#define IntGetWndThreadId(WndObj) \ + WndObj->WThread->Thread->Cid.UniqueThread + +#define IntGetWndProcessId(WndObj) \ + WndObj->WThread->Thread->ThreadsProcess->UniqueProcessId + +PWINDOW_OBJECT FASTCALL +IntGetProcessWindowObject(PW32THREAD Thread, HWND hWnd); + +BOOL FASTCALL +IntIsWindow(HWND hWnd); + +PWINDOW_OBJECT* FASTCALL +UserListChildWnd(PWINDOW_OBJECT Window); + +HWND* FASTCALL +IntWinListChildren(PWINDOW_OBJECT Window); + +NTSTATUS FASTCALL +InitWindowImpl (VOID); + +NTSTATUS FASTCALL +CleanupWindowImpl (VOID); + +VOID FASTCALL +IntGetClientRect (PWINDOW_OBJECT WindowObject, PRECT Rect); + +PWINDOW_OBJECT FASTCALL +UserGetActiveWindow(VOID); + +BOOL FASTCALL +UserIsWindowVisible (PWINDOW_OBJECT hWnd); + +BOOL FASTCALL +UserIsChildWindow (PWINDOW_OBJECT Parent, PWINDOW_OBJECT Child); + +VOID FASTCALL +IntUnlinkWindow(PWINDOW_OBJECT Wnd); + +VOID FASTCALL +IntLinkWindow(PWINDOW_OBJECT Wnd, PWINDOW_OBJECT WndParent, PWINDOW_OBJECT WndPrevSibling); + +PWINDOW_OBJECT FASTCALL +UserGetAncestor(PWINDOW_OBJECT Wnd, UINT Type); + +PWINDOW_OBJECT FASTCALL +UserGetParent(PWINDOW_OBJECT Wnd); + +inline PWINDOW_OBJECT FASTCALL +UserGetOwner(PWINDOW_OBJECT Wnd); + +INT FASTCALL +IntGetWindowRgn(HWND hWnd, HRGN hRgn); + +INT FASTCALL +IntGetWindowRgnBox(HWND hWnd, RECT *Rect); + +PWINDOW_OBJECT FASTCALL +UserGetShellWindow(); + +BOOL FASTCALL +IntGetWindowInfo(PWINDOW_OBJECT WindowObject, PWINDOWINFO pwi); + +VOID FASTCALL +IntGetWindowBorderMeasures(PWINDOW_OBJECT WindowObject, UINT *cx, UINT *cy); + +BOOL FASTCALL +IntAnyPopup(VOID); + +BOOL FASTCALL +IntIsWindowInDestroy(PWINDOW_OBJECT Window); + +DWORD IntRemoveWndProcHandle(WNDPROC Handle); +DWORD IntRemoveProcessWndProcHandles(HANDLE ProcessID); +DWORD IntAddWndProcHandle(WNDPROC WindowProc, BOOL IsUnicode); + +BOOL FASTCALL +co_UserShowOwnedPopups( HWND owner, BOOL fShow ); + /************************* WINSTA.C ****************************/ inline PWINSTATION_OBJECT FASTCALL UserGetCurrentWinSta(); @@ -441,47 +533,16 @@ /************************* MENU.C ****************************/ PMENU_OBJECT FASTCALL -coUserGetSystemMenu(PWINDOW_OBJECT WindowObject, BOOL bRevert, BOOL RetMenu); +co_UserGetSystemMenu(PWINDOW_OBJECT WindowObject, BOOL bRevert, BOOL RetMenu); -PMENU_OBJECT FASTCALL UserAllocMenuObject(HMENU* h); - inline PMENU_OBJECT FASTCALL UserGetMenuObject(HMENU hMenu); -PMENU_OBJECT FASTCALL -UserCreateMenu(BOOL PopupMenu); - - -BOOL FASTCALL -IntFreeMenuItem(PMENU_OBJECT MenuObject, PMENU_ITEM MenuItem, - BOOL RemoveFromList, BOOL bRecurse); - -BOOL FASTCALL -IntRemoveMenuItem(PMENU_OBJECT MenuObject, UINT uPosition, UINT uFlags, - BOOL bRecurse); - -UINT FASTCALL -IntDeleteMenuItems(PMENU_OBJECT MenuObject, BOOL bRecurse); - VOID FASTCALL UserDestroyMenu(PMENU_OBJECT MenuObject, BOOL bRecurse); -PMENU_OBJECT FASTCALL -IntCloneMenu(PMENU_OBJECT Source); - BOOL FASTCALL -UserSetMenuFlagRtoL(PMENU_OBJECT MenuObject); - -BOOL FASTCALL -IntSetMenuContextHelpId(PMENU_OBJECT MenuObject, DWORD dwContextHelpId); - -BOOL FASTCALL -UserGetMenuInfo(PMENU_OBJECT MenuObject, PROSMENUINFO lpmi); - -BOOL FASTCALL IntIsMenu(HMENU hMenu); -BOOL FASTCALL -UserSetMenuInfo(PMENU_OBJECT MenuObject, PROSMENUINFO lpmi); BOOL FASTCALL UserMenuItemInfo( @@ -492,21 +553,8 @@ BOOL Set ); -int FASTCALL -UserGetMenuItemByFlag(PMENU_OBJECT MenuObject, UINT uSearchBy, UINT fFlag, - PMENU_ITEM *MenuItem, PMENU_ITEM *PrevMenuItem); -UINT FASTCALL -IntEnableMenuItem(PMENU_OBJECT MenuObject, UINT uIDEnableItem, UINT uEnable); - -DWORD FASTCALL -IntCheckMenuItem(PMENU_OBJECT MenuObject, UINT uIDCheckItem, UINT uCheck); - BOOL FASTCALL -IntSetMenuDefaultItem(PMENU_OBJECT MenuObject, UINT uItem, UINT fByPos); - - -BOOL FASTCALL UserSetMenu( PWINDOW_OBJECT WindowObject, HMENU Menu, @@ -518,11 +566,6 @@ BOOL FASTCALL IntCleanupMenus(PW32PROCESS Win32Process); -BOOL FASTCALL -IntInsertMenuItem(PMENU_OBJECT MenuObject, UINT uItem, BOOL fByPosition, - PROSMENUITEMINFO ItemInfo); - - NTSTATUS FASTCALL InitMenuImpl(VOID); @@ -532,19 +575,19 @@ /************************* CARET.C ****************************/ BOOL FASTCALL -coUserShowCaret(PWINDOW_OBJECT Wnd); +co_UserShowCaret(PWINDOW_OBJECT Wnd); BOOL FASTCALL -coUserSetCaretPos(int X, int Y); +co_UserSetCaretPos(int X, int Y); BOOL FASTCALL -coUserHideCaret(PWINDOW_OBJECT Wnd); +co_UserHideCaret(PWINDOW_OBJECT Wnd); BOOL FASTCALL UserSwitchCaretShowing(PTHRDCARETINFO Info); BOOL FASTCALL -coUserDestroyCaret(PW32THREAD Win32Thread); +co_UserDestroyCaret(PW32THREAD Win32Thread); BOOL FASTCALL UserSetCaretBlinkTime(UINT uMSeconds); @@ -554,14 +597,35 @@ /************************* WINPOS.C ****************************/ +UINT +FASTCALL co_WinPosArrangeIconicWindows(PWINDOW_OBJECT parent); +LRESULT FASTCALL +co_WinPosGetNonClientSize(PWINDOW_OBJECT Wnd, RECT* WindowRect, RECT* ClientRect); +UINT FASTCALL +co_WinPosGetMinMaxInfo(PWINDOW_OBJECT Window, POINT* MaxSize, POINT* MaxPos, + POINT* MinTrack, POINT* MaxTrack); +UINT FASTCALL +co_WinPosMinMaximize(PWINDOW_OBJECT WindowObject, UINT ShowFlag, RECT* NewPos); +BOOLEAN FASTCALL +co_WinPosSetWindowPos(HWND Wnd, HWND WndInsertAfter, INT x, INT y, INT cx, + INT cy, UINT flags); +BOOLEAN FASTCALL +co_WinPosShowWindow(PWINDOW_OBJECT Wnd, INT Cmd); +USHORT FASTCALL +co_WinPosWindowFromPoint(PWINDOW_OBJECT ScopeWin, PUSER_MESSAGE_QUEUE OnlyHitTests, POINT *WinPoint, + PWINDOW_OBJECT* Window); +VOID FASTCALL co_WinPosActivateOtherWindow(PWINDOW_OBJECT Window); +PINTERNALPOS FASTCALL WinPosInitInternalPos(PWINDOW_OBJECT WindowObject, + POINT *pt, PRECT RestoreRect); + BOOL FASTCALL UserGetClientOrigin(PWINDOW_OBJECT hWnd, LPPOINT Point); /************************* SCROLLBAR.C ****************************/ DWORD FASTCALL -coUserShowScrollBar(PWINDOW_OBJECT Wnd, int wBar, DWORD bShow); +co_UserShowScrollBar(PWINDOW_OBJECT Wnd, int wBar, DWORD bShow); #define IntGetScrollbarInfoFromWindow(Window, i) \ ((PSCROLLBARINFO)(&((Window)->Scroll + i)->ScrollBarInfo)) @@ -572,7 +636,7 @@ #define SBOBJ_TO_SBID(Obj) ((Obj) - OBJID_HSCROLL) #define SBID_IS_VALID(id) (id == SB_HORZ || id == SB_VERT || id == SB_CTL) -BOOL FASTCALL coIntCreateScrollBars(PWINDOW_OBJECT Window); +BOOL FASTCALL co_IntCreateScrollBars(PWINDOW_OBJECT Window); BOOL FASTCALL IntDestroyScrollBars(PWINDOW_OBJECT Window); @@ -603,7 +667,7 @@ PHOOK FASTCALL UserCreateHookObject(); PHOOK FASTCALL UserGetHookObject(HHOOK hHook); -LRESULT FASTCALL coHOOK_CallHooks(INT HookId, INT Code, WPARAM wParam, LPARAM lParam); +LRESULT FASTCALL co_HOOK_CallHooks(INT HookId, INT Code, WPARAM wParam, LPARAM lParam); VOID FASTCALL HOOK_DestroyThreadHooks(PETHREAD Thread); @@ -724,7 +788,7 @@ BOOL FASTCALL MsqIsHung(PUSER_MESSAGE_QUEUE MessageQueue); NTSTATUS FASTCALL -coMsqSendMessage(PUSER_MESSAGE_QUEUE MessageQueue, +co_MsqSendMessage(PUSER_MESSAGE_QUEUE MessageQueue, HWND Wnd, UINT Msg, WPARAM wParam, LPARAM lParam, UINT uTimeout, BOOL Block, BOOL HookMessage, ULONG_PTR *uResult); @@ -738,7 +802,7 @@ VOID FASTCALL MsqPostQuitMessage(PUSER_MESSAGE_QUEUE MessageQueue, ULONG ExitCode); BOOLEAN STDCALL -coMsqFindMessage(IN PUSER_MESSAGE_QUEUE MessageQueue, +co_MsqFindMessage(IN PUSER_MESSAGE_QUEUE MessageQueue, IN BOOLEAN Hardware, IN BOOLEAN Remove, IN HWND Wnd, @@ -756,9 +820,9 @@ NTSTATUS FASTCALL MsqInitializeImpl(VOID); BOOLEAN FASTCALL -coMsqDispatchOneSentMessage(PUSER_MESSAGE_QUEUE MessageQueue); +co_MsqDispatchOneSentMessage(PUSER_MESSAGE_QUEUE MessageQueue); NTSTATUS FASTCALL -coMsqWaitForNewMessages(PUSER_MESSAGE_QUEUE MessageQueue, HWND WndFilter, +co_MsqWaitForNewMessages(PUSER_MESSAGE_QUEUE MessageQueue, HWND WndFilter, UINT MsgFilterMin, UINT MsgFilterMax); VOID FASTCALL MsqSendNotifyMessage(PUSER_MESSAGE_QUEUE MessageQueue, @@ -774,7 +838,7 @@ VOID FASTCALL -coMsqPostKeyboardMessage(UINT uMsg, WPARAM wParam, LPARAM lParam); +co_MsqPostKeyboardMessage(UINT uMsg, WPARAM wParam, LPARAM lParam); VOID FASTCALL MsqPostHotKeyMessage(PVOID Thread, HWND hWnd, WPARAM wParam, LPARAM lParam); VOID FASTCALL @@ -854,7 +918,7 @@ BOOLEAN ClipChildren, BOOLEAN ClipSiblings); VOID FASTCALL -coVIS_WindowLayoutChanged(PWINDOW_OBJECT Window, HRGN UncoveredRgn); +co_VIS_WindowLayoutChanged(PWINDOW_OBJECT Window, HRGN UncoveredRgn); /**************************** NTUSER.C ************************/ _____ Modified: branches/win32k rewrite attempt/win32k/include/win32.h --- branches/win32k rewrite attempt/win32k/include/win32.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/win32.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -28,11 +28,9 @@ typedef struct _W32PROCESS { -// FAST_MUTEX ClassListLock; PEPROCESS Process; LIST_ENTRY ClassListHead; LIST_ENTRY ExpiredTimersList; -// FAST_MUTEX MenuListLock; LIST_ENTRY MenuListHead; FAST_MUTEX PrivateFontListLock; LIST_ENTRY PrivateFontListHead; _____ Modified: branches/win32k rewrite attempt/win32k/include/window.h --- branches/win32k rewrite attempt/win32k/include/window.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/window.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -33,7 +33,6 @@ { /*---------- USER_OBJECT_HDR --------------*/ HWND hSelf; /* typesafe handle */ - LONG refs_placeholder; BYTE hdrFlags_placeholder; /*---------- USER_OBJECT_HDR --------------*/ @@ -82,8 +81,8 @@ LIST_ENTRY ThreadListEntry; /* Handle to the parent window. */ PWINDOW_OBJECT ParentWnd; - /* Handle to the owner window. */ - HWND hOwner; + /* the owner window. */ + HWND hOwnerWnd; /* DC Entries (DCE) */ PDCE Dce; @@ -103,7 +102,7 @@ PUSER_MESSAGE_QUEUE Queue; PW32THREAD WThread; }; - HWND hWndLastPopup; /* handle to last active popup window (wine doesn't use pointer, for unk. reason)*/ + HWND hLastActiveWnd; /* handle to last active popup window (wine doesn't use pointer, for unk. reason)*/ PINTERNALPOS InternalPos; // ULONG Status; /* counter for tiled child windows */ @@ -122,111 +121,8 @@ #define WINDOWOBJECT_RESTOREMAX (0x00000020) -#define HAS_DLGFRAME(Style, ExStyle) \ - (((ExStyle) & WS_EX_DLGMODALFRAME) || \ - (((Style) & WS_DLGFRAME) && (!((Style) & WS_THICKFRAME)))) -#define HAS_THICKFRAME(Style, ExStyle) \ - (((Style) & WS_THICKFRAME) && \ - (!(((Style) & (WS_DLGFRAME | WS_BORDER)) == WS_DLGFRAME))) -#define HAS_THINFRAME(Style, ExStyle) \ - (((Style) & WS_BORDER) || (!((Style) & (WS_CHILD | WS_POPUP)))) - -#define IntIsDesktopWindow(WndObj) \ - (WndObj->ParentWnd == NULL) - -#define IntIsBroadcastHwnd(hWnd) \ - (hWnd == HWND_BROADCAST || hWnd == HWND_TOPMOST) - -#if 0 -#define IntWndBelongsToThread(WndObj, W32Thread) \ - (((WndObj->W32Thread->Thread && WndObj->W32Thread->Thread->Tcb.Win32Thread)) && \ - (WndObj->W32Thread->Thread->Tcb.Win32Thread == W32Thread)) -#endif - -#define IntWndBelongsToThread(WndObj, _W32Thread) (QUEUE_2_WTHREAD((WndObj)->Queue) == (_W32Thread)) - - -#define IntGetWndThreadId(WndObj) \ - WndObj->WThread->Thread->Cid.UniqueThread - -#define IntGetWndProcessId(WndObj) \ - WndObj->WThread->Thread->ThreadsProcess->UniqueProcessId - -PWINDOW_OBJECT FASTCALL -IntGetProcessWindowObject(PW32THREAD Thread, HWND hWnd); - -BOOL FASTCALL -IntIsWindow(HWND hWnd); - -PWINDOW_OBJECT* FASTCALL -UserListChildWnd(PWINDOW_OBJECT Window); - -HWND* FASTCALL -IntWinListChildren(PWINDOW_OBJECT Window); - -NTSTATUS FASTCALL -InitWindowImpl (VOID); - -NTSTATUS FASTCALL -CleanupWindowImpl (VOID); - -VOID FASTCALL -IntGetClientRect (PWINDOW_OBJECT WindowObject, PRECT Rect); - -PWINDOW_OBJECT FASTCALL -UserGetActiveWindow(VOID); - -BOOL FASTCALL -UserIsWindowVisible (PWINDOW_OBJECT hWnd); - -BOOL FASTCALL -UserIsChildWindow (PWINDOW_OBJECT Parent, PWINDOW_OBJECT Child); - -VOID FASTCALL -IntUnlinkWindow(PWINDOW_OBJECT Wnd); - -VOID FASTCALL -IntLinkWindow(PWINDOW_OBJECT Wnd, PWINDOW_OBJECT WndParent, PWINDOW_OBJECT WndPrevSibling); - -PWINDOW_OBJECT FASTCALL -UserGetAncestor(PWINDOW_OBJECT Wnd, UINT Type); - -PWINDOW_OBJECT FASTCALL -UserGetParent(PWINDOW_OBJECT Wnd); - -PWINDOW_OBJECT FASTCALL -IntGetOwner(PWINDOW_OBJECT Wnd); - -INT FASTCALL -IntGetWindowRgn(HWND hWnd, HRGN hRgn); - -INT FASTCALL -IntGetWindowRgnBox(HWND hWnd, RECT *Rect); - -PWINDOW_OBJECT FASTCALL -UserGetShellWindow(); - -BOOL FASTCALL -IntGetWindowInfo(PWINDOW_OBJECT WindowObject, PWINDOWINFO pwi); - -VOID FASTCALL -IntGetWindowBorderMeasures(PWINDOW_OBJECT WindowObject, UINT *cx, UINT *cy); - -BOOL FASTCALL -IntAnyPopup(VOID); - -BOOL FASTCALL -IntIsWindowInDestroy(PWINDOW_OBJECT Window); - -DWORD IntRemoveWndProcHandle(WNDPROC Handle); -DWORD IntRemoveProcessWndProcHandles(HANDLE ProcessID); -DWORD IntAddWndProcHandle(WNDPROC WindowProc, BOOL IsUnicode); - -BOOL FASTCALL -coUserShowOwnedPopups( HWND owner, BOOL fShow ); - #endif /* _WIN32K_WINDOW_H */ /* EOF */ _____ Modified: branches/win32k rewrite attempt/win32k/include/winpos.h --- branches/win32k rewrite attempt/win32k/include/winpos.h 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/include/winpos.h 2005-09-02 20:45:56 UTC (rev 17607) @@ -14,26 +14,5 @@ NtGdiPtInRegion((WndObject)->WindowRegion, (INT)((x) - (WndObject)->WindowRect.left), \ (INT)((y) - (WndObject)->WindowRect.top)))) -UINT -FASTCALL coWinPosArrangeIconicWindows(PWINDOW_OBJECT parent); -LRESULT FASTCALL -coWinPosGetNonClientSize(PWINDOW_OBJECT Wnd, RECT* WindowRect, RECT* ClientRect); -UINT FASTCALL -coWinPosGetMinMaxInfo(PWINDOW_OBJECT Window, POINT* MaxSize, POINT* MaxPos, - POINT* MinTrack, POINT* MaxTrack); -UINT FASTCALL -coWinPosMinMaximize(PWINDOW_OBJECT WindowObject, UINT ShowFlag, RECT* NewPos); -BOOLEAN FASTCALL -coWinPosSetWindowPos(HWND Wnd, HWND WndInsertAfter, INT x, INT y, INT cx, - INT cy, UINT flags); -BOOLEAN FASTCALL -coWinPosShowWindow(PWINDOW_OBJECT Wnd, INT Cmd); -USHORT FASTCALL -coWinPosWindowFromPoint(PWINDOW_OBJECT ScopeWin, PUSER_MESSAGE_QUEUE OnlyHitTests, POINT *WinPoint, - PWINDOW_OBJECT* Window); -VOID FASTCALL coWinPosActivateOtherWindow(PWINDOW_OBJECT Window); -PINTERNALPOS FASTCALL WinPosInitInternalPos(PWINDOW_OBJECT WindowObject, - POINT *pt, PRECT RestoreRect); - #endif /* _WIN32K_WINPOS_H */ _____ Modified: branches/win32k rewrite attempt/win32k/ntuser/accelerator.c --- branches/win32k rewrite attempt/win32k/ntuser/accelerator.c 2005-09-02 19:05:01 UTC (rev 17606) +++ branches/win32k rewrite attempt/win32k/ntuser/accelerator.c 2005-09-02 20:45:56 UTC (rev 17607) @@ -54,6 +54,45 @@ #define NDEBUG #include <debug.h> + +/* USER OBJECT FUNCTIONS *******************************************************/ + +BOOLEAN FASTCALL UserDestroyAccel(PACCELERATOR_TABLE Accel) +{ + ASSERT_LOCK(ulExclusive); + + if (Accel->Table != NULL) + { + ExFreePool(Accel->Table); + } + + UserFreeHandle(&gHandleTable, Accel->hdr.hSelf); + UserFree(Accel); + + return TRUE; +} + + +PACCELERATOR_TABLE FASTCALL UserCreateAccelObject() +{ + PACCELERATOR_TABLE Accel; + HACCEL hAccel; + + ASSERT_LOCK(ulExclusive); + + Accel = (PACCELERATOR_TABLE)UserAllocZero(sizeof(ACCELERATOR_TABLE)); + if (!Accel) return NULL; + + hAccel = UserAllocHandle(&gHandleTable, Accel, otAccel); + if (!hAccel){ + UserFree(Accel); + return NULL; + } + Accel->hSelf=hAccel; + return Accel; +} + + /* FUNCTIONS *****************************************************************/ NTSTATUS FASTCALL @@ -71,21 +110,19 @@ int STDCALL NtUserCopyAcceleratorTable( - HACCEL Table, + HACCEL hAccel, LPACCEL Entries, int EntriesCount) { - PACCELERATOR_TABLE AcceleratorTable; + PACCELERATOR_TABLE Accel; NTSTATUS Status; int Ret; DECLARE_RETURN(int); [truncated at 1000 lines; 13185 more skipped]

1 0

[hpoussin] 17606: Don't fill twice the output buffer. Bug spotted by Filip
by hpoussin＠svn.reactos.com 02 Sep '05

02 Sep '05

Don't fill twice the output buffer. Bug spotted by Filip Modified: trunk/reactos/lib/setupapi/devinst.c _____ Modified: trunk/reactos/lib/setupapi/devinst.c --- trunk/reactos/lib/setupapi/devinst.c 2005-09-02 18:51:32 UTC (rev 17605) +++ trunk/reactos/lib/setupapi/devinst.c 2005-09-02 19:05:01 UTC (rev 17606) @@ -3873,12 +3873,6 @@ if (deviceInfo) { /* good one found */ - if (DeviceInfoData) - { - memcpy(&DeviceInfoData->ClassGuid, &deviceInfo->ClassGuid, sizeof(GUID)); - DeviceInfoData->DevInst = 0; /* FIXME */ - DeviceInfoData->Reserved = (ULONG_PTR)deviceInfo; - } ret = TRUE; } else

1 0

[hpoussin] 17605: No need to allocate too much memory, as SetupDiGetDeviceRegistryPropertyW returns a size in bytes, not in characters. Bug spotted by Filip
by hpoussin＠svn.reactos.com 02 Sep '05

02 Sep '05

No need to allocate too much memory, as SetupDiGetDeviceRegistryPropertyW returns a size in bytes, not in characters. Bug spotted by Filip Modified: trunk/reactos/lib/setupapi/devinst.c _____ Modified: trunk/reactos/lib/setupapi/devinst.c --- trunk/reactos/lib/setupapi/devinst.c 2005-09-02 13:12:44 UTC (rev 17604) +++ trunk/reactos/lib/setupapi/devinst.c 2005-09-02 18:51:32 UTC (rev 17605) @@ -3466,7 +3466,7 @@ while (!Result && GetLastError() == ERROR_INSUFFICIENT_BUFFER) { HeapFree(GetProcessHeap(), 0, HardwareIDs); - HardwareIDs = HeapAlloc(GetProcessHeap(), 0, RequiredSize * sizeof(WCHAR)); + HardwareIDs = HeapAlloc(GetProcessHeap(), 0, RequiredSize); if (!HardwareIDs) { SetLastError(ERROR_NOT_ENOUGH_MEMORY); @@ -3491,7 +3491,7 @@ while (!Result && GetLastError() == ERROR_INSUFFICIENT_BUFFER) { HeapFree(GetProcessHeap(), 0, CompatibleIDs); - CompatibleIDs = HeapAlloc(GetProcessHeap(), 0, RequiredSize * sizeof(WCHAR)); + CompatibleIDs = HeapAlloc(GetProcessHeap(), 0, RequiredSize); if (!CompatibleIDs) { SetLastError(ERROR_NOT_ENOUGH_MEMORY);

1 0

[navaraf] 17604: Check for Request == NULL.
by navaraf＠svn.reactos.com 02 Sep '05

02 Sep '05

Check for Request == NULL. Modified: trunk/reactos/ntoskrnl/lpc/reply.c _____ Modified: trunk/reactos/ntoskrnl/lpc/reply.c --- trunk/reactos/ntoskrnl/lpc/reply.c 2005-09-02 11:29:40 UTC (rev 17603) +++ trunk/reactos/ntoskrnl/lpc/reply.c 2005-09-02 13:12:44 UTC (rev 17604) @@ -263,6 +263,12 @@ Request = EiDequeueMessagePort(Port); KeReleaseSpinLock(&Port->Lock, oldIrql); + if (Request == NULL) + { + ObDereferenceObject(Port); + return STATUS_UNSUCCESSFUL; + } + if (Request->Message.u2.s2.Type == LPC_CONNECTION_REQUEST) { PORT_MESSAGE Header;

1 0

[navaraf] 17603: Fix open mode passed to CreateFile in DoSaveFile function. Patch by Johannes Anderwald.
by navaraf＠svn.reactos.com 02 Sep '05

02 Sep '05

Fix open mode passed to CreateFile in DoSaveFile function. Patch by Johannes Anderwald. Modified: trunk/reactos/subsys/system/notepad/dialog.c _____ Modified: trunk/reactos/subsys/system/notepad/dialog.c --- trunk/reactos/subsys/system/notepad/dialog.c 2005-09-01 23:32:59 UTC (rev 17602) +++ trunk/reactos/subsys/system/notepad/dialog.c 2005-09-02 11:29:40 UTC (rev 17603) @@ -143,7 +143,7 @@ DWORD size; hFile = CreateFile(Globals.szFileName, GENERIC_WRITE, FILE_SHARE_WRITE, - NULL, OPEN_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL); + NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL); if(hFile == INVALID_HANDLE_VALUE) { ShowLastError();

1 0

[blight] 17602: Link against string.a instead of hal and ntoskrnl, otherwise Windows will not load the driver.
by blight＠svn.reactos.com 01 Sep '05

01 Sep '05

Link against string.a instead of hal and ntoskrnl, otherwise Windows will not load the driver. Modified: trunk/reactos/drivers/video/displays/framebuf/framebuf.xml _____ Modified: trunk/reactos/drivers/video/displays/framebuf/framebuf.xml --- trunk/reactos/drivers/video/displays/framebuf/framebuf.xml 2005-09-01 21:09:07 UTC (rev 17601) +++ trunk/reactos/drivers/video/displays/framebuf/framebuf.xml 2005-09-01 23:32:59 UTC (rev 17602) @@ -2,9 +2,8 @@ <importlibrary definition="framebuf.def" /> <include base="framebuf">.</include> <define name="__USE_W32API" /> - <library>ntoskrnl</library> - <library>hal</library> <library>win32k</library> + <library>string</library> <file>enable.c</file> <file>palette.c</file> <file>pointer.c</file>

1 0

[gdalsnes] 17601: fixed RtlCompareUnicodeString and RtlCompareString (said stuff were equal when it was not) by ripping from wine
by gdalsnes＠svn.reactos.com 01 Sep '05

01 Sep '05

fixed RtlCompareUnicodeString and RtlCompareString (said stuff were equal when it was not) by ripping from wine Modified: trunk/reactos/lib/rtl/unicode.c _____ Modified: trunk/reactos/lib/rtl/unicode.c --- trunk/reactos/lib/rtl/unicode.c 2005-09-01 02:57:27 UTC (rev 17600) +++ trunk/reactos/lib/rtl/unicode.c 2005-09-01 21:09:07 UTC (rev 17601) @@ -237,47 +237,28 @@ LONG STDCALL RtlCompareString( - IN PSTRING String1, - IN PSTRING String2, + IN PSTRING s1, + IN PSTRING s2, IN BOOLEAN CaseInsensitive) { - ULONG len1, len2; - PCHAR s1, s2; - CHAR c1, c2; + unsigned int len; + LONG ret = 0; + LPCSTR p1, p2; - if (String1 && String2) - { - len1 = String1->Length; - len2 = String2->Length; - s1 = String1->Buffer; - s2 = String2->Buffer; + len = min(s1->Length, s2->Length); + p1 = s1->Buffer; + p2 = s2->Buffer; - if (s1 && s2) - { - if (CaseInsensitive) - { - for(;;) - { - c1 = len1-- ? RtlUpperChar (*s1++) : 0; - c2 = len2-- ? RtlUpperChar (*s2++) : 0; - if (!c1 || !c2 || c1 != c2) - return c1 - c2; - } - } - else - { - for(;;) - { - c1 = len1-- ? *s1++ : 0; - c2 = len2-- ? *s2++ : 0; - if (!c1 || !c2 || c1 != c2) - return c1 - c2; - } - } - } + if (CaseInsensitive) + { + while (!ret && len--) ret = RtlUpperChar(*p1++) - RtlUpperChar(*p2++); } - - return 0; + else + { + while (!ret && len--) ret = *p1++ - *p2++; + } + if (!ret) ret = s1->Length - s2->Length; + return ret; } @@ -290,40 +271,12 @@ BOOLEAN STDCALL RtlEqualString( - IN PSTRING String1, - IN PSTRING String2, + IN PSTRING s1, + IN PSTRING s2, IN BOOLEAN CaseInsensitive) { - ULONG i; - CHAR c1, c2; - PCHAR p1, p2; - - if (String1->Length != String2->Length) - return FALSE; - - p1 = String1->Buffer; - p2 = String2->Buffer; - for (i = 0; i < String1->Length; i++) - { - if (CaseInsensitive == TRUE) - { - c1 = RtlUpperChar (*p1); - c2 = RtlUpperChar (*p2); - } - else - { - c1 = *p1; - c2 = *p2; - } - - if (c1 != c2) - return FALSE; - - p1++; - p2++; - } - - return TRUE; + if (s1->Length != s2->Length) return FALSE; + return !RtlCompareString(s1, s2, CaseInsensitive ); } @@ -336,41 +289,12 @@ BOOLEAN STDCALL RtlEqualUnicodeString( - IN CONST UNICODE_STRING *String1, - IN CONST UNICODE_STRING *String2, + IN CONST UNICODE_STRING *s1, + IN CONST UNICODE_STRING *s2, IN BOOLEAN CaseInsensitive) { - ULONG i; - WCHAR wc1, wc2; - PWCHAR pw1, pw2; - - if (String1->Length != String2->Length) - return FALSE; - - pw1 = String1->Buffer; - pw2 = String2->Buffer; - - for (i = 0; i < String1->Length / sizeof(WCHAR); i++) - { - if (CaseInsensitive == TRUE) - { - wc1 = RtlUpcaseUnicodeChar (*pw1); - wc2 = RtlUpcaseUnicodeChar (*pw2); - } - else - { - wc1 = *pw1; - wc2 = *pw2; - } - - if (wc1 != wc2) - return FALSE; - - pw1++; - pw2++; - } - - return TRUE; + if (s1->Length != s2->Length) return FALSE; + return !RtlCompareUnicodeString((PUNICODE_STRING)s1, (PUNICODE_STRING)s2, CaseInsensitive ); } @@ -1983,58 +1907,38 @@ } - - /* * @implemented */ LONG STDCALL RtlCompareUnicodeString( - IN PUNICODE_STRING String1, - IN PUNICODE_STRING String2, + IN PUNICODE_STRING s1, + IN PUNICODE_STRING s2, IN BOOLEAN CaseInsensitive) { - ULONG len1, len2; - PWCHAR s1, s2; - WCHAR c1, c2; + unsigned int len; + LONG ret = 0; + LPCWSTR p1, p2; - if (String1 && String2) - { - len1 = String1->Length / sizeof(WCHAR); - len2 = String2->Length / sizeof(WCHAR); - s1 = String1->Buffer; - s2 = String2->Buffer; + len = min(s1->Length, s2->Length) / sizeof(WCHAR); + p1 = s1->Buffer; + p2 = s2->Buffer; - if (s1 && s2) - { - if (CaseInsensitive) - { - while (1) - { - c1 = len1-- ? RtlUpcaseUnicodeChar (*s1++) : 0; - c2 = len2-- ? RtlUpcaseUnicodeChar (*s2++) : 0; - if (!c1 || !c2 || c1 != c2) - return c1 - c2; - } - } - else - { - while (1) - { - c1 = len1-- ? *s1++ : 0; - c2 = len2-- ? *s2++ : 0; - if (!c1 || !c2 || c1 != c2) - return c1 - c2; - } - } - } + if (CaseInsensitive) + { + while (!ret && len--) ret = RtlUpcaseUnicodeChar(*p1++) - RtlUpcaseUnicodeChar(*p2++); } - - return 0; + else + { + while (!ret && len--) ret = *p1++ - *p2++; + } + if (!ret) ret = s1->Length - s2->Length; + return ret; } + /* * @implemented */

1 0

[amunger] 17600: Correct some checks, fix bug 748 (qemu networking). Fix by his vaingloriousness Alex Ionescu.
by amunger＠svn.reactos.com 01 Sep '05

01 Sep '05

Correct some checks, fix bug 748 (qemu networking). Fix by his vaingloriousness Alex Ionescu. Modified: trunk/reactos/drivers/net/ndis/ndis/miniport.c _____ Modified: trunk/reactos/drivers/net/ndis/ndis/miniport.c --- trunk/reactos/drivers/net/ndis/ndis/miniport.c 2005-08-31 14:46:04 UTC (rev 17599) +++ trunk/reactos/drivers/net/ndis/ndis/miniport.c 2005-09-01 02:57:27 UTC (rev 17600) @@ -1335,7 +1335,7 @@ Status = IoGetDeviceProperty(Adapter->NdisMiniportBlock.PhysicalDeviceObject, DevicePropertyLegacyBusType, Size, &Adapter->NdisMiniportBlock.BusType, &Size); - if (!NT_SUCCESS(Status) || Adapter->NdisMiniportBlock.BusType == (NDIS_INTERFACE_TYPE)-1) + if (!NT_SUCCESS(Status) || Adapter->NdisMiniportBlock.BusType == InterfaceTypeUndefined) { NdisInitUnicodeString(&ParamName, L"BusType"); NdisReadConfiguration(&NdisStatus, &ConfigParam, ConfigHandle, @@ -1349,7 +1349,7 @@ Status = IoGetDeviceProperty(Adapter->NdisMiniportBlock.PhysicalDeviceObject, DevicePropertyBusNumber, Size, &Adapter->NdisMiniportBlock.BusNumber, &Size); - if (!NT_SUCCESS(Status) || Adapter->NdisMiniportBlock.BusNumber == (NDIS_INTERFACE_TYPE)-1) + if (!NT_SUCCESS(Status) || Adapter->NdisMiniportBlock.BusNumber == 0xFFFFFFF0) { NdisInitUnicodeString(&ParamName, L"BusNumber"); NdisReadConfiguration(&NdisStatus, &ConfigParam, ConfigHandle,

1 0

[gdalsnes] 17599: forgot to commit thisone
by gdalsnes＠svn.reactos.com 31 Aug '05

31 Aug '05

forgot to commit thisone Added: branches/win32k rewrite attempt/win32k/include/handle.h _____ Added: branches/win32k rewrite attempt/win32k/include/handle.h --- branches/win32k rewrite attempt/win32k/include/handle.h 2005-08-30 16:28:36 UTC (rev 17598) +++ branches/win32k rewrite attempt/win32k/include/handle.h 2005-08-31 14:46:04 UTC (rev 17599) @@ -0,0 +1,41 @@ +#ifndef _WIN32K_HANDLE_H +#define _WIN32K_HANDLE_H + +#define FIRST_USER_HANDLE 0x0020 /* first possible value for low word of user handle */ +#define LAST_USER_HANDLE 0xffef /* last possible value for low word of user handle */ + + +typedef struct _USER_HANDLE_ENTRY +{ + void *ptr; /* pointer to object */ + unsigned short type; /* object type (0 if free) */ + unsigned short generation; /* generation counter */ +} USER_HANDLE_ENTRY, * PUSER_HANDLE_ENTRY; + + + +typedef struct _USER_HANDLE_TABLE +{ + PUSER_HANDLE_ENTRY handles; + PUSER_HANDLE_ENTRY freelist; + int nb_handles; + int allocated_handles; +} USER_HANDLE_TABLE, * PUSER_HANDLE_TABLE; + + + +typedef enum _USER_OBJECT_TYPE +{ + /* 0 = free */ + otWindow = 1, + otMenu, + otAccel, + otCursor, + otHook, + otMonitor + +} USER_OBJECT_TYPE; + +#endif /* _WIN32K_HANDLE_H */ + +/* EOF */

1 0

[hbirr] 17598: Print the exception code and the faulting address, if the address is available.
by hbirr＠svn.reactos.com 30 Aug '05

30 Aug '05

Print the exception code and the faulting address, if the address is available. Modified: trunk/reactos/lib/kernel32/except/except.c _____ Modified: trunk/reactos/lib/kernel32/except/except.c --- trunk/reactos/lib/kernel32/except/except.c 2005-08-30 16:26:38 UTC (rev 17597) +++ trunk/reactos/lib/kernel32/except/except.c 2005-08-30 16:28:36 UTC (rev 17598) @@ -164,8 +164,13 @@ /* Print a stack trace. */ DbgPrint("Unhandled exception\n"); - DbgPrint("Address:\n"); - DbgPrint(" %8x %s\n", + DbgPrint("ExceptionCode: %8x\n", ExceptionInfo->ExceptionRecord->ExceptionCode); + if (ExceptionInfo->ExceptionRecord->ExceptionCode == STATUS_ACCESS_VIOLATION && + ExceptionInfo->ExceptionRecord->NumberParameters == 2) + { + DbgPrint("Faulting Address: %8x\n", ExceptionInfo->ExceptionRecord->ExceptionInformation[1]); + } + DbgPrint("Address: %8x %s\n", ExceptionInfo->ExceptionRecord->ExceptionAddress, _module_name_from_addr(ExceptionInfo->ExceptionRecord->ExceptionAddress, &StartAddr, szMod, sizeof(szMod))); _dump_context ( ExceptionInfo->ContextRecord );

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs