Ros-dev May 2010

ros-dev@reactos.org

24 participants
25 discussions

Re: [ros-dev] [ros-diffs] [tkreuzer] 47484: - add explicit braces to avoid ambiguous 'else' - fix deprecated conversion from string constant to 'char*'
by Ged Murphy 31 May '10

31 May '10

This is Wine code, it'll get lost in the next sync On 31 May 2010 05:13, <tkreuzer(a)svn.reactos.org> wrote: > Author: tkreuzer > Date: Mon May 31 06:13:40 2010 > New Revision: 47484 > > URL: http://svn.reactos.org/svn/reactos?rev=47484&view=rev > Log: > - add explicit braces to avoid ambiguous 'else' > - fix deprecated conversion from string constant to 'char*' > > Modified: > trunk/reactos/dll/win32/browseui/internettoolbar.cpp > > Modified: trunk/reactos/dll/win32/browseui/internettoolbar.cpp > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/browseui/interne… > > ============================================================================== > --- trunk/reactos/dll/win32/browseui/internettoolbar.cpp [iso-8859-1] > (original) > +++ trunk/reactos/dll/win32/browseui/internettoolbar.cpp [iso-8859-1] Mon > May 31 06:13:40 2010 > @@ -414,10 +414,16 @@ > { > SMINFO *infoPtr = (SMINFO *)lParam; > if ((infoPtr->dwMask & SMIM_FLAGS) != 0) > + { > if (psmd->uId == FCIDM_MENU_FAVORITES) > + { > infoPtr->dwFlags |= > SMIF_DROPCASCADE; > - else{ > - infoPtr->dwFlags |= > SMIF_TRACKPOPUP;} > + } > + else > + { > + infoPtr->dwFlags |= > SMIF_TRACKPOPUP; > + } > + } > if ((infoPtr->dwMask & SMIM_ICON) != 0) > infoPtr->iIcon = -1; > } > @@ -1213,7 +1219,7 @@ > CComPtr<IObjectWithSite> > objectWithSite; > CComPtr<IContextMenu> contextMenu; > CMINVOKECOMMANDINFO > commandInfo; > - char > *searchGUID = "{169A0691-8DF9-11d1-A1C4-00C04FD75D13}"; > + const char > *searchGUID = "{169A0691-8DF9-11d1-A1C4-00C04FD75D13}"; > HRESULT > hResult; > > // TODO: Query shell if this command is enabled first > > >

2 2

Re: [ros-dev] [ros-diffs] [mjmartin] 47393: [win32k] - The timer is created usingUserCreateObject. It may be a good idea to save the handle in the timer object so that it can be deleted later. - Dereference the object before attempting to delete it.
by Ged Murphy 31 May '10

31 May '10

There's no lock on the list access. On 29 May 2010 07:51, <mjmartin(a)svn.reactos.org> wrote: > Author: mjmartin > Date: Sat May 29 08:51:03 2010 > New Revision: 47393 > > URL: http://svn.reactos.org/svn/reactos?rev=47393&view=rev > Log: > [win32k] > - The timer is created usingUserCreateObject. It may be a good idea to save > the handle in the timer object so that it can be deleted later. > - Dereference the object before attempting to delete it. > > Modified: > trunk/reactos/subsystems/win32/win32k/ntuser/timer.c > > Modified: trunk/reactos/subsystems/win32/win32k/ntuser/timer.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… > > ============================================================================== > --- trunk/reactos/subsystems/win32/win32k/ntuser/timer.c [iso-8859-1] > (original) > +++ trunk/reactos/subsystems/win32/win32k/ntuser/timer.c [iso-8859-1] Sat > May 29 08:51:03 2010 > @@ -50,13 +50,21 @@ > if (!FirstpTmr) > { > FirstpTmr = UserCreateObject(gHandleTable, NULL, &Handle, otTimer, > sizeof(TIMER)); > - if (FirstpTmr) InitializeListHead(&FirstpTmr->ptmrList); > + if (FirstpTmr) > + { > + FirstpTmr->head.h = Handle; > + InitializeListHead(&FirstpTmr->ptmrList); > + } > Ret = FirstpTmr; > } > else > { > Ret = UserCreateObject(gHandleTable, NULL, &Handle, otTimer, > sizeof(TIMER)); > - if (Ret) InsertTailList(&FirstpTmr->ptmrList, &Ret->ptmrList); > + if (Ret) > + { > + Ret->head.h = Handle; > + InsertTailList(&FirstpTmr->ptmrList, &Ret->ptmrList); > + } > } > return Ret; > } > @@ -66,14 +74,17 @@ > FASTCALL > RemoveTimer(PTIMER pTmr) > { > + BOOL Ret = FALSE; > if (pTmr) > { > /* Set the flag, it will be removed when ready */ > RemoveEntryList(&pTmr->ptmrList); > - UserDeleteObject( UserHMGetHandle(pTmr), otTimer); > - return TRUE; > - } > - return FALSE; > + UserDereferenceObject(pTmr); > + Ret = UserDeleteObject( UserHMGetHandle(pTmr), otTimer); > + } > + if (!Ret) DPRINT1("Warning unable to delete timer\n"); > + > + return Ret; > } > > PTIMER > @@ -528,9 +539,7 @@ > { > if ((pTmr) && (pTmr->pti == pti) && (pTmr->pWnd == Window)) > { > - RemoveEntryList(&pTmr->ptmrList); > - UserDeleteObject( UserHMGetHandle(pTmr), otTimer); > - TimersRemoved = TRUE; > + TimersRemoved = RemoveTimer(pTmr); > } > pLE = pTmr->ptmrList.Flink; > pTmr = CONTAINING_RECORD(pLE, TIMER, ptmrList); > @@ -557,9 +566,7 @@ > { > if ((pTmr) && (pTmr->pti == pti)) > { > - RemoveEntryList(&pTmr->ptmrList); > - UserDeleteObject( UserHMGetHandle(pTmr), otTimer); > - TimersRemoved = TRUE; > + TimersRemoved = RemoveTimer(pTmr); > } > pLE = pTmr->ptmrList.Flink; > pTmr = CONTAINING_RECORD(pLE, TIMER, ptmrList); > > >

4 4

Recent Bootvid Commit
by Ros Arm 30 May '10

30 May '10

Hello! While the state of the original code is horrendous, and removing it was a step in the good direction, this simplified implementation omits many of the optimizations and specific VGA tricks that the original "author" of the code had intended to duplicate. If you'd like, I would be open to talking to eVb (we both have some experience with VGA programming) to create an efficient 4BitBlt routine similar to the original one that has been removed, but one that actually has basis in actual fact and isn't of the nature this original implementation was. I can see the original code was converting packed data into per-plane (planar) data and had special edge handling, but it looks like the original author did not realize this, and most of the modulo-and-shifts appear as bitwise ANDs (compiler optimizations), which is just plain confusing. Since the rest of bootvid is probably the same "quality", we could take a good look at it, both from a coding and legal standpoint, if you'd like. -r Author: gschneider Date: Sun May 30 01:54:47 2010 New Revision: 47431 URL: http://svn.reactos.org/svn/reactos?rev=47431&view=rev Log: [BOOTVID] Dramatically simplify 4bpp blitting routine See issue #5103 for more details. Modified: trunk/reactos/drivers/base/bootvid/i386/vga.c Modified: trunk/reactos/drivers/base/bootvid/i386/vga.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/base/bootvid/i386/… ============================================================================== --- trunk/reactos/drivers/base/bootvid/i386/vga.c [iso-8859-1] (original) +++ trunk/reactos/drivers/base/bootvid/i386/vga.c [iso-8859-1] Sun May 30 01:54:47 2010 @@ -402,33 +402,20 @@ IN ULONG BitsPerPixel, IN ULONG Delta) { - ULONG LeftAnd, LeftShifted, LeftPlusOne, LeftPos; - ULONG lMask, rMask; - UCHAR NotlMask; - ULONG Distance; - ULONG DistanceMinusLeftBpp; - ULONG SomeYesNoFlag, SomeYesNoFlag2; - PUCHAR PixelPosition, m; - PUCHAR i, k; - ULONG j; - ULONG x; - ULONG Plane; - UCHAR LeftArray[84]; - PUCHAR CurrentLeft; - PUCHAR l; - ULONG LoopCount; - UCHAR pMask, PlaneShift; - BOOLEAN Odd; - UCHAR Value; + ULONG sx, dx, dy; + UCHAR color; + ULONG offset = 0; + const ULONG Bottom = Top + Height; + const ULONG Right = Left + Width; /* Check if the buffer isn't 4bpp */ if (BitsPerPixel != 4) { /* FIXME: TODO */ DbgPrint("Unhandled BitBlt\n" - "%lxx%lx @ (%lx,%lx)\n" - "Bits Per Pixel %lx\n" - "Buffer: %p. Delta: %lx\n", + "%lux%lu @ (%lu|%lu)\n" + "Bits Per Pixel %lu\n" + "Buffer: %p. Delta: %lu\n", Width, Height, Left, @@ -439,181 +426,28 @@ return; } - /* Get the masks and other values */ - LeftAnd = Left & 0x7; - lMask = lMaskTable[LeftAnd]; - Distance = Width + Left; - rMask = rMaskTable[(Distance - 1) & 0x7]; - Left >>= 3; - - /* Set some values */ - SomeYesNoFlag = FALSE; - SomeYesNoFlag2 = FALSE; - Distance = (Distance - 1) >> 3; - DistanceMinusLeftBpp = Distance - Left; - - /* Check if the distance is equal to the left position and add the masks */ - if (Left == Distance) lMask += rMask; - - /* Check if there's no distance offset */ - if (DistanceMinusLeftBpp) - { - /* Set the first flag on */ - SomeYesNoFlag = TRUE; - - /* Decrease offset and check if we still have one */ - if (--DistanceMinusLeftBpp) - { - /* Still have a distance offset */ - SomeYesNoFlag2 = TRUE; - } - } - - /* Calculate initial pixel position */ - PixelPosition = (PUCHAR)VgaBase + (Top * 80) + Left; - - /* Set loop buffer variable */ - i = Buffer; - - /* Switch to mode 0 */ - ReadWriteMode(0); - - /* Leave now if the height is 0 */ - if (Height <= 0) return; - - /* Set more weird values */ - CurrentLeft = &LeftArray[Left]; - NotlMask = ~(UCHAR)lMask; - LeftPlusOne = Left + 1; - LeftShifted = (lMask << 8) | 8; - j = Height; - - /* Start the height loop */ + /* 4bpp blitting */ + dy = Top; do { - /* Start the plane loop */ - Plane = 0; + sx = 0; do { - /* Clear the current value */ - *CurrentLeft = 0; - LoopCount = 0; - - /* Set the buffer loop variable for this loop */ - k = i; - - /* Calculate plane shift and pixel mask */ - PlaneShift = 1 << Plane; - pMask = PixelMask[LeftAnd]; - - /* Check if we have a width */ - if (Width > 0) - { - /* Loop it */ - l = CurrentLeft; - x = Width; - do - { - /* Check if we're odd and increase the loop count */ - Odd = LoopCount & 1 ? TRUE : FALSE; - LoopCount++; - if (Odd) - { - /* Check for the plane shift */ - if (*k & PlaneShift) - { - /* Write the pixel mask */ - *l |= pMask; - } - - /* Increase buffer position */ - k++; - } - else - { - /* Check for plane shift */ - if ((*k >> 4) & PlaneShift) - { - /* Write the pixel mask */ - *l |= pMask; - } - } - - /* Shift the pixel mask */ - pMask >>= 1; - if (!pMask) - { - /* Move to the next current left position and clear it */ - l++; - *l = 0; - - /* Set the pixel mask to 0x80 */ - pMask = 0x80; - } - } while (--x); - } - - /* Set the plane value */ - __outpw(0x3C4, (1 << (Plane + 8) | 2)); - - /* Select the bitmask register and write the mask */ - __outpw(0x3CE, (USHORT)LeftShifted); - - /* Read the current Pixel value */ - Value = READ_REGISTER_UCHAR(PixelPosition); - - /* Add our mask */ - Value = (Value & NotlMask) | *CurrentLeft; - - /* Set current left for the loop, and write new pixel value */ - LeftPos = LeftPlusOne; - WRITE_REGISTER_UCHAR(PixelPosition, Value); - - /* Set loop pixel position and check if we should loop */ - m = PixelPosition + 1; - if (SomeYesNoFlag2) - { - /* Set the bitmask to 0xFF for all 4 planes */ - __outpw(0x3CE, 0xFF08); - - /* Check if we have any distance left */ - if (DistanceMinusLeftBpp > 0) - { - /* Start looping it */ - x = DistanceMinusLeftBpp; - do - { - /* Write the value */ - WRITE_REGISTER_UCHAR(m, LeftArray[LeftPos]); - - /* Go to the next position */ - m++; - LeftPos++; - } while (--x); - } - } - - /* Check if the first flag is on */ - if (SomeYesNoFlag) - { - /* Set the mask value */ - __outpw(0x3CE, (rMask << 8) | 8); - - /* Read the current Pixel value */ - Value = READ_REGISTER_UCHAR(m); - - /* Add our mask */ - Value = (Value & ~(UCHAR)rMask) | LeftArray[LeftPos]; - - /* Set current left for the loop, and write new pixel value */ - WRITE_REGISTER_UCHAR(m, Value); - } - } while (++Plane < 4); - - /* Update pixel position, buffer and height */ - PixelPosition += 80; - i += Delta; - } while (--j); + /* Extract color */ + color = Buffer[offset + sx]; + + /* Calc destination x */ + dx = Left + (sx << 1); + + /* Set two pixels */ + SetPixel(dx, dy, color >> 4); + SetPixel(dx + 1, dy, color & 0x0F); + + sx++; + } while (dx < Right); + offset += Delta; + dy++; + } while (dy < Bottom); } VOID

2 1

Translations of usrmgr cpl
by Eric Kohl 26 May '10

26 May '10

Hi! It is really funny to see that translators don't seem to understand the simple meaning of: /* * Attention Translators: * DO NOT TRANSLATE THESE RESOURCES YET! */ It means: DO NOT TRANSLATE THESE RESOURCES YET!!!!!!! Got it??? Yes, but why? Because usrmgr is not finished yet from a programmers point of view. Most of the underlying infrastructure ist entirely missing. Right now, usrmgr is completely useless and its look and feel might change without prior notice. So please don't waste your precious time by adding more translations! Please translate other missing texts instead, but leave usrmgr alone. At least as long as the little note at the top is still in place. Thank you very much for your attention! Regards, Eric

2 1

"- Setup can not handle more than one primary partition per disk."
by Javier Agustìn Fernàndez Arroyo 25 May '10

25 May '10

hello, isnt this string at usetup obsolete? AFAIK usetup does allow more partitions from a long time now....

2 1

Kernel Question
by ahmet alper parker 23 May '10

23 May '10

Hi all, I am new to this mail group and have a question that probably needs your kind of technical knowledge to answer. I want to use reactos, however, up to its completion, can I use some libraries on windows to replace them? In example, wine libraries can be superseded by original windows libraries if you own them. I want to make the inverse of this simply for security reasons to use a kernel fully opensource and which have a documented api. So can I use ntkrnlos.exe and hal.dll or similar in original windows installations to replace propriety kernel and libraries for security reasons? If yes, which of them should I use, and are they complete to replace them? Also a second question may arise as "the priority of reactos development can be focused on firstly these parts of the project or not?"... Best Regards... Ahmet Alper Parker

5 12

Re: [ros-dev] [ros-diffs] [tkreuzer] 47295: [WIN32K / USER32] Convert the window text string from UNICODE_STRING to LARGE_STRING and fix NtUserCreateWindowEx parameters. We currently still pass UNICODE only LARGE_STRINGs, as the rest of the code in w
by James Tabor 23 May '10

23 May '10

Hi! That is for bAnsi:1 to set if ansi. Good job over all! James On Sat, May 22, 2010 at 7:07 AM, Timo Kreuzer <timo.kreuzer(a)web.de> wrote: > Aleksey Bragin wrote: >> Shouldn't the LARGE_UNICODE_STRING be used there instead? >> > On Windows the window name is ANSI for an ANSI caller and UNICODE for a > UNICODE caller. > The class name is always in UNICODE though. > > See also http://www.reactos.org/wiki/Techwiki:Win32k/LARGE_UNICODE_STRING > PS: Still on sick leave.

2 1

Re: [ros-dev] [ros-diffs] [cgutman] 47309: [KMTEST] - Add support for recovering from crashed tests - Add check to prevent us from running the test every boot - Delete some useless code - Record test result information in the registry - Under the Kmtest\Parameters key, you will find CurrentStage which is the stage that testing is on (almost always 8 if it boots). You will also find <Test Name>SuccessCount which is the number of successful tests, <Test Name>FailureCount which is the number of failed tests, <
by Colin Finck 23 May '10

23 May '10

cgutman(a)svn.reactos.org wrote: > + DbgPrint("%S: %d test executed (0 marked as todo, %d failures), > %d skipped.\n", TestName, total, failures, skipped); Nice! Though testman won't parse this if you don't change "test" to "tests" ;-) Best regards, Colin

1 0

22 May '10

Shouldn't the LARGE_UNICODE_STRING be used there instead? WBR, Aleksey. On May 22, 2010, at 5:05 AM, tkreuzer(a)svn.reactos.org wrote: > Author: tkreuzer > Date: Sat May 22 03:05:31 2010 > New Revision: 47295 > > URL: http://svn.reactos.org/svn/reactos?rev=47295&view=rev > Log: > [WIN32K / USER32] > Convert the window text string from UNICODE_STRING to LARGE_STRING > and fix NtUserCreateWindowEx parameters. We currently still pass > UNICODE only LARGE_STRINGs, as the rest of the code in win32k > expects this. > Fixes display of large text windows, like the winzip license. > See issue #2900 for more details. > > Modified: > trunk/reactos/dll/win32/user32/windows/window.c > trunk/reactos/include/reactos/win32k/ntuser.h > trunk/reactos/subsystems/win32/win32k/include/window.h > trunk/reactos/subsystems/win32/win32k/ntuser/desktop.c > trunk/reactos/subsystems/win32/win32k/ntuser/message.c > trunk/reactos/subsystems/win32/win32k/ntuser/painting.c > trunk/reactos/subsystems/win32/win32k/ntuser/window.c

2 1

Re: [ros-dev] [ros-diffs] [cgutman] 47286: [DHCP] - Delete the old dhcp service - Part 1 of x
by Aleksey Bragin 21 May '10

21 May '10

I'm afraid you just deleted all dhcp development history. Please revert these 3 commits and redo them properly using svn copy + svn delete. WBR, Aleksey Bragin. On May 21, 2010, at 9:44 AM, cgutman(a)svn.reactos.org wrote: > Author: cgutman > Date: Fri May 21 07:44:35 2010 > New Revision: 47286 > > URL: http://svn.reactos.org/svn/reactos?rev=47286&view=rev > Log: > [DHCP] > - Delete the old dhcp service > - Part 1 of x > > Removed: > trunk/reactos/base/services/dhcp/adapter.c > trunk/reactos/base/services/dhcp/alloc.c > trunk/reactos/base/services/dhcp/api.c > trunk/reactos/base/services/dhcp/compat.c > trunk/reactos/base/services/dhcp/design.txt > trunk/reactos/base/services/dhcp/dhclient.c > trunk/reactos/base/services/dhcp/dhcp.rbuild > trunk/reactos/base/services/dhcp/dhcp.rc > trunk/reactos/base/services/dhcp/dhcpmain.c > trunk/reactos/base/services/dhcp/dispatch.c > trunk/reactos/base/services/dhcp/hash.c > trunk/reactos/base/services/dhcp/include/cdefs.h > trunk/reactos/base/services/dhcp/include/debug.h > trunk/reactos/base/services/dhcp/include/dhcp.h > trunk/reactos/base/services/dhcp/include/dhcpd.h > trunk/reactos/base/services/dhcp/include/dhctoken.h > trunk/reactos/base/services/dhcp/include/hash.h > trunk/reactos/base/services/dhcp/include/inet.h > trunk/reactos/base/services/dhcp/include/osdep.h > trunk/reactos/base/services/dhcp/include/predec.h > trunk/reactos/base/services/dhcp/include/privsep.h > trunk/reactos/base/services/dhcp/include/rosdhcp.h > trunk/reactos/base/services/dhcp/include/site.h > trunk/reactos/base/services/dhcp/include/stdint.h > trunk/reactos/base/services/dhcp/include/sysconf.h > trunk/reactos/base/services/dhcp/include/tree.h > trunk/reactos/base/services/dhcp/include/version.h > trunk/reactos/base/services/dhcp/memory.c > trunk/reactos/base/services/dhcp/options.c > trunk/reactos/base/services/dhcp/pipe.c > trunk/reactos/base/services/dhcp/privsep.c > trunk/reactos/base/services/dhcp/socket.c > trunk/reactos/base/services/dhcp/tables.c > trunk/reactos/base/services/dhcp/timer.c > trunk/reactos/base/services/dhcp/tree.c > trunk/reactos/base/services/dhcp/util.c

4 3

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev May 2010