Ros-diffs

ros-diffs@reactos.org

81390 discussions

[jimtabor] 72522: [NtUser] - Fix test results for msg:msg_queue:test_broadcast.
by jimtabor＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: jimtabor Date: Wed Aug 31 17:59:41 2016 New Revision: 72522 URL: http://svn.reactos.org/svn/reactos?rev=72522&view=rev Log: [NtUser] - Fix test results for msg:msg_queue:test_broadcast. Modified: trunk/reactos/win32ss/user/ntuser/message.c Modified: trunk/reactos/win32ss/user/ntuser/message.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/messag… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/message.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/message.c [iso-8859-1] Wed Aug 31 17:59:41 2016 @@ -554,6 +554,11 @@ } } +static BOOL is_message_broadcastable(UINT msg) +{ + return msg < WM_USER || msg >= 0xc000; +} + UINT FASTCALL GetWakeMask(UINT first, UINT last ) { @@ -1200,11 +1205,13 @@ return FALSE; } - if (Wnd == HWND_BROADCAST) + if (Wnd == HWND_BROADCAST || Wnd == HWND_TOPMOST) { HWND *List; PWND DesktopWindow; ULONG i; + + if (!is_message_broadcastable(Msg)) return TRUE; DesktopWindow = UserGetDesktopWindow(); List = IntWinListChildren(DesktopWindow); @@ -1495,6 +1502,8 @@ return co_IntSendMessageTimeoutSingle(hWnd, Msg, wParam, lParam, uFlags, uTimeout, uResult); } + if (!is_message_broadcastable(Msg)) return TRUE; + DesktopWindow = UserGetDesktopWindow(); if (NULL == DesktopWindow) { @@ -1516,26 +1525,14 @@ for (Child = Children; NULL != *Child; Child++) { - if (hWnd == HWND_TOPMOST) - { - DesktopWindow = UserGetWindowObject(*Child); - if (DesktopWindow && DesktopWindow->ExStyle & WS_EX_TOPMOST) - { - ERR("HWND_TOPMOST Found\n"); - co_IntSendMessageTimeoutSingle(*Child, Msg, wParam, lParam, uFlags, uTimeout, uResult); - } - } - else - { - PWND pwnd = UserGetWindowObject(*Child); - if (!pwnd) continue; - - if ( pwnd->fnid == FNID_MENU || - pwnd->pcls->atomClassName == gpsi->atomSysClass[ICLS_SWITCH] ) - continue; - - co_IntSendMessageTimeoutSingle(*Child, Msg, wParam, lParam, uFlags, uTimeout, uResult); - } + PWND pwnd = UserGetWindowObject(*Child); + if (!pwnd) continue; + + if ( pwnd->fnid == FNID_MENU || + pwnd->pcls->atomClassName == gpsi->atomSysClass[ICLS_SWITCH] ) + continue; + + co_IntSendMessageTimeoutSingle(*Child, Msg, wParam, lParam, uFlags, uTimeout, uResult); } ExFreePoolWithTag(Children, USERTAG_WINDOWLIST);

1 0

[jimtabor] 72521: [NtGDI] - Patch by Sebastian Gasiorek : Fix incorrect placement of the text in the Tab buttons. Implement TA_UPDATECP. See CORE-1578 and CORE-11415.
by jimtabor＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: jimtabor Date: Wed Aug 31 16:20:57 2016 New Revision: 72521 URL: http://svn.reactos.org/svn/reactos?rev=72521&view=rev Log: [NtGDI] - Patch by Sebastian Gasiorek : Fix incorrect placement of the text in the Tab buttons. Implement TA_UPDATECP. See CORE-1578 and CORE-11415. Modified: trunk/reactos/win32ss/gdi/ntgdi/freetype.c Modified: trunk/reactos/win32ss/gdi/ntgdi/freetype.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/freetype… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] Wed Aug 31 16:20:57 2016 @@ -3408,9 +3408,15 @@ IntLPtoDP(dc, (POINT *)lprc, 2); } - Start.x = XStart; - Start.y = YStart; - IntLPtoDP(dc, &Start, 1); + if(pdcattr->lTextAlign & TA_UPDATECP) + { + Start.x = pdcattr->ptlCurrent.x; + Start.y = pdcattr->ptlCurrent.y; + } else { + Start.x = XStart; + Start.y = YStart; + IntLPtoDP(dc, &Start, 1); + } RealXStart = ((LONGLONG)Start.x + dc->ptlDCOrig.x) << 6; YStart = Start.y + dc->ptlDCOrig.y; @@ -3842,6 +3848,11 @@ String++; } + + if (pdcattr->lTextAlign & TA_UPDATECP) { + pdcattr->ptlCurrent.x = DestRect.right - dc->ptlDCOrig.x; + } + IntUnLockFreeType; DC_vFinishBlit(dc, NULL) ;

1 0

[jimtabor] 72520: [User32] - MDI Sync/Port from wine head, not sure about Staging.
by jimtabor＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: jimtabor Date: Wed Aug 31 16:05:34 2016 New Revision: 72520 URL: http://svn.reactos.org/svn/reactos?rev=72520&view=rev Log: [User32] - MDI Sync/Port from wine head, not sure about Staging. Modified: trunk/reactos/win32ss/user/user32/windows/mdi.c Modified: trunk/reactos/win32ss/user/user32/windows/mdi.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/window… ============================================================================== --- trunk/reactos/win32ss/user/user32/windows/mdi.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/windows/mdi.c [iso-8859-1] Wed Aug 31 16:05:34 2016 @@ -87,11 +87,9 @@ WINE_DEFAULT_DEBUG_CHANNEL(mdi); #define MDI_MAXTITLELENGTH 0xa1 -#ifdef __REACTOS__ -#define WM_MDICALCCHILDSCROLL 0x003F // ReactOS: Uses correct value. -#else // Really? ---v win:test_mdi looks for the above value. -#define WM_MDICALCCHILDSCROLL 0x10ac /* this is exactly what Windows uses */ -#endif + +#define WM_MDICALCCHILDSCROLL 0x003F /* this is exactly what Windows uses */ + /* "More Windows..." definitions */ #define MDI_MOREWINDOWSLIMIT 9 /* after this number of windows, a "More Windows..." option will appear under the Windows menu */ @@ -405,17 +403,6 @@ return (LRESULT)oldFrameMenu; } } - else - { - /* SetMenu() may already have been called, meaning that this window - * already has its menu. But they may have done a SetMenu() on - * an MDI window, and called MDISetMenu() after the fact, meaning - * that the "if" to this "else" wouldn't catch the need to - * augment the frame menu. - */ - if( ci->hwndChildMaximized ) - MDI_AugmentFrameMenu( hwndFrame, ci->hwndChildMaximized ); - } return 0; } @@ -435,7 +422,7 @@ if (!IsMenu(ci->hWindowMenu)) { - WARN("Window menu handle %p is no more valid\n", ci->hWindowMenu); + WARN("Window menu handle %p is no longer valid\n", ci->hWindowMenu); return 0; } @@ -536,8 +523,7 @@ lpMinMax->ptMaxPosition.x = rect.left; lpMinMax->ptMaxPosition.y = rect.top; - TRACE("max rect (%ld,%ld - %ld, %ld)\n", - rect.left,rect.top,rect.right,rect.bottom); + TRACE("max rect %s\n", wine_dbgstr_rect(&rect)); } /********************************************************************** @@ -1130,9 +1116,7 @@ ci->hBmpClose = 0; NtUserSetWindowFNID( hwnd, FNID_MDICLIENT); // wine uses WIN_ISMDICLIENT #else - WND *wndPtr = WIN_GetPtr( hwnd ); - wndPtr->flags |= WIN_ISMDICLIENT; - WIN_ReleasePtr( wndPtr ); + if (message == WM_NCCREATE) win_set_flags( hwnd, WIN_ISMDICLIENT, 0 ); #endif } return unicode ? DefWindowProcW( hwnd, message, wParam, lParam ) : @@ -1342,10 +1326,7 @@ { RECT rect; - rect.left = 0; - rect.top = 0; - rect.right = LOWORD(lParam); - rect.bottom = HIWORD(lParam); + SetRect(&rect, 0, 0, LOWORD(lParam), HIWORD(lParam)); AdjustWindowRectEx(&rect, GetWindowLongPtrA(ci->hwndActiveChild, GWL_STYLE), 0, GetWindowLongPtrA(ci->hwndActiveChild, GWL_EXSTYLE) ); MoveWindow(ci->hwndActiveChild, rect.left, rect.top,

1 0

[aandrejevic] 72519: [NTVDM:BIOS] The 640x480x256 video mode has 640 bytes per scanline.
by aandrejevic＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: aandrejevic Date: Wed Aug 31 15:37:53 2016 New Revision: 72519 URL: http://svn.reactos.org/svn/reactos?rev=72519&view=rev Log: [NTVDM:BIOS] The 640x480x256 video mode has 640 bytes per scanline. Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c [iso-8859-1] Wed Aug 31 15:37:53 2016 @@ -43,7 +43,7 @@ 0xA000, /* Window A segment, or zero if not supported */ 0x0000, /* Window B segment, or zero if not supported */ 0x00000000, /* Window position function pointer */ - 800, /* Bytes per scanline */ + 640, /* Bytes per scanline */ 640, /* Width */ 480, /* Height */ 8, /* Character cell width */

1 0

[aandrejevic] 72518: [NTVDM:SVGA] Halfplement (or perhaps hackplement) SVGA packed pixel addressing.
by aandrejevic＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: aandrejevic Date: Wed Aug 31 05:38:56 2016 New Revision: 72518 URL: http://svn.reactos.org/svn/reactos?rev=72518&view=rev Log: [NTVDM:SVGA] Halfplement (or perhaps hackplement) SVGA packed pixel addressing. Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.h Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c [iso-8859-1] Wed Aug 31 05:38:56 2016 @@ -317,7 +317,12 @@ static inline DWORD VgaGetAddressSize(VOID) { - if (VgaCrtcRegisters[VGA_CRTC_UNDERLINE_REG] & VGA_CRTC_UNDERLINE_DWORD) + if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) + { + /* Packed pixel addressing */ + return 1; + } + else if (VgaCrtcRegisters[VGA_CRTC_UNDERLINE_REG] & VGA_CRTC_UNDERLINE_DWORD) { /* Double-word addressing */ return 4; // sizeof(DWORD) @@ -735,7 +740,8 @@ DWORD Address = StartAddressLatch; BYTE BytePanning = (VgaCrtcRegisters[VGA_CRTC_PRESET_ROW_SCAN_REG] >> 5) & 3; WORD LineCompare = VgaCrtcRegisters[VGA_CRTC_LINE_COMPARE_REG] - | ((VgaCrtcRegisters[VGA_CRTC_OVERFLOW_REG] & VGA_CRTC_OVERFLOW_LC8) << 4); + | ((VgaCrtcRegisters[VGA_CRTC_OVERFLOW_REG] & VGA_CRTC_OVERFLOW_LC8) << 4) + | ((VgaCrtcRegisters[VGA_CRTC_MAX_SCAN_LINE_REG] & VGA_CRTC_MAXSCANLINE_LC9) << 3); BYTE PixelShift = VgaAcRegisters[VGA_AC_HORZ_PANNING_REG] & 0x0F; /* @@ -815,99 +821,109 @@ X = j + ((PixelShift < 8) ? PixelShift : -1); } - /* Check the shifting mode */ - if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_SHIFT256) + if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) { - /* 4 bits shifted from each plane */ - - /* Check if this is 16 or 256 color mode */ - if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) + // TODO: Check for high color modes + + /* 256 color mode */ + PixelData = VgaMemory[Address + X]; + } + else + { + /* Check the shifting mode */ + if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_SHIFT256) { - /* One byte per pixel */ - PixelData = VgaMemory[WRAP_OFFSET((Address + (X / VGA_NUM_BANKS)) * AddressSize) - * VGA_NUM_BANKS + (X % VGA_NUM_BANKS)]; - } - else - { - /* 4-bits per pixel */ - - PixelData = VgaMemory[WRAP_OFFSET((Address + (X / (VGA_NUM_BANKS * 2))) * AddressSize) - * VGA_NUM_BANKS + (X % VGA_NUM_BANKS)]; - - /* Check if we should use the highest 4 bits or lowest 4 */ - if (((X / VGA_NUM_BANKS) % 2) == 0) + /* 4 bits shifted from each plane */ + + /* Check if this is 16 or 256 color mode */ + if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) { - /* Highest 4 */ - PixelData >>= 4; + /* One byte per pixel */ + PixelData = VgaMemory[WRAP_OFFSET((Address + (X / VGA_NUM_BANKS)) * AddressSize) + * VGA_NUM_BANKS + (X % VGA_NUM_BANKS)]; } else { - /* Lowest 4 */ - PixelData &= 0x0F; + /* 4-bits per pixel */ + + PixelData = VgaMemory[WRAP_OFFSET((Address + (X / (VGA_NUM_BANKS * 2))) * AddressSize) + * VGA_NUM_BANKS + (X % VGA_NUM_BANKS)]; + + /* Check if we should use the highest 4 bits or lowest 4 */ + if (((X / VGA_NUM_BANKS) % 2) == 0) + { + /* Highest 4 */ + PixelData >>= 4; + } + else + { + /* Lowest 4 */ + PixelData &= 0x0F; + } } } - } - else if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_SHIFTREG) - { - /* Check if this is 16 or 256 color mode */ - if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) + else if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_SHIFTREG) { - // TODO: NOT IMPLEMENTED - DPRINT1("8-bit interleaved mode is not implemented!\n"); - } - else - { - /* - * 2 bits shifted from plane 0 and 2 for the first 4 pixels, - * then 2 bits shifted from plane 1 and 3 for the next 4 - */ - DWORD BankNumber = (X / 4) % 2; - DWORD Offset = Address + (X / 8); - BYTE LowPlaneData = VgaMemory[WRAP_OFFSET(Offset * AddressSize) * VGA_NUM_BANKS + BankNumber]; - BYTE HighPlaneData = VgaMemory[WRAP_OFFSET(Offset * AddressSize) * VGA_NUM_BANKS + (BankNumber + 2)]; - - /* Extract the two bits from each plane */ - LowPlaneData = (LowPlaneData >> (6 - ((X % 4) * 2))) & 0x03; - HighPlaneData = (HighPlaneData >> (6 - ((X % 4) * 2))) & 0x03; - - /* Combine them into the pixel */ - PixelData = LowPlaneData | (HighPlaneData << 2); - } - } - else - { - /* 1 bit shifted from each plane */ - - /* Check if this is 16 or 256 color mode */ - if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) - { - /* 8 bits per pixel, 2 on each plane */ - - for (k = 0; k < VGA_NUM_BANKS; k++) + /* Check if this is 16 or 256 color mode */ + if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) { - /* The data is on plane k, 4 pixels per byte */ - BYTE PlaneData = VgaMemory[WRAP_OFFSET((Address + (X >> 2)) * AddressSize) * VGA_NUM_BANKS + k]; - - /* The mask of the first bit in the pair */ - BYTE BitMask = 1 << (((3 - (X % VGA_NUM_BANKS)) * 2) + 1); - - /* Bits 0, 1, 2 and 3 come from the first bit of the pair */ - if (PlaneData & BitMask) PixelData |= 1 << k; - - /* Bits 4, 5, 6 and 7 come from the second bit of the pair */ - if (PlaneData & (BitMask >> 1)) PixelData |= 1 << (k + 4); + // TODO: NOT IMPLEMENTED + DPRINT1("8-bit interleaved mode is not implemented!\n"); + } + else + { + /* + * 2 bits shifted from plane 0 and 2 for the first 4 pixels, + * then 2 bits shifted from plane 1 and 3 for the next 4 + */ + DWORD BankNumber = (X / 4) % 2; + DWORD Offset = Address + (X / 8); + BYTE LowPlaneData = VgaMemory[WRAP_OFFSET(Offset * AddressSize) * VGA_NUM_BANKS + BankNumber]; + BYTE HighPlaneData = VgaMemory[WRAP_OFFSET(Offset * AddressSize) * VGA_NUM_BANKS + (BankNumber + 2)]; + + /* Extract the two bits from each plane */ + LowPlaneData = (LowPlaneData >> (6 - ((X % 4) * 2))) & 0x03; + HighPlaneData = (HighPlaneData >> (6 - ((X % 4) * 2))) & 0x03; + + /* Combine them into the pixel */ + PixelData = LowPlaneData | (HighPlaneData << 2); } } else { - /* 4 bits per pixel, 1 on each plane */ - - for (k = 0; k < VGA_NUM_BANKS; k++) + /* 1 bit shifted from each plane */ + + /* Check if this is 16 or 256 color mode */ + if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT) { - BYTE PlaneData = VgaMemory[WRAP_OFFSET((Address + (X >> 3)) * AddressSize) * VGA_NUM_BANKS + k]; - - /* If the bit on that plane is set, set it */ - if (PlaneData & (1 << (7 - (X % 8)))) PixelData |= 1 << k; + /* 8 bits per pixel, 2 on each plane */ + + for (k = 0; k < VGA_NUM_BANKS; k++) + { + /* The data is on plane k, 4 pixels per byte */ + BYTE PlaneData = VgaMemory[WRAP_OFFSET((Address + (X >> 2)) * AddressSize) * VGA_NUM_BANKS + k]; + + /* The mask of the first bit in the pair */ + BYTE BitMask = 1 << (((3 - (X % VGA_NUM_BANKS)) * 2) + 1); + + /* Bits 0, 1, 2 and 3 come from the first bit of the pair */ + if (PlaneData & BitMask) PixelData |= 1 << k; + + /* Bits 4, 5, 6 and 7 come from the second bit of the pair */ + if (PlaneData & (BitMask >> 1)) PixelData |= 1 << (k + 4); + } + } + else + { + /* 4 bits per pixel, 1 on each plane */ + + for (k = 0; k < VGA_NUM_BANKS; k++) + { + BYTE PlaneData = VgaMemory[WRAP_OFFSET((Address + (X >> 3)) * AddressSize) * VGA_NUM_BANKS + k]; + + /* If the bit on that plane is set, set it */ + if (PlaneData & (1 << (7 - (X % 8)))) PixelData |= 1 << k; + } } } } @@ -1687,6 +1703,7 @@ /* Save the scanline size */ ScanlineSizeLatch = ((DWORD)VgaCrtcRegisters[VGA_CRTC_OFFSET_REG] + (((DWORD)VgaCrtcRegisters[SVGA_CRTC_EXT_DISPLAY_REG] & SVGA_CRTC_EXT_OFFSET_BIT8) << 4)) * 2; + if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) ScanlineSizeLatch <<= 2; /* Save the starting address */ StartAddressLatch = MAKEWORD(VgaCrtcRegisters[VGA_CRTC_START_ADDR_LOW_REG], @@ -1768,6 +1785,7 @@ /* Save the scanline size */ ScanlineSizeLatch = ((DWORD)VgaCrtcRegisters[VGA_CRTC_OFFSET_REG] + (((DWORD)VgaCrtcRegisters[SVGA_CRTC_EXT_DISPLAY_REG] & SVGA_CRTC_EXT_OFFSET_BIT8) << 4)) * 2; + if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) ScanlineSizeLatch <<= 2; /* Save the starting address */ StartAddressLatch = MAKEWORD(VgaCrtcRegisters[VGA_CRTC_START_ADDR_LOW_REG], @@ -1797,12 +1815,11 @@ { VideoAddress = VgaTranslateAddress(Address); -#if 0 - /* Packed pixel mode - not used yet */ + /* Check for packed pixel, chain-4, and odd-even mode */ if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) { /* Just copy from the video memory */ - PVOID VideoMemory = &VgaMemory[VideoAddress * VGA_NUM_BANKS + (Address & 3)]; + PVOID VideoMemory = &VgaMemory[VideoAddress + (Address & 3)]; switch (Size) { @@ -1830,10 +1847,7 @@ #endif } } -#endif - - /* Check for chain-4 and odd-even mode */ - if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) + else if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) { i = 0; @@ -1979,38 +1993,78 @@ /* Also ignore if write access to all planes is disabled */ if ((VgaSeqRegisters[VGA_SEQ_MASK_REG] & 0x0F) == 0x00) return TRUE; - /* Loop through each byte */ - for (i = 0; i < Size; i++) - { - VideoAddress = VgaTranslateAddress(Address + i); - - for (j = 0; j < VGA_NUM_BANKS; j++) - { - /* Make sure the page is writeable */ - if (!(VgaSeqRegisters[VGA_SEQ_MASK_REG] & (1 << j))) continue; - - /* Check if this is chain-4 mode */ - if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) - { - if (((Address + i) & 0x03) != j) + if (!(VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES)) + { + /* Loop through each byte */ + for (i = 0; i < Size; i++) + { + VideoAddress = VgaTranslateAddress(Address + i); + + for (j = 0; j < VGA_NUM_BANKS; j++) + { + /* Make sure the page is writeable */ + if (!(VgaSeqRegisters[VGA_SEQ_MASK_REG] & (1 << j))) continue; + + /* Check if this is chain-4 mode */ + if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) { - /* This plane will not be accessed */ - continue; + if (((Address + i) & 0x03) != j) + { + /* This plane will not be accessed */ + continue; + } } - } - - /* Check if this is odd-even mode */ - if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_OE) - { - if (((Address + i) & 0x01) != (j & 1)) + + /* Check if this is odd-even mode */ + if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_OE) { - /* This plane will not be accessed */ - continue; + if (((Address + i) & 0x01) != (j & 1)) + { + /* This plane will not be accessed */ + continue; + } } - } - - /* Copy the value to the VGA memory */ - VgaMemory[VideoAddress * VGA_NUM_BANKS + j] = VgaTranslateByteForWriting(BufPtr[i], j); + + /* Copy the value to the VGA memory */ + VgaMemory[VideoAddress * VGA_NUM_BANKS + j] = VgaTranslateByteForWriting(BufPtr[i], j); + } + } + } + else + { + PVOID VideoMemory; + + // TODO: Apply the page write mask! + // TODO: Check whether the write mode stuff applies to packed-pixel modes + + /* Just copy to the video memory */ + VideoAddress = VgaTranslateAddress(Address); + VideoMemory = &VgaMemory[VideoAddress + (Address & 3)]; + + switch (Size) + { + case sizeof(UCHAR): + *(PUCHAR)VideoMemory = *(PUCHAR)Buffer; + return TRUE; + + case sizeof(USHORT): + *(PUSHORT)VideoMemory = *(PUSHORT)Buffer; + return TRUE; + + case sizeof(ULONG): + *(PULONG)VideoMemory = *(PULONG)Buffer; + return TRUE; + + case sizeof(ULONGLONG): + *(PULONGLONG)VideoMemory = *(PULONGLONG)Buffer; + return TRUE; + + default: +#if defined(__GNUC__) + __builtin_memcpy(VideoMemory, Buffer, Size); +#else + RtlCopyMemory(VideoMemory, Buffer, Size); +#endif } } Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.h [iso-8859-1] Wed Aug 31 05:38:56 2016 @@ -182,6 +182,7 @@ /* CRTC max scanline register bits */ #define VGA_CRTC_MAXSCANLINE_DOUBLE (1 << 7) +#define VGA_CRTC_MAXSCANLINE_LC9 (1 << 6) /* CRTC mode control register bits */ #define VGA_CRTC_MODE_CONTROL_WRAP (1 << 5)

1 0

[jimtabor] 72517: [NtUser] - Fix mouse move auto selecting while tracking popup menu. See CORE-2338.
by jimtabor＠svn.reactos.org 31 Aug '16

31 Aug '16

Author: jimtabor Date: Wed Aug 31 02:24:14 2016 New Revision: 72517 URL: http://svn.reactos.org/svn/reactos?rev=72517&view=rev Log: [NtUser] - Fix mouse move auto selecting while tracking popup menu. See CORE-2338. Modified: trunk/reactos/win32ss/user/ntuser/menu.c Modified: trunk/reactos/win32ss/user/ntuser/menu.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/menu.c… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/menu.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/menu.c [iso-8859-1] Wed Aug 31 02:24:14 2016 @@ -3471,7 +3471,15 @@ if ( PtMenu ) { if (IS_SYSTEM_MENU(PtMenu)) + { Index = 0; + //// ReactOS only HACK: CORE-2338 + // Windows tracks mouse moves to the system menu but does not open it. + // Only keyboard tracking can do that. + // + TRACE("SystemMenu\n"); + return TRUE; // Stay inside the Loop! + } else MENU_FindItemByCoords( PtMenu, pmt->Pt, &Index ); }

1 0

[aandrejevic] 72516: [NTVDM:BIOS] Implement VESA function AH = 07h.
by aandrejevic＠svn.reactos.org 30 Aug '16

30 Aug '16

Author: aandrejevic Date: Tue Aug 30 22:52:49 2016 New Revision: 72516 URL: http://svn.reactos.org/svn/reactos?rev=72516&view=rev Log: [NTVDM:BIOS] Implement VESA function AH = 07h. Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vbe.c [iso-8859-1] Tue Aug 30 22:52:49 2016 @@ -653,6 +653,104 @@ break; } + /* Get/Set Display Start */ + case 0x07: + { + DWORD StartAddress; + BYTE Value; + PCVBE_MODE Mode = VbeGetModeByNumber(Bda->VideoMode); + BYTE OldCrtcIndex = IOReadB(VGA_CRTC_INDEX_COLOR); + + if (getBL() & 0x80) + { + /* Wait for a vertical retrace */ + if (!(IOReadB(VGA_INSTAT1_READ_COLOR) & VGA_STAT_VRETRACE)) + { + setCF(1); + break; + } + + setCF(0); + } + + switch (getBL() & 0x7F) + { + /* Set Display Start */ + case 0x00: + { + setAL(0x4F); + + if (Mode == NULL || Mode->Info == NULL) + { + /* This is not a VBE mode */ + // TODO: Support anyway, perhaps? It can be done. + setAH(0x01); + break; + } + + StartAddress = getCX() + getDX() * Mode->Info->BytesPerScanline; + + IOWriteB(VGA_CRTC_INDEX_COLOR, SVGA_CRTC_OVERLAY_REG); + Value = IOReadB(VGA_CRTC_DATA_COLOR); + Value &= ~SVGA_CRTC_EXT_ADDR_BIT19; + Value |= (StartAddress >> 12) & SVGA_CRTC_EXT_ADDR_BIT19; + IOWriteB(VGA_CRTC_DATA_COLOR, Value); + + IOWriteB(VGA_CRTC_INDEX_COLOR, SVGA_CRTC_EXT_DISPLAY_REG); + Value = IOReadB(VGA_CRTC_DATA_COLOR); + Value &= ~(SVGA_CRTC_EXT_ADDR_BIT16 | SVGA_CRTC_EXT_ADDR_BITS1718); + Value |= (StartAddress >> 16) & SVGA_CRTC_EXT_ADDR_BIT16; + Value |= (StartAddress >> 15) & SVGA_CRTC_EXT_ADDR_BITS1718; + IOWriteB(VGA_CRTC_DATA_COLOR, Value); + + IOWriteB(VGA_CRTC_INDEX_COLOR, VGA_CRTC_START_ADDR_HIGH_REG); + IOWriteB(VGA_CRTC_DATA_COLOR, (StartAddress >> 8) & 0xFF); + IOWriteB(VGA_CRTC_INDEX_COLOR, VGA_CRTC_START_ADDR_LOW_REG); + IOWriteB(VGA_CRTC_DATA_COLOR, StartAddress & 0xFF); + + setAH(0); + break; + } + + /* Get Display Start */ + case 0x01: + { + setAL(0x4F); + StartAddress = 0; + + if (Mode == NULL || Mode->Info == NULL) + { + /* This is not a VBE mode */ + // TODO: Support anyway, perhaps? It can be done. + setAH(0x01); + break; + } + + IOWriteB(VGA_CRTC_INDEX_COLOR, SVGA_CRTC_OVERLAY_REG); + StartAddress = (IOReadB(VGA_CRTC_DATA_COLOR) & SVGA_CRTC_EXT_ADDR_BIT19) << 12; + + IOWriteB(VGA_CRTC_INDEX_COLOR, SVGA_CRTC_EXT_DISPLAY_REG); + Value = IOReadB(VGA_CRTC_DATA_COLOR); + StartAddress |= (Value & SVGA_CRTC_EXT_ADDR_BIT16) << 16; + StartAddress |= (Value & SVGA_CRTC_EXT_ADDR_BITS1718) << 15; + + IOWriteB(VGA_CRTC_INDEX_COLOR, VGA_CRTC_START_ADDR_HIGH_REG); + StartAddress |= IOReadB(VGA_CRTC_DATA_COLOR) << 8; + IOWriteB(VGA_CRTC_INDEX_COLOR, VGA_CRTC_START_ADDR_LOW_REG); + StartAddress |= IOReadB(VGA_CRTC_DATA_COLOR); + + setCX(StartAddress % Mode->Info->BytesPerScanline); + setDX(StartAddress / Mode->Info->BytesPerScanline); + + setAH(0); + break; + } + } + + IOWriteB(VGA_CRTC_INDEX_COLOR, OldCrtcIndex); + break; + } + default: { DPRINT1("VESA BIOS Extensions function %02Xh NOT IMPLEMENTED!\n", getAL());

1 0

[dchapyshev] 72515: [WIN32K] - Stubpliment UOI_FLAGS flag in NtUserGetObjectInformation * Fixes 32 tests in user32_apitest GetUserObjectInformation
by dchapyshev＠svn.reactos.org 30 Aug '16

30 Aug '16

Author: dchapyshev Date: Tue Aug 30 21:45:39 2016 New Revision: 72515 URL: http://svn.reactos.org/svn/reactos?rev=72515&view=rev Log: [WIN32K] - Stubpliment UOI_FLAGS flag in NtUserGetObjectInformation * Fixes 32 tests in user32_apitest GetUserObjectInformation Modified: trunk/reactos/win32ss/user/ntuser/winsta.c Modified: trunk/reactos/win32ss/user/ntuser/winsta.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/winsta… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/winsta.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/winsta.c [iso-8859-1] Tue Aug 30 21:45:39 2016 @@ -722,7 +722,21 @@ switch (nIndex) { case UOI_FLAGS: - Status = STATUS_NOT_IMPLEMENTED; + nDataSize = sizeof(USEROBJECTFLAGS); + if (nLength >= nDataSize) + { + PUSEROBJECTFLAGS ObjectFlags = pvInformation; + + ObjectFlags->fInherit = 0; + ObjectFlags->fReserved = 0; + ObjectFlags->dwFlags = 0; + + Status = STATUS_SUCCESS; + } + else + { + Status = STATUS_BUFFER_TOO_SMALL; + } ERR("UOI_FLAGS unimplemented!\n"); break;

1 0

[aandrejevic] 72514: [NTVDM:SVGA] Chain-4 addressing isn't really the same as packed-pixel addressing. Also, fix a bug in the optimized odd/even mode reading code.
by aandrejevic＠svn.reactos.org 30 Aug '16

30 Aug '16

Author: aandrejevic Date: Tue Aug 30 21:07:26 2016 New Revision: 72514 URL: http://svn.reactos.org/svn/reactos?rev=72514&view=rev Log: [NTVDM:SVGA] Chain-4 addressing isn't really the same as packed-pixel addressing. Also, fix a bug in the optimized odd/even mode reading code. Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/video/svga.c [iso-8859-1] Tue Aug 30 21:07:26 2016 @@ -1797,8 +1797,9 @@ { VideoAddress = VgaTranslateAddress(Address); - /* Check for chain-4 and odd-even mode */ - if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) +#if 0 + /* Packed pixel mode - not used yet */ + if (VgaSeqRegisters[SVGA_SEQ_EXT_MODE_REG] & SVGA_SEQ_EXT_MODE_HIGH_RES) { /* Just copy from the video memory */ PVOID VideoMemory = &VgaMemory[VideoAddress * VGA_NUM_BANKS + (Address & 3)]; @@ -1829,12 +1830,65 @@ #endif } } +#endif + + /* Check for chain-4 and odd-even mode */ + if (VgaSeqRegisters[VGA_SEQ_MEM_REG] & VGA_SEQ_MEM_C4) + { + i = 0; + + /* Write the unaligned part first */ + if (Address & 3) + { + switch (Address & 3) + { + case 1: + BufPtr[i++] = VgaMemory[VideoAddress * VGA_NUM_BANKS + 1]; + case 2: + BufPtr[i++] = VgaMemory[VideoAddress * VGA_NUM_BANKS + 2]; + case 3: + BufPtr[i++] = VgaMemory[VideoAddress * VGA_NUM_BANKS + 3]; + } + + VideoAddress += 4; + } + + /* Copy the aligned dwords */ + while ((i + 3) < Size) + { + *(PULONG)&BufPtr[i] = *(PULONG)&VgaMemory[VideoAddress * VGA_NUM_BANKS]; + + i += 4; + VideoAddress += 4; + } + + /* Write the remaining part */ + if (i < Size) + { + switch (Size - i - 3) + { + case 3: + BufPtr[i] = VgaMemory[VideoAddress * VGA_NUM_BANKS + ((Address + i) & 3)]; + i++; + case 2: + BufPtr[i] = VgaMemory[VideoAddress * VGA_NUM_BANKS + ((Address + i) & 3)]; + i++; + case 1: + BufPtr[i] = VgaMemory[VideoAddress * VGA_NUM_BANKS + ((Address + i) & 3)]; + i++; + } + } + } else if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_OE) { i = 0; /* Check if the starting address is odd */ - if (Address & 1) BufPtr[i++] = VgaMemory[(VideoAddress++) * VGA_NUM_BANKS + 1]; + if (Address & 1) + { + BufPtr[i++] = VgaMemory[VideoAddress * VGA_NUM_BANKS + 1]; + VideoAddress += 2; + } while (i < (Size - 1)) {

1 0

[jimtabor] 72513: [NtUser] - Fix crash while using mouse or keyboard for menu selection in UxThmeme. - Sync/Ports and copy paste error fixes.
by jimtabor＠svn.reactos.org 30 Aug '16

30 Aug '16

Author: jimtabor Date: Tue Aug 30 19:30:45 2016 New Revision: 72513 URL: http://svn.reactos.org/svn/reactos?rev=72513&view=rev Log: [NtUser] - Fix crash while using mouse or keyboard for menu selection in UxThmeme. - Sync/Ports and copy paste error fixes. Modified: trunk/reactos/win32ss/user/ntuser/menu.c Modified: trunk/reactos/win32ss/user/ntuser/menu.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/menu.c… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/menu.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/menu.c [iso-8859-1] Tue Aug 30 19:30:45 2016 @@ -2219,7 +2219,7 @@ if (lpitem->fType & MF_SYSMENU) { - if ( (Wnd->style & WS_MINIMIZE)) + if (!(Wnd->style & WS_MINIMIZE)) { NC_GetInsideRect(Wnd, &rect); UserDrawSysMenuButton(Wnd, hdc, &rect, lpitem->fState & (MF_HILITE | MF_MOUSESELECT)); @@ -2720,7 +2720,7 @@ // No menu. Do not reserve any space return 0; } - + if (lprect == NULL) { return UserGetSystemMetrics(SM_CYMENU); @@ -2952,11 +2952,15 @@ BOOL sendMenuSelect, PMENU topmenu) { HDC hdc; - PWND pWnd = ValidateHwndNoErr(menu->hWnd); + PWND pWnd; TRACE("M_SI: owner=%p menu=%p index=0x%04x select=0x%04x\n", pwndOwner, menu, wIndex, sendMenuSelect); - if (!menu || !menu->cItems || !pWnd) return; + if (!menu || !menu->cItems) return; + + pWnd = ValidateHwndNoErr(menu->hWnd); + + if (!pWnd) return; if (menu->iItem == wIndex) return; @@ -3126,6 +3130,8 @@ PWND pWnd; TRACE("owner=%x menu=%p 0x%04x\n", WndOwner, Menu, SelectFirst); + + if (!Menu) return Menu; if (Menu->iItem == NO_SELECTED_ITEM) return Menu; @@ -3584,7 +3590,7 @@ { /* switch to the system menu */ MenuTmp = get_win_sys_menu(hNewWnd); - hNewMenu = UserHMGetHandle(MenuTmp); + if (MenuTmp) hNewMenu = UserHMGetHandle(MenuTmp); } else return FALSE; @@ -4355,6 +4361,9 @@ { co_IntSendMessage( UserHMGetHandle(pWnd), WM_INITMENUPOPUP, (WPARAM) UserHMGetHandle(menu), 0); } + + if (menu->fFlags & MNF_SYSMENU) + MENU_InitSysMenuPopup( menu, pWnd->style, pWnd->pcls->style, HTSYSMENU); if (MENU_ShowPopup(pWnd, menu, 0, wFlags, x, y, 0, 0 )) ret = MENU_TrackMenu( menu, wFlags | TPM_POPUPMENU, 0, 0, pWnd, @@ -5114,7 +5123,7 @@ ItemInfo.cbSize = sizeof(MENUITEMINFOW); ItemInfo.fMask = MIIM_FTYPE | MIIM_STRING | MIIM_STATE | MIIM_SUBMENU; - ItemInfo.fType = 0; + ItemInfo.fType = MF_POPUP; ItemInfo.fState = MFS_ENABLED; ItemInfo.dwTypeData = NULL; ItemInfo.cch = 0; @@ -5183,7 +5192,7 @@ if (OldMenu) { OldMenu->fFlags &= ~MNF_SYSMENU; - IntDestroyMenuObject(OldMenu, TRUE); + IntDestroyMenuObject(OldMenu, TRUE); } }

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