[NDK]
Define HAL_IRQ_TRANSLATOR_VERSION constant

[NTOSKRNL]
When needed, use HAL_IRQ_TRANSLATOR_VERSION from NDK and don't define it locally

svn path=/trunk/; revision=51057

[NTDLL/LDR]
- Rewrite loader lock APIs. Now they support proper flags, return correct error codes and generate/check a cookie.

svn path=/trunk/; revision=51056

[SETUP]
- Mario Kacmar: Added 'VBR only' as possible option to the reactos.exe - it is necessary to add it to other languages (I would not mess up codepages).

See issue #5734 for more details.

svn path=/trunk/; revision=51055

[DRIVERS]
- Change () to (VOID) in function declarators. Part 5/XX.

svn path=/trunk/; revision=51054

[SHELL32]
Fix missing "Open" menu entry on right click for All Users desktop shortcuts.
Patch by Thomas Faber <thfabba at gmx dot de>

See issue #5998 for more details.

svn path=/trunk/; revision=51053

[HAL\x86]
Define a flag when building HAL with ACPI support

svn path=/trunk/; revision=51052

[NTDLL/LDR]
- Rewrite TLS part of the ntdll loader. Old TLS-related messy, duplicated and hard to maintain code is gone.
- Disclaimer: All new ldr-rewrite code is almost entirely based on Alex's patch made in 2006, with my improvements, bugfixes and adapting existing codepaths.
- File naming convention: Files of the ldr*.c mask are considered as a new rewritten loader code. All other files in ntdll/ldr directory are old code which soon may be deprecated.

svn path=/trunk/; revision=51051

[win32k]
- Partly revert r51031

svn path=/trunk/; revision=51050

[RAPPS]
- Add a "mIRC 6" entry to have a working IRC client while keeping "mIRC 7" for testing. (see bug 6005)
  Keep in mind that the automatically downloaded app database is the same for previous releases, current releases and Trunk, so just changing the version for the release wouldn't have been a solution.
- Add SumatraPDF 1.2 (working) to the app database. Patch by Vincenzo Cotugno (vins820 at hotmail dot com), German translation by me.
- Change all .txt files to svn:eol-style CRLF, these are target files which should have Windows line-endings.

svn path=/trunk/; revision=51048

[MSAFD]
- Print the caller in MsafdReturnWithErrno
- Initialize SockAsyncCompletionPort handle
- Check if the creation of the completion port failed
- Defer free of the socket information struct until the socket handle has been closed because the socket may have a pending select. When the socket is closed, the pending select is canceled, thus a apc routine is invoked which access the freed socket information struct.
- Fixes mIRC crash
- Should be merged to 0.3.13 branch

svn path=/trunk/; revision=51047

[RAPPS]
Added French translation for all the available applications. Only strings that other translators have translated are translated here.
Regarding OOo 2.4, as 2.4.3 version isn't available in French (wut?!), it has been set to 2.4.2.
If anything wrong, let me know.

svn path=/trunk/; revision=51046

[RAPPS]
Adam Stachowicz (saibamenppl@gmail.com)
- Fix mIRC version number in Rapps

See issue #5999 for more details.

svn path=/trunk/; revision=51044

Michael Martin
- Fix "EnableUserModePnpManager() failed!" in second stage setup.

Tested by Eric Kohl and igorko.
See issue #5989 for more details.

svn path=/trunk/; revision=51043

[WDMAUD.DRV]
- Correct function declaration of WdmAudOpenSoundDeviceByLegacy and remove a duplicate (wut..).  Should fix build.

svn path=/trunk/; revision=51042

[DLL]
- Change () to (VOID) in function declarators. Part 4/XX.

svn path=/trunk/; revision=51041

[BASE]
- Change () to (VOID) in function declarators. Part 3/XX.

svn path=/trunk/; revision=51039

[RAPPS]
Rafal Harabien (rafalh1992@o2.pl)
- Fix links and descriptions in the Rapps database, also update some application versions

I took the last patch in the bug report and applied the changes to remood.txt manually (due to mixed encoding within the same file).
See issue #5999 for more details.

svn path=/trunk/; revision=51034

[RAPPS]
Fix a bug while testing Rapps under Windows. If you had an application with no registered DisplayName installed, Rapps used to consider every application with no RegName in its database (= currently all Rapps applications) installed. Therefore the list of available applications has always been empty.
Now an application is always added to the ListView if no RegName is set for it.

svn path=/trunk/; revision=51033

[RAPPS]
Fix french translation file encoding

svn path=/trunk/; revision=51032

[win32]
- Rafal Harabien:
- Always call IntKeyboardInput and co_MsqPostKeyboardMessage while holding the user lock
- Simplify cleaning up sent messages
See issue #5580 for more details.

svn path=/trunk/; revision=51031

[RAPPS]
French translation by Jason Toscano <JasonGenie at gmail dot com>
Corrections/adjustements by me
See issue #5993 for more details.

svn path=/trunk/; revision=51030

[USER32]
- Rafal Harabien: Add some missing bitmaps to user32. They are copied from Wine. Change "old" cursors identifiers to defines from winuser.h (OCR_*).
See issue #6000 for more details.

svn path=/trunk/; revision=51029

[FTP]
- FTP has been a really, really naughty app.. Fix up function prototypes and definitions after () -> (void) change.
  Fixes build.. if I'm lucky.

svn path=/trunk/; revision=51028

[APPLICATIONS]
- Change all C function prototype declarations with an empty argument list to use void, or the list specified in the function definition.
  According to the C language specification, such a declaration means "no information about the number or types of the parameters is supplied".
  This can expose cases where gcc allows us to pass an excessive amount of arguments to functions in certain cases.
  Thanks to Thomas Faber for informing us about this.

svn path=/trunk/; revision=51027

[DDK]
Fix ioaccess.h svn properties. Spotted by Jonathan Yong.

svn path=/trunk/; revision=51026

[NTOSKRNL]
Revert r51017 that was reverting r51016.
In fact, it seems that cmlib assert error, whatever that eats randomly doesn't come from software issue inside ReactOS, but from a build issue.
Indeed, as you may see in the different tests I performed it shows that the same build produces the same failure, again and again. But, new build shows that there is no assert anylonger with that commit.
Don't ask me why or how it happens, I just don't get it.
But most of the time, it works, no one knows why ("as usual" (tm)). But it's cool.

Someone from BE team and/or buildslaves managers should take a serious look at this issue.

Just in case: if that commit re-hits cmlib assert, don't blame it immediatly, just commit something else, and see whether trunk is still broken.
But, as you can see in testman, we have now test results for r51016 ;-)

svn path=/trunk/; revision=51025

[EXPLORER]
Make fix of r51022 compiler independant, on Johannes' suggestion.

svn path=/trunk/; revision=51024

[win32k]
-In WH_KEYBOARD hook, lparam isn't a pointer to KBDLLHOOKSTRUCT
See issue #5991 for more details.

svn path=/trunk/; revision=51023

[EXPLORER]
Fix array access out of bounds.
See issue #5990 for more details

svn path=/trunk/; revision=51022

[Win32k]
- Move the drag detection routine to kernel space, this will decrease the number of kernel calls that is noticeable with slower systems running an emulator.

svn path=/trunk/; revision=51021

[SETUPAPI]
Implement CM_Register_Device_Driver[_Ex].

svn path=/trunk/; revision=51020

[OPENGL32]
  - use well defined macro instead of weird pointer calculation
  - reset thread specific data on a thread detach call
Fixes crash in heroes 3 and probably others

svn path=/trunk/; revision=51019

[GDI32_APITEST]
  - Test Rectangle function.
Don't laugh, ROS fails those tests and fixing this breaks the GUI.

svn path=/trunk/; revision=51018

[NTOSKRNL]
Revert r51016 that quite magically breaks Qemu test bot (Registry corruption?).
Everything was (and is still fine) on VirtualBox.
It will be put back online once the issue will have been fixed.

svn path=/trunk/; revision=51017

[NTOSKRNL]
Add most of the missing HAL stubs for both public & private dispatch tables

svn path=/trunk/; revision=51016

[NDK]
pHalAllocateMapRegisters type functions are to return NTSTATUS and not BOOLEAN

svn path=/trunk/; revision=51015

[GDI32]
   - allocate sufficient buffer when creating a DIBSECTION compatible bitmap.
Fixes bug 5822

svn path=/trunk/; revision=51014

[NOTEPAD]
- Edijs: Zero initialize the caption string.
See issue #5992 for more details.

svn path=/trunk/; revision=51013

[NTOSKRNL]
Implemented FstubTranslatorNull, FstubTranslateResource, FstubTranslateRequirement, xHalGetInterruptTranslator.
In other words: please welcome interrupt translator in ReactOS.
It's not used yet.

svn path=/trunk/; revision=51012

FatCreateCcb has no params.
Fixes building on msvc

svn path=/trunk/; revision=51011

[MSAFD]
- Fix bug introduced in 51007

svn path=/trunk/; revision=51010

[GDI32]
- Protect SetDIBitsToDevice with seh and track it. This is for vbrun60spXX. Code fix ups and cleanup.

svn path=/trunk/; revision=51009

[Win32k]
- Fix an uncovered bug. This should fix coolplayer and winamp type applications. Does not fix drawing issues.

svn path=/trunk/; revision=51008

[MSAFD]
- Correct calculating PollBufferSize
- Found by DPH

svn path=/trunk/; revision=51007

[syssetup]
- EnableUserModePnpManager:  StartServiceW can fail due to the service already running. Check last error for such a case. Should fix bug #5846.

svn path=/trunk/; revision=51006

[WIN32K]
- Fix calculation of dest rect in IntSetDIBits
- Rewrite NtGdiStretchDIBitsInternal. It now calls NtGdiSetDIBitsToDeviceInternal in case of SRCCOPY and no stretching and NtGdiCreateDIBitmapInternal + IntEngStretchBlt in the default case. This fixes RLE compressed dibs.
- The code isn't yet perfect, Windows behaviour is pretty complex, especially in regard to topdown/bottomup DIBs and the whole code needs to be rewritten. It will cause one more failed bitmap test, but another one that was formerly only passing with legacy 9x behaviour now behaves correctly.

See issue #5886 for more details.

svn path=/trunk/; revision=51005

[win32csr]
Fix zero termination of harderror string parameters, when they are ansi.

See issue #5983 for more details.

svn path=/trunk/; revision=51004

[CRT]
Patch by Thomas Faber: Fix nt version of mbstowcs.

See issue #5983 for more details.

svn path=/trunk/; revision=51003

[NTOSKRNL]
Implemented FstubFixupEfiPartition():
Purpose is to let ReactOS handle realistic partitions length in case of an EFI/GPT protective MBR.
Indeed, when formatting a disk to GPT, you add a protective MBR that contains a fake partition entry, with type 0xEE having the following property: its size is set to maximum allocatable size in MBR.
In that case, if disk is small, you may go beyond disk size.
This is why a fix to partition length may be needed when re-reading partition entry.
This is the way ReactOS creates protective MBR.
You may find some more information here: http://msdn.microsoft.com/en-us/windows/hardware/gg463525

Fixed xHalIoReadPartitionTable():
- Allocate a buffer that has the proper size (instead of allocating a buffer which is too big). Especially since this leads to partial init only some lines after.
- Remove redundant & useless affectation.
- Check/fix 0xEE partition length in case of an EFI/GPT protective MBR (see upper point).

svn path=/trunk/; revision=51002

[ROSTESTS]
Properly skip bitmap and outline tests in gdi32_winetest:font.

svn path=/trunk/; revision=51001

[WIN32K]
Fix a warning about an uninitialized variable :D

svn path=/trunk/; revision=51000

[WIN32K]
Fix IntCreateCompatibleBitmap, previously when the given DC was a memory DC with a dibsection selected, the function would only work if the dibsection was <= 8 bpp.

svn path=/trunk/; revision=50999

[WIN32K]
clip the destination rect against the bounds of the destination and source surface. Fixes a crash of gdi32_winetest bitmap that occurs as soon as IntCreateCompatibleBitmap is fixed. Other Eng* code needs to be checked as well.

svn path=/trunk/; revision=50998

[WIN32K]
Update all brushes in DC_vPrepareDCsForBlit. This allows us to simplify the code elsewhere.

svn path=/trunk/; revision=50997

[NTOSKRNL]
  - Beatify cpu supported features output once and for all
DPRINT1 is not printf...

svn path=/trunk/; revision=50996

[FASTFAT]
Do not set file attributes in case they are set to 0 or -1

svn path=/trunk/; revision=50995

[SERVICES]
Leave the critical section when the allocation of the control packet fails.

Bug spotted by hzlilx.

svn path=/trunk/; revision=50994

[NTOSKRNL]
  - shrink cpu features output.
  - Add reference for the detection of non SYSENTER aware but claiming so CPUs detection

svn path=/trunk/; revision=50993

[NTOS/FSRTL]
- Remove unnecessary upcasing, thanks Pierre.

svn path=/trunk/; revision=50992

[NTOSKRNL]
  - Print supported CPU features.
  - Simplify detection of non working but claimed SYSENTER support for intel processors
  - Workaround a virtualbox "feature" : claim that SYSENTER isn't supported despite the fact that it is.
Fix hang on registration of mshtml.dll in 2nd stage when using Virtualbox.

svn path=/trunk/; revision=50991

[SETUPAPI]
Implement CM_Query_Arbitrator_Free_Data[_Ex] and CM_Query_Arbitrator_Free_Size[_Ex].

svn path=/trunk/; revision=50990

[WIN32K]
Use GreCreateBitmapEx instead of EngCreateBitmap in NtGdiSetDIBitsToDeviceInternal, so that the RLE hack works. Fix initialization of XLATEOBJ.

svn path=/trunk/; revision=50989

[WIN32K]
Fix the RLE hack (partly)

svn path=/trunk/; revision=50988

[GDI32]
- Remove the old SetDIBBits, it severed us well.... Hold on to the win32k call.
- Tested, Area.exe, wine gdi32 bitmaps test, AbiWord 2.8.6, OOo 2.4.3, SM 2.0.11 and ReactOS applications.
- Aimp 2.61.583 (FULL, pinted okay), CoolPlayer 219, winamp 0.98d and winamp 2.95 (not FUll). The rest have drawing issue with DIB. See bug 5886.

svn path=/trunk/; revision=50987

[Win32k]
- Fix a crash in WidenPath, tested with Area.exe.

svn path=/trunk/; revision=50986

[UNIATA]
- Rafal Harabien: Sync UniATA to 0.40a5. Changelog:
* Fixed bug with BSOD on newer SATA/AHCI Intel chips.
* Added support for different number of devices on different channls.
* Updated AHCI support code (not ready yet).
* All ReactOS specific changes have been left.
See issue #5976 for more details.

svn path=/trunk/; revision=50985

Fix PNP_QueryArbitratorFreeData() and PNP_QueryArbitratorFreeSize() prototypes.

svn path=/trunk/; revision=50984

[KERNEL32]
Fixed implementation of both GetFileTime/SetFileTime

svn path=/trunk/; revision=50983

- Fix a typo, thanks Herve!

svn path=/trunk/; revision=50982

[NTOS/FSRTL]
- Implement parameters checking in FsRtlIsNameInExpressionPrivate.
- Add two shortcuts for common wildcard invocations to make the function faster.
- Second (main part of the function) is still under review.

svn path=/trunk/; revision=50981

[KERNEL32]
FlushFileBuffers:
- In spite of what MSDN pretends, it appears that FlushFileBuffers flushes console input buffer when it's called with a console handle. Then, do it here as well.
- Get rid of SetLastErrorByStatus here...

svn path=/trunk/; revision=50980

[KERNEL32]
Apply consistent formatting to FlushFileBuffers.
No code changes yet

svn path=/trunk/; revision=50979

[HEAP]
- RtlpInsertUnCommittedPages: Don't rely on linked list data in an UCR Descriptor after destroying it.
  No functionality change because the linked list data was still "as expected", but not something we want to rely on.
- RtlpCreateUnCommittedRange: Fix a typo that caused the head of UCR Segment list of the Heap to be treated as an UCR Segment header.
  Side effects of the typo: When this list wasn't empty, the (fake) UCR Segment it described was grown to contain more UCR descriptors.
  Due to the data involved, this always happened to be the initial Heap Segment, which contains the Heap Header.
  Writing the new UCR descriptors caused the Heap Header and trailing Heap Entries to be partially corrupted.

svn path=/trunk/; revision=50978

[SERVICES]
- Use one instead of two critical sections to synchronize service start and control actions.
- Fail, if an already running service is started again.

svn path=/trunk/; revision=50977

- Update Ukraianian translation by Sakara Eugene

svn path=/trunk/; revision=50976

[SETUPAPI]
Implement CM_Get_Device_Interface_List[_Ex]A/W and CM_Get_Device_Interface_List_Size[_Ex]A/W.

svn path=/trunk/; revision=50975

[EXPLORER-NEW]
  - Allow displaying seconds in tray upon registry setting
Patch by Edijs Kolesnikovics

svn path=/trunk/; revision=50974

[OPENGL32]
  - Naive try to fix bug 5057

svn path=/trunk/; revision=50973

[WIN32K]
Unlock the font in PATH_ExtTextOut. Fixes a failed assertion, when running google earth.

svn path=/trunk/; revision=50972

[WIN32K]
Silence a DPRINT

svn path=/trunk/; revision=50971

[WIN32K]
add dumping of locked handles to missing places.

svn path=/trunk/; revision=50970

[WIN32K]
Add A function to dump all locked handles and call it when an assertion about 0 locks fails.

svn path=/trunk/; revision=50969

[USETUP]
- Fix a buffer overflow (overread) when adding a locale key to the registry. The history of this bug is funny:
1. Eric wrote the code, which sets a key of REG_SZ type, as 4 widechars plus terminating zero, but passes 8 as the bytesize of the buffer. It's not fully correct (a terminating zero is absent from the bytesize of the buffer, but MSDN doesn't specify if it should be added or not, and hardcoding "8" is not the best idea too) but not dramatic. That was revision 9596, 7 years ago.
2. Lentin notices something is not right in this code, and decides to "fix" it by multiplying that same hardcoded value by.... guess what? sizeof(PWCHAR)! That is, size of a pointer, which on an x86 would be 4 bytes. Massive out of bounds access obviously happens. That was revision 31642, 3 years ago.
3. Very soon Colin reshuffles and improves the code based on patch #2635, however the problem still goes unnoticed (r31655+).
See issue #5810 for more details.

svn path=/trunk/; revision=50968

- Revert 47615. Please fix actual sysreg instead of adding inconsistency between 1st, 2nd and 3rd stages debugging connections. This should fix sysreg3's inability to do backtraces.
See issue #5811 for more details.

svn path=/trunk/; revision=50967

[RTL]
  - Fix compilation when NDEBUG is not defined

svn path=/trunk/; revision=50966

[USER32_APITEST]
Add a cursor and an icon and use those instead of system cursor. Fixes tests on Windows.

svn path=/trunk/; revision=50965

[SHELL32]
- Return the icon index, not the icon id
- Fixes bug 5742

svn path=/trunk/; revision=50964

[SETUPAPI]
Implement CM_Register_Device_Interface[_Ex]A/W and CM_Unregister_Device_Interface[_Ex]A/W.

svn path=/trunk/; revision=50963

[Win32k]
- Missed the clearing of the clip region for the window at the end of of its life. The mouse is looking for the window, doing so, scanning through all of them, even the ones that are at deaths door.

svn path=/trunk/; revision=50962

[Win32k]
- Fix the select of the clip region for a window. Apologies to Rafal Harabien select works like select, also don't
assume the original author is always right.

svn path=/trunk/; revision=50961

[WIN32K]
Acquire ownership of monitor region before deleting it.

svn path=/trunk/; revision=50960

[WIN32K]
Revert part of r50941. First its architecturally unclean to lock a DC in a USER function, 2nd its a bug to keep the lock while sending a message.

svn path=/trunk/; revision=50959

[FASTFAT]
- Prevent code duplication by using vfatAttachFCBToFileObject() for volume opening, instead of rewritting the whole function.
- Properly check requested disposition when opening a volume.
This fixes bug #5839. Trying to exec \\.\C: in explorer run dialog now ends with an error, as it does on Windows.

svn path=/trunk/; revision=50958

[SHELL32]
patch by Thomas Faber: fix a TRACE

svn path=/trunk/; revision=50957

[USER32]
Patch by Thomas Faber:
When the Edit receives WM_KILLFOCUS, it will notify its parent (the ListView)
of losing focus, which in turn will send WM_CLOSE to destroy the edit control.
This will cause the edit to receive WM_DESTROY and free the EDITSTATE.
When control returns to the WM_KILLFOCUS handler, this would call
EDIT_UnlockBuffer on the now invalid EDITSTATE.
Fix this by checking the validity of the EDITSTATE before calling EDIT_UnlockBuffer.
Fixes explorer crash, when cancelling file renaming.
See issue #5895 for more details.

svn path=/trunk/; revision=50956

[USER32]
Sync edit.c with wine head.

svn path=/trunk/; revision=50955

[RTL/PATH]
- Update path.c code to Wine 1.3.14. Fixes a buffer overwrite happening in RtlDosSearchPath_U() when invoked by "shell32_winetest.exe shlexec".
- RtlDosPathNameToNtPathName_U remains unsynced.
- Author names added to the header of the file.
See issue #5964 for more details.

svn path=/trunk/; revision=50954

[USER32]
Go back to use of SetWindowLongW instead of GetWindowLongPtrW for GWL_STYLE to reduce diff to wine code.

svn path=/trunk/; revision=50953

[USER32]
Revert a previous change that shouldn't have been comitted yet.

svn path=/trunk/; revision=50952

[FORMATTING]
user32/edit.c: Sync formatting changes from wine head, removes some unneccessary casts, no code actual change.

svn path=/trunk/; revision=50951