[WIN32K]
NtGdiGetDIBitsInternal: don't use the unsafe usermode pointer after the BITMAPINFO has been converted and protect writing back the result with SEH.

svn path=/branches/reactos-yarotows/; revision=47692

[WIN32K]
  - remove duplicate prototypes.
  - Add missing trail to debug print
  - replace some now superfluous checks on bitmap palette with sanity ASSERTS
  - begin using BITMAPV5HEADER as a default bitmap format, and introduce conversion to and from everything prior that.
Congratulations to BITMAPCOREHEADER, for its backward compatibility personal failure.

svn path=/branches/reactos-yarotows/; revision=47690

[WIN32K]
Introduce GreCreateBitmapEx, which has the extended functionality needed by some callers, GreCreateBitmap calls GreCreateBitmapEx to keep the simple syntax. Use it in DIB_CreateDIBSection to fix build. Also pass the size of the image (still ignored), which is needed for compressed bitmaps (we currently assume that RLEs take as much space as an uncompressed bitmap)

svn path=/branches/reactos-yarotows/; revision=47663

[WIN32K]
Rewrite the bitmap API. There were a lot of bugs. NtGdiCreateBitmap allowed a negative height, leading to either topdown or bottomup bitmaps, a behaviour that Windows doesn't have. The function copied the bitmap bits directly from the caller to the bitmap using RtlCopyMemory, ignoring different scanline length and direction (resulting in bitmaps being upside down), not SEH protected. This function (IntSetBitmapBits) is replaced by a better solution UnsafeSetBitmapBits, that takes these things into account. The name is chosen to give a hint that the function can/should be SEH protected. IntSetBitmapBits is still there, as its retarded behaviour is actually required in some places. There were also IntCreateBitmap and IntGdiCreateBitmap, now both being replaced by GreCreateBitmap. The code that set the palette is removed, as it's already done in SURFACE_AllocSurface, here gpalRGB is replaced with gpalBGR, fixing some inverted color issues. The alignment correction in SURFACE_bSetBitmapBits is reapplied, now that the callers are behaving as they are supposed to do.

svn path=/branches/reactos-yarotows/; revision=47641

[WIN32K]
Make the bitmap data for the extpens (making extpens bitmaps is a bad idea anyway) 4 bytes long instead of 3, to fulfill alignment requirements.

svn path=/branches/reactos-yarotows/; revision=47638

[WIN32K]
Disable the additional alignment code for now, it breaks selections (text / icons)

svn path=/branches/reactos-yarotows/; revision=47619

[WIN32K]
Add Windows compatible alignment code.

svn path=/branches/reactos-yarotows/; revision=47618

[WIN32K]
- Move RLE specific code to it's own file (rlecomp.c)
- Relace BitsPerFormat function with an array of UCHARs
- Rewrite surface creation. Surfaces are now allocated from one central function SURFACE_AllocSurface, which sets the size, iType, iUniq, the handle and the default palette.
- Implement SURFACE_vSetDefaultPalette, which sets the default RGB palette, based on bit depth.
- Implement SURFACE_bSetBitmapBits, wich sets cjBits, pvBits, pvScan0 and lDelta and allocates memory if neccessary.
- Use these functions for EngCreateBitmap, EngCreateDeviceBitmap, EngCreateDeviceSurface and IntCreateBitmap

svn path=/branches/reactos-yarotows/; revision=47612

[WIN32K]
Once again: Do not free the memory for the bitmap bits, when you have not allocated them.

svn path=/branches/reactos-yarotows/; revision=47611

[WIN32K]
Add another failure check to EngAllocSectionMem

svn path=/branches/reactos-yarotows/; revision=47610

[WIN32K]
Do not free the memory for the bitmap bits, when you have not allocated them. Fixes bugcheck with VBox driver.

svn path=/branches/reactos-yarotows/; revision=47609

[WIN32K]
- Allocate bitmaps as kernel sections, instead of from paged pool, like it's done in windows.
- Fix SURFACE_Cleanup. It was only freeing the memory for API_BITMAPs. If memory was allocated by a driver it never got freed.
- Add BMF_RLE_HACK flag to free decompressed RLE bits
- Support FL_ZERO_MEMORY in EngAllocSectionMem
- Set SURFOBJ::iType when creating a surface

svn path=/branches/reactos-yarotows/; revision=47606

[WIN32K]
Make the SURFACE structure a bit more like the half documented windows version.

svn path=/branches/reactos-yarotows/; revision=47603

[WIN32K]
- Simplify the logic of the mapping functions a bit, by using either FILEVIEW or ENGSECTION, not both.
- Set FILEVIEW::LastWriteTime

svn path=/branches/reactos-yarotows/; revision=47591

[WIN32K]
Implement EngCreateSection, EngMapSection, EngFreeSectionMem, EngAllocSectionMem, EngLoadModuleEx, EngLoadModule, EngLoadModuleForWrite, EngMapModule, EngFreeModule, EngMapFile, EngUnmapFile. Unused & untested.

svn path=/branches/reactos-yarotows/; revision=47583

[WIN32K] UserDrawIconEx
  - There is no need to recalculate size of the Icon, we already have this piece of information
  - Draw monochrome cursor/icons, not only their mask.

svn path=/branches/reactos-yarotows/; revision=47495

[WIN32K]
  - Simplify UserDrawIconEx
  - NtUserCreateCursorIconHandle : Get Information from bitmaps even on indirect creation and fix calculation of icon height in case of monochrome icons.
Fixes ~15 user32:cursoricon winetests.

svn path=/branches/reactos-yarotows/; revision=47451

[USER32]
  - Change ChangeDisplaySettingsExA to look more like wine's one
[WIN32K]
  - Return correct value if buffer is to small in UserChangeDisplaySettings (wine tests)
  - Do not set an invalid size to the devmode we're passing ti UserChangeDisplaySettings, we might access garbage
Now all ChangeDisplaySettings related wine tests pass.

svn path=/branches/reactos-yarotows/; revision=47386

Sync with trunk r47367

svn path=/branches/reactos-yarotows/; revision=47369

[WIN32K]
  - sync subsystems/win32/win32k.objects/cliprgn.c with trunk.

svn path=/branches/reactos-yarotows/; revision=47368

[WIN32K]
  - merge r47350, slightly modified to get it the yarotows way

svn path=/branches/reactos-yarotows/; revision=47366

revert something that should never have been commited

svn path=/branches/reactos-yarotows/; revision=47357

[WIN32K]
  - Set right flags when reading display modes from registry
  - Implement UserEnumRegistryDisplaySettings

svn path=/branches/reactos-yarotows/; revision=47356

[WIN32K]
  - Check devmode size in NtUserChangeDisplaySettings
  - Copy memory instead of setting fields in UserEnumDisplaySettings, so we don't lost anything.

svn path=/branches/reactos-yarotows/; revision=47355

[WIN32K]
  - UserChangeDisplaySettings : add some checks and fixes to what was passed to us
More wine tests pass.

svn path=/branches/reactos-yarotows/; revision=47354

[WIN32K]
  - Set DISPLAY_DEVICE_PRIMARY_DEVICE to Graphics device object when creating the primary device object
  - switch state flags of the graphics device objects when switching mode
  - Only compare valid fields when searching for a device mode in a device object
  - implement CDS_TEST flag in NtUserSetDisplaySettings

svn path=/branches/reactos-yarotows/; revision=47353

[WIN32K]
  - Initialize virtual device coordinates when creating a DC
All gdi32::mapping tests pass now

svn path=/branches/reactos-yarotows/; revision=47352

sync with trunk r47346

svn path=/branches/reactos-yarotows/; revision=47347

[USETUP]
- Polish and Czech (#5429) translations of required disk space by Maciej Bialas and Radek Liska
- Some fixes and updates to Italian and Spanish resources

svn path=/trunk/; revision=47346

[USETUP]
Make the required minimum disk space configurable from the txtsetup.sif file.

svn path=/trunk/; revision=47345

[USETUP]
Made the 'Copying file...' status line message left aligned like all the other status line messages.

svn path=/trunk/; revision=47343

[USETUP]
- Add german translation of required minimum disk space warning.

svn path=/trunk/; revision=47342

[USETUP]
- Check for required minimum disk space eventually warn the user.
- Added Italian and Spanish warnings, the other languages need translation.
- Patch by R.T.Sivakumar modified by me.
See issue #3302 for more details.

svn path=/trunk/; revision=47341

[WIN32K / WIN32CSR]
Get rid of FASTCALL specifier for static functions and functions with no parameters.

svn path=/trunk/; revision=47340

[WIN32K]
When destroying a window, generate a mouse move message, so that the underlying window is notified about the mouse position and can update the pointer if neccessary. Fixes bug #4499 and bug #3893

See issue #4499 for more details.

svn path=/trunk/; revision=47339

- Revert 47310. Please find a way to enable kmtest only when building the testing cd image, not the installation/live one.

svn path=/trunk/; revision=47338

[WIN32CSR] fix tuiconsole

svn path=/trunk/; revision=47337

[SMSS]
- Add the system environment variables PROCESSOR_LEVEL and PROCESSOR_REVISION.

svn path=/trunk/; revision=47336

[WIN32CSR]
- Make consistent use of RECT/SMALL_RECT structures: a RECT uses pixel coordinates relative to the window client area and is endpoint-exclusive; a SMALL_RECT uses character coordinates relative to the screen buffer and is endpoint-inclusive.
- Allow text selections outside of the visible window
- Implement GetConsoleSelectionInfo

svn path=/trunk/; revision=47335

[REGEDIT]
Ignore the case when sorting value names.

svn path=/trunk/; revision=47334

[SMSS]
- Remove the system environment variable OS from the hivesys*.inf files.
- Change the type of the system environment variable PATHEXT in the hivesys*.inf files from REG_EXPAND_SZ to REG_SZ.
- Let SMSS add the system environment variables OS and NUMBER_OF_PROCESSORS to the registry.

svn path=/trunk/; revision=47333

[KERNEL32], [WIN32CSR] More fixes for console winetest
- BasepInitConsole: Initialize console input EXE name
- GetConsoleProcessList: Use capture buffer; only copy IDs if buffer has enough room for all of them; return total number of processes.

svn path=/trunk/; revision=47331

[SMSS]
- Remove the system environment variable PROCESSOR_ARCHITECTURE from the hivesys*.inf files.
- Let SMSS add the system environment variables PROCESSOR_ARCHITECTURE and PROCESSOR_IDENTIFIER to the registry.

svn path=/trunk/; revision=47330

[INTL]
- Polish translation of Metric and Imperial by Olaf Siejka.

svn path=/trunk/; revision=47329

[REGEDIT]
- Don't display "finished" message if search is aborted.
Patch by Katayama Hirofumi.
See issue #5421 for more details.

svn path=/trunk/; revision=47328

Perform case insensitive comparison against the selected language id.

svn path=/trunk/; revision=47327

[KERNEL32], [WIN32CSR] Implement the CREATE_NO_WINDOW flag which creates a console with an invisible window.

svn path=/trunk/; revision=47326

[win32k]
- For the Low Level Mouse Hook (WH_MOUSE_LL), input can come from the mouse driver or mouse_event. Both of which result in a call to UserSetCursorPos.
UserMode SetCursorPos API also ends up here. Add BOOL parameter that can be used to determine if hooks are to be called.
- Move the code related to calling the hook procedure from MsqInsertSystemMessage into UserSetCursorPos and call the hook procedure here if needed.
If hook procedure returns non 0 value. Dont insert the system message.
- Fixes a recursive call to the hook procedure resulting thread using to much stack exposed by user32 winetest for input.

svn path=/trunk/; revision=47325

[WINLOGON]
- Set the APPDATA environment variable without loading shell32.dll.
This should fix bug #5398.

svn path=/trunk/; revision=47324

[kernel32_winetest]
- kernel32 tests need a resource file

svn path=/trunk/; revision=47322

[WIN32CSR] Implement FILE_SHARE_* flags for console handles. Fixes some more winetests.

svn path=/trunk/; revision=47319

[WIN32CSR] Delete even the active screen buffer when all handles are closed. Fixes a winetest.

svn path=/trunk/; revision=47318

[WIN32CSR]
- Simplify locking: having a lock for each screen buffer is overkill since most programs only use one screen buffer at a time. (besides, almost all APIs were taking the console lock anyway) Reduce to just having one lock for a console.
- Instead of keeping track of how many references a screen buffer has, keep track of handles only.  When all handles to a screen buffer are closed, it should be deleted even if it's the active buffer (not yet implemented).

svn path=/trunk/; revision=47317

[WIN32CSR] Protect ProcessData->Console with the HandleTableLock.

svn path=/trunk/; revision=47316

[WIN32CSR] Clean up the debris from r47314: Removed some redundant code, reorganized headers, fixed win32csr.rbuild indentation

svn path=/trunk/; revision=47315

[CSRSRV], [WIN32CSR]
- Move code for managing console handles from csrsrv to win32csr, where the rest of the console code is. No changes in functionality intended.
- Unify the csrsrv->win32csr callbacks (now numbering 4) into one table to avoid excessive code duplication.

svn path=/trunk/; revision=47314

[WIN32CSR] Make CsrFreeConsole close the process's console handles.

svn path=/trunk/; revision=47313

[INTL]
Translation for entries added by r47298.
Patch by Radek Liska. Fixes bug #5416.

svn path=/trunk/; revision=47312

[DEVMGR]
devman.dll improvements by Viliam Lejcik:

- display version info for selected driver file in 'Driver File Details' dialog
- resource modification - changed some labels to read-only editboxes

Fixes bug #4875.

svn path=/trunk/; revision=47311

[HIVESYS.INF]
- Enable kmtest

svn path=/trunk/; revision=47310

[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, <Test Name>TotalCount which is the total number of tests, and <Test Name>SkippedCount which is the number of tests that have been skipped
- Enjoy your reg testing! :)

svn path=/trunk/; revision=47309

[NTOSKRNL]
- Print a warning instead of crashing when a driver provides a NULL pointer in the MajorFunction array

svn path=/trunk/; revision=47308

[NTOSKRNL]
- Free the string buffer after the DPRINT1 that prints the contents of the string
- Fixes debug print corruption found by kmtest

svn path=/trunk/; revision=47307

[REACTOS.DFF]
- Add kmtest.sys and kmtestassist.sys to bootcd (optional)

svn path=/trunk/; revision=47306

[SHELL32]
- Duplicate the string instead of freeing the memory twice at two different positions
- Fixes a heap warning when stating cmd from the desktop
See issue #4924 for more details.

svn path=/trunk/; revision=47305

[WIN32K]
- Simplify EBRUSHOBJ_vInit and EBRUSHOBJ_bRealizeBrush a bit more

svn path=/branches/reactos-yarotows/; revision=47304

[INTL]
Translate 'Metric' and 'Imperial' strings into Italian and Spanish.

svn path=/trunk/; revision=47299

[INTL]
- Replace hard-coded unit strings ('Metric' and 'Imperial') by resource strings.
- Add the required resource strings to all supported languages.
- Translators: Please translate these strings.

svn path=/trunk/; revision=47298

[WIN32K]
Remove WS_EX_SETANSICREATOR from Ex style in co_IntCreateWindowEx

svn path=/trunk/; revision=47297

[USER32]
Fix uninitialized variable

svn path=/trunk/; revision=47296

[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.

svn path=/trunk/; revision=47295

[IPHLPAPI]
- Fix some code left over from a previous attempt

svn path=/trunk/; revision=47294

[DHCPCSVC]
- Remove the unused include folder

svn path=/trunk/; revision=47293

[DHCP/DHCPCSVC]
- Restore the SVN history
- Part 3 of 3 (hopefully)

svn path=/trunk/; revision=47292

[DHCP]
- Restore SVN history
- Part 2 of x

svn path=/trunk/; revision=47291

[DHCPCSVC]
- Restore SVN history
- Part 1 of x

svn path=/trunk/; revision=47290

DHCPCSVC]
- Move code from dhcp to dhcpcsvc
- Export ServiceMain from dhcpcsvc
- Now our DHCP service is compatible with the Windows architecture
[DHCP]
- Remove dhcp from build
[IPHLPAPI]
- Use dhcpcsvc APIs to control the DHCP service
- Add a missing DhcpCApiInitialize
- Fix include path in the rbuild file
[BOOTDATA]
- Add DHCP to the service list to be loaded by svchost in the netsvcs group
- Add the correct registry keys in the DHCP service key for loading by svchost
- Remove dhcp.exe from bootcd

- Part 3 of 3

svn path=/trunk/; revision=47288

[DHCP]
- Delete the old DHCP service
- Part 2 of x

svn path=/trunk/; revision=47287

[DHCP]
- Delete the old dhcp service
- Part 1 of x

svn path=/trunk/; revision=47286

[win32k]
- Move the initialization of MasterTimer into InitInputImp which is called from win32k DriverEntry routine instead of initializing it in the secondary thread RawInputThreadMain.

svn path=/trunk/; revision=47285

[win32k]
- Set MasterTimer initial value to NULL and Initialize MasterTimer at the beginning of RawInputThreadMain before doing anything else.
- Add ASSERTs to catch if MasterTimer is NULL.

svn path=/trunk/; revision=47284

[win32k]
-Remove an incorrect debug message and silence one

svn path=/trunk/; revision=47283

[WIN32CSR]
Silence a debugprint

svn path=/trunk/; revision=47282

[win32k]
-Correctly adjust values returned from co_WinPosGetMinMaxInfo

svn path=/trunk/; revision=47281

[win32k]
-Sync co_WinPosGetMinMaxInfo with wine

svn path=/trunk/; revision=47280

[WIN32K]
   Some cleanup, no code change.

svn path=/branches/reactos-yarotows/; revision=47279

[WIN32K]
Simplify EXLATEOBJ_vInitXlateFromDCs and remove EXLATEOBJ_vInitBrushXlate. Less code, more fun.

svn path=/branches/reactos-yarotows/; revision=47278

[userenv]
german translation of sendto
Patch by Egon Ashrafinia
See issue #5411 for more details.

svn path=/trunk/; revision=47277

[userenv]
- Fix Italian commonfiles
- Fix a typo and some Spanish environment variables

svn path=/trunk/; revision=47276

[win32k]
-Store the monitor work area in the monitor and not in the desktop

svn path=/trunk/; revision=47275

[win32k]
-WM_WINDOWPOSCHANGED should contain the final window position

svn path=/trunk/; revision=47273

sync with trunk (r47268)

svn path=/branches/reactos-yarotows/; revision=47271

[WIN32K]
Add back a break that got removed by accident. Fixes icon color regression.

svn path=/branches/reactos-yarotows/; revision=47270

[WIN32K]
- When initializing the EXLATEOBJ to translate the solid brush color in EBRUSHOBJ_vSetSolidBrushColor, also pass the EBRUSHOBJs back color. This fixes wrong colors when the destination surface is mono (e.g. disabled buttons)

svn path=/branches/reactos-yarotows/; revision=47269

[QMGR]
- Add the FLG_ADDREG_APPEND flag when writing the netsvcs value because, unlike WINE, we are actually going to have values in there

svn path=/trunk/; revision=47268

[SETUPAPI]
- Fix an incorrect length value that corrupted REG_MULTI_SZ values when they had strings appended
- I'm not sure if this is synced with WINE, but if it is, they need this patch too

svn path=/trunk/; revision=47267

Add SvcHost registry entries.

svn path=/trunk/; revision=47266

[REGEDIT]
- Japanese resource fix by Katayama Hirofumi
- Adopt making the abort search button default for all languages
See issue #5409 for more details.

svn path=/trunk/; revision=47265

[Win32k]
- Rename co_InflateRect to RECTL_vInflateRect and move it to rect.c
- Sync WinPosFillMinMaxInfoStruct with wine
Fixes some user32:win tests

svn path=/trunk/; revision=47264

[Kernel32]
- Implement UTF7 Support. Patch by Katayama Hirofumi.

svn path=/trunk/; revision=47263