[KMTESTS:SE]
- Remove broken prototypes

svn path=/trunk/; revision=59546

[KMTEST]
- Fix platform includes

svn path=/trunk/; revision=59545

[KMTESTS:RTL]
- Add test for REG_MULTI_SZ in RtlQueryRegistryValues

svn path=/trunk/; revision=59544

[browseui.cpp]
- Improve the hack for the global constructors by manually calling them only during DLL_PROCESS_ATTACH

svn path=/trunk/; revision=59543

[NTOS]
- Use more pool tagging

svn path=/trunk/; revision=59542

[shell32]
- Add the accelerators table that will be used for the shell view control
- Rewrite CDefView::OpenSelectedItems and CDefView::OnContextMenu.
- Rewrite accelerators handling for the shell view. Instead of handling them manually in CDefView::OnNotify, we now translate it properly in CDefView::TranslateAccelerator and forward the result WM_COMMAND message to the IContextMenu2 using a new method called CDefView::OnExplorerCommand.
- Now we don't do call ShellExecute or use clipboard function right in CDefView and this also is the first step to properly handle opening or exploring a folder (which is needed for explorer_new)

svn path=/trunk/; revision=59541

[EXPLORER]
- Don't compile FAT support code (WTF) if it's not used

svn path=/trunk/; revision=59540

[PSDK] Fix broken (DBG_)UNREFERENCED_LOCAL_VARIABLE macros

svn path=/trunk/; revision=59539

[WIN32K] Modify DBG_LOGEVENT to reference the parameter variable in non-logging mode, bring back cRefs variable in GDIOBJ_vReferenceObjectByPointer

svn path=/trunk/; revision=59538

[SPEC2DEF]
- Fix warnings

svn path=/trunk/; revision=59537

[CMAKE]
- Fix MSVC 64 bit host-tools build

svn path=/trunk/; revision=59536

[PORTCLS]
- Silence warnings. Using (unnecessary) virtual destructors here rather than final because that's what the 99 other classes already do.

svn path=/trunk/; revision=59535

[CMAKE]
- And here's finally a reason why we shouldn't use g++'s host-headers and libraries
- Switch back to wishy-washy-what's-a-standard mode for C++ globally, and enable C++11 only for netshell

svn path=/trunk/; revision=59534

[CMAKE]
- Why the heck do we need GNU C++ extensions aka fix build

svn path=/trunk/; revision=59533

[NETSHELL]
- Fix warnings

svn path=/trunk/; revision=59532

[PSDK]
- Fix PRINTPROVIDOR::fpAddPortEx prototype

svn path=/trunk/; revision=59531

[REGEDIT]
Fix the creation of empty REG_MULTI_SZ values (they have to contain only one NULL char, since a multi-string has the form:
str1\0str2\0...strN\0\0
where each strI\0 is a null-terminated string, and a multi-string is terminated by a null string.
And therefore an empty multi-string has only one null char, corresponding to the terminating null string.

svn path=/trunk/; revision=59529

[USBEHCI]: Straggler fix.

svn path=/trunk/; revision=59528

[MISC]: Cleanup a bunch of unused variables, in cases where it was obvious these variables were left-overs from debug code, debug prints, or in some cases commented-out functionality (in which case the variables were commented out instead of removed). If you own ACLUI, CLASS2, USB, USER32, or EXT2LIB you may want to check that these changes are accurate (i.e.: such as the last commit where the variable was actually not being used due to a bug).

svn path=/trunk/; revision=59527

[USBCCGP]: FDO_HandleResetCyclePort should complete the pending IRPs, not the reset IRP. In fact, since the caller (FDO_HandleInternalDeviceControl) already completes the reset IRP, this was also causing double IRP completion. THIS IS WHY GCC 4.7 WARNINGS ARE IMPORTANT.

svn path=/trunk/; revision=59526

[FBTUSB]
* Silence some warnings.

svn path=/trunk/; revision=59525

[WINMM_WINETEST]
* Silence some warnings.

svn path=/trunk/; revision=59524

[GLU32]
* Silence some warnings.

svn path=/trunk/; revision=59523

[UNIATA]
* Silence some warnings.

svn path=/trunk/; revision=59522

[LIBXML2]
* Disable warnings.

svn path=/trunk/; revision=59521

[CARDLIB]
* Silence some warnings.

svn path=/trunk/; revision=59520

[ADNS]
* Silence some warnings.

svn path=/trunk/; revision=59519

[SMSS]
- In the token parsing function SmpParseToken, just return success directly if the input string has zero length, instead of running code which does nothing at the end, because the length was zero...
- Give a default value for the "BootExecute" entry in the SMSS registry configuration table (this value is used when there is no BootExecute registry value inside HKLM\System\CurrentControlSet\Control\Session Manager) and precise that it is a multi-string value.
- Remove a redundant "Environment" entry in this table, too.

svn path=/trunk/; revision=59518

[RTL]
- Fix a potential overflow bug.
- Don't hardcode the "/ 2" in indices computation (2 == sizeof(WCHAR))
- Fix a string enumeration bug when enumerating strings inside REG_MULTI_SZ values, in RtlQueryRegistryValues.

svn path=/trunk/; revision=59517

[WIN32K]: Get rid of unused variables, fix all warnings on GCC 4.7

svn path=/trunk/; revision=59516

Final fixes for the SYNCHRONIZE flag
CORE-7361 #resolve #comment Fixed in revisions 59512, 59514 and 59515.

svn path=/trunk/; revision=59515

Add the SYNCHRONIZE flag to the DesiredAccess parameter when using the flags FILE_SYNCHRONOUS_IO_(NON)ALERT in NtCreateFile or NtOpenFile, even if one uses DesiredAccess == GENERIC_xxx (which should map to some FILE_GENERIC_xxx already containing the SYNCHRONIZE flag).
Indeed, it appears after extended testing that, when calling from user-mode, Windows checks the real presence of the SYNCHRONIZE flag when using FILE_SYNCHRONOUS_IO_(NON)ALERT even if we pass any of the GENERIC_xxx access flags...

Otherwise, when using FILE_GENERIC_(READ/WRITE) flags directly, there is no need to add the SYNCHRONIZE flag since it is already incorporated in it.

See CORE-7361 for more details.

svn path=/trunk/; revision=59514

[WIDL]
- Apply Wine commit "widl: Avoid short printf formats with int variables." (7ca08fee) by Alexandre Julliard. Fixes clang warnings

svn path=/trunk/; revision=59513

Add the SYNCHRONIZE flag to the DesiredAccess parameter when using the flags FILE_SYNCHRONOUS_IO_(NON)ALERT in NtCreateFile or NtOpenFile, even if one uses DesiredAccess == GENERIC_xxx (which should map to some FILE_GENERIC_xxx already containing the SYNCHRONIZE flag).
Indeed, it appears after extended testing that, when calling from user-mode, Windows checks the real presence of the SYNCHRONIZE flag when using FILE_SYNCHRONOUS_IO_(NON)ALERT even if we pass any of the GENERIC_xxx access flags...

Part 1/X

svn path=/trunk/; revision=59512

Fix ASSERTMSG definition and usage (which is: ASSERTMSG(Message, expression) and not the other way around, see msdn.microsoft.com/en-us/library/windows/hardware/ff542113(v=vs.85).aspx , http://osronline.com/DDKx/ddtools/debugfns_41yr.htm and google...) (Part 4/4)
Everything fixed now \o/

svn path=/trunk/; revision=59511

Fix ASSERTMSG definition and usage (which is: ASSERTMSG(Message, expression) and not the other way around, see msdn.microsoft.com/en-us/library/windows/hardware/ff542113(v=vs.85).aspx , http://osronline.com/DDKx/ddtools/debugfns_41yr.htm and google...) (Part 3/4)
NOTE: Fixing builds ;)

svn path=/trunk/; revision=59510

Fix ASSERTMSG definition and usage (which is: ASSERTMSG(Message, expression) and not the other way around, see msdn.microsoft.com/en-us/library/windows/hardware/ff542113(v=vs.85).aspx , http://osronline.com/DDKx/ddtools/debugfns_41yr.htm and google...) (Part 2/X)
WARNING: Might break builds now.

svn path=/trunk/; revision=59509

Fix ASSERTMSG definition and usage (which is: ASSERTMSG(Message, expression) and not the other way around, see msdn.microsoft.com/en-us/library/windows/hardware/ff542113(v=vs.85).aspx , http://osronline.com/DDKx/ddtools/debugfns_41yr.htm and google...) (Part 1/X)

svn path=/trunk/; revision=59508

[DBGHELPHOST]
* Fix some warnings. Brought to you by Thomas Faber.

svn path=/trunk/; revision=59507

[WIDL]
- Fix some warnings

svn path=/trunk/; revision=59506

* Improve the way we create rossym debug info. The effort results in ~85% *smaller* build folder with ninja all with no code changes.

[DBGHELPHOST]
* Introduce a self-contained, static library version of dbghelp, in order to leverage its PE DWARF support. Thanks to Jerome Gardou for his work on the compatibility layer, and to Thomas Faber for helping me with the review/improvements.
* Unify the new host lib with the existing dll codebase using preprocessor conditions. This prevents code duplication.
* Skip as much unneeded functionality as possible when compiling dbghelphost to keep it light and straight to the point.
[RSYM]
* Introduce the required functions that allow parsing DWARF (using dbghelphost) and using it (along with coff symbols) to create the rossym debug info. Brought to you by Awesome Arty with some bugfixes from Jerome Gardou. Many thanks to Thomas Faber for assisting me with the testing/bug hunting.
[CMAKE/GCC]
* Introduce a combination of dwarf and debug emission flags that ensure the smallest debug info size among all the possible options we have.
* Introduce compressed debug sections that I already included the support for in RosBE 2.1. Thanks to the mingw-w64 folks (Kai Tietz) for the patch.
* Don't compress debug sections of C++ modules for now due to a bug in the toolchain that leads to spamming the build at link time with a warning.
* Don't run rsym on the RC shared libraries. Thanks to Thomas Faber for spotting this.
[DBGHELP]
* Update the ros diff to reflect the changes introduced by the dbghelphost unification.

svn path=/trunk/; revision=59505

[INFLIB]
- Avoid use of swprintf, which is blatantly incompatible with -fshort-wchar
CORE-6918 #resolve

svn path=/trunk/; revision=59504

[UNICODE]
- Fix missing isdigit

svn path=/trunk/; revision=59503

[RSYM]
* Formatting. No intended code changes.

svn path=/trunk/; revision=59496

[NTOS]
- Fix CcMdlReadComplete2 prototype.
- Fix a bug in Dominique Cote's code: when performing FsRtlGetFileSize, the length of the buffer should be set accordingly to the information class asked for, that is - FILE_STANDARD_INFORMATION structure. Previous calculation formula makes no sense at all and leads to memory corruption (if filesystem driver is stupid enough) or inability to get file size (if filesystem driver is coded by good developers).

svn path=/trunk/; revision=59495

[DDK]
- Fix ExAllocateFromZone. I understand it's deprecated, but it having a broken implementation which can't work at all because the one who coded this function failed to understand basic algorithms of linked lists is nonsense.

svn path=/trunk/; revision=59494

[NTOS/EX]
- ExInitializeZone: Don't add to an uninitialized TotalSegmentSize value, set it instead.

svn path=/trunk/; revision=59493

Dedicated to Timo.

svn path=/trunk/; revision=59492

[NTOS]
- Implement enabling Large Pages support if it is detected. The Page Size Extension (PSE) bit in CR4 is enabled when paging is disabled. To achieve that, a temporary virtual and physical identity mapping is created, which is discarded after PSE is enabled.

svn path=/trunk/; revision=59491

[CMAKE]
- Add configure support for VS 2013
- Add a missing /nologo to assembler invocations
- Properly complain about old MSVC versions during configure instead of failing the build

svn path=/trunk/; revision=59488

[SAMSRV]
Add security descriptors for the account and builtin domains.

svn path=/trunk/; revision=59487

[KERNEL32]
Fix some broken logic: we can duplicate console handles only if both the source and the target processes are in fact the current process.

svn path=/trunk/; revision=59486

[KERNEL32]
Fix parameters usage when calling NtDuplicateObject.

svn path=/trunk/; revision=59485

[SAMSRV]
Implement SamIFree_SAMPR_GET_MEMBERS_BUFFER.

svn path=/trunk/; revision=59484

[BROWSEUI]
- This is not Wine source. Remove hacks accordingly

svn path=/trunk/; revision=59483

[IEFRAME][MSHTML_WINETEST]
- Use SHANDLE_PTR in IWebBrowserApp::get_HWND. Already sent to Wine.

svn path=/trunk/; revision=59482

[CMAKE]
- Do not allow add_delay_importlibs to be called on static libraries

svn path=/trunk/; revision=59481

[WINSRV]
Fix a delayed-imports problem, catched by Timo and Thomas with a MSVC build. Caused by the fact that the static library consrv specified delayed imports for it, thinking that they will be added to the list of delayed imports of winsrv. In fact it doesn't work and those imports become real imports.

svn path=/trunk/; revision=59480

[MSHTML_WINETEST]
- Use winetest_interactive to disable tests

svn path=/trunk/; revision=59479

[NTOSKRNL]
Fix return of MiProtectedPoolUnProtectLinks() which was always returning null, instead of returning
safe unprotected links when needed

svn path=/trunk/; revision=59478

[UUID]
- Remove unused file whose only purpose is to confuse people

svn path=/trunk/; revision=59477

[PSDK]
- Reduce shdeprecated.idl differences to Wine. Fixes a few warnings

svn path=/trunk/; revision=59476

[PSDK]
- Move some stuff around in shdeprecated.idl to reduce the difference with Wine
- Move TLENUMF to tlogstg.idl

svn path=/trunk/; revision=59475

[BROWSEUI][EXPLORER_NEW]
- Add missing includes aka fix build

svn path=/trunk/; revision=59474

[PSDK]
- Move ITravelEntry, ITravelLog, IShellService and IBrowserService* to shdeprecated.idl

svn path=/trunk/; revision=59473

[INCLUDE]
- Make our test/debug headers C++-compatible

svn path=/trunk/; revision=59472

Forgot this file

svn path=/trunk/; revision=59471

[CONSRV]
Create a new consrv_new to study its interfacing with the console driver (condrv).

svn path=/trunk/; revision=59470

[BROWSEUI]
- Export CCommonBrowser, CRegTreeOptions, CGlobalFolderSettings
- Fix IBrowserService3 method signatures in CCommonBrowser
- Add IContextMenu3 to CBandSiteMenu

svn path=/trunk/; revision=59468

[BROWSEUI]
- Fix the order of some COM interfaces. This should have no functional effect, but makes testing easier.

svn path=/trunk/; revision=59467

[COM_APITEST]
- Add additional classes and interfaces

svn path=/trunk/; revision=59466

[COM_APITEST]
- Add support for classes with ThreadingModel values other than Apartment

svn path=/trunk/; revision=59465

[PSDK]
- Add CLSID_CCommonBrowser, CLSID_DeskBar, CLSID_DeskBarApp

svn path=/trunk/; revision=59464

[KMTESTS:RTL]
- Fix a warning

svn path=/trunk/; revision=59463

[SYSSETUP]
- Correct spelling of 'Accessibility' in shortcuts.inf. Patch by Lee Schroeder.
CORE-7342 #resolve

svn path=/trunk/; revision=59462

[CONSRV-WINSRV]
Now that the console server works fine, move it into its real place (as in Windows), that is, inside winsrv.dll, instead of hosting it in consrv.dll.

svn path=/trunk/; revision=59461

[KERNEL32]
CreateEvent --> NtCreateEvent --> NotificationEvent or SynchronizationEvent (contrary to CreateTimer --> NtCreateTimer).

svn path=/trunk/; revision=59459

[CONFIGURE.SH]
Fix formatting

svn path=/trunk/; revision=59458

[CONFIGURE.SH]
Kill bashisms introduced by r59197 and replace them by sh code.

This is mainly to preserve maximum portability of the code, and to
fix configure.sh on platforms where /bin/sh doesn't point to /bin/sh ;-).

svn path=/trunk/; revision=59457

[SAMSRV]
SamrChangePasswordUser: Update the PasswordLastSet time after a new password has been set.

svn path=/trunk/; revision=59456

Remove an unneccesary commented definition of RegOpenCurrentUser.

svn path=/trunk/; revision=59455

[SAMSRV]
SamrChangePasswordUser: Use SampSetUserPassword to set the password hashes and update the password history.

svn path=/trunk/; revision=59453

[NPFS]
NpfsCreateNamedPipe: Check for invalid disposition values. Fixes 3 winetest failures.

svn path=/trunk/; revision=59451

[SAMSRV]
SamrChangePasswordUser: Check the old password before storing the new one.

svn path=/trunk/; revision=59449

[CONDRV]
- Hey Arch! You're displaying Major function codes, not IOCTL codes. Also, remove unnecessary casts (coming from some old code), and a use-after free.
- Add some memory helpers.

svn path=/trunk/; revision=59448

[CONDRV]
Initial commit of the ReactOS Console Driver.
For the moment, it's just able to load/unload, creating a "controller" device
and being able to DPRINT1 lots of messages when you try to access to it.

svn path=/trunk/; revision=59447

[ATL]
- Arch, don't forget to initialize your pointer

svn path=/trunk/; revision=59446

[SAMSRV]
Start work on SamrChangePasswordUser. The new passwords can be stored but the required checks are still missing.

svn path=/trunk/; revision=59445

[ATL]
- Don't pretend success if we actually failed in CComModule::GetClassObject

svn path=/trunk/; revision=59444

[CMAKE]
- Try again to fix version detection. If anyone sees compilers that are not .00 versions, please complain. This is required because a simply "contains" check can match all parts of the version number -- e.g. 16.00.40219.01 contains "19." in addition to "16."

svn path=/trunk/; revision=59443

[COM_APITEST]
- Add a test verifying the interfaces exposed by various COM classes, grouped by server dll
ROSTESTS-98

svn path=/trunk/; revision=59442

[NDK]
- Add RtlStringFromGUID

svn path=/trunk/; revision=59441

[PSDK]
- Fix IShellFolderView too

svn path=/trunk/; revision=59440

[PSDK]
- Declare IID only once aka fix GCC build

svn path=/trunk/; revision=59439

[PSDK]
- Fix ShellItem/ShellFolderItem confusion and some misc GUIDs

svn path=/trunk/; revision=59438

Revert revision r59381 since it fails to detect VS2010 (and I suppose the other ones, then).
Indeed with VS2010, checking for the string content " 16." instead of "16." makes version checking fail.
After checking with a hex editor what was the character preceding the number "1" of the "16." part of the
version string returned by cl.exe, I saw that it wasn't a space as thought by the author, but the character
of code 0xA0 ...

svn path=/trunk/; revision=59437

[KERNEL32]
Forgot to add those files in the previous commit r59435:
We now store only console handles and not console pointers themselves.

svn path=/trunk/; revision=59436

[CONSRV]
- Use a handle table for storing consoles. Now checking validity of console handles is done in O(1) instead of O(n) as before.
  I will see later on whether it would be good to have both a handle table and a linked list (which is removed here) for storing them
  (when condrv will be a real console driver, handles to consoles will be obtained as you would obtain usual handles to devices, etc...
  so that consoles will be internally stored in a linked list again).
- Add a "dummy" frontend to have the possibility to adopt default actions when no frontend is attached to a console, and to always have
  a valid virtual table for them.

svn path=/trunk/; revision=59435

[CONSRV]
Fix copy of text in text-mode screenbuffers, in case we are copying NULL chars.

svn path=/trunk/; revision=59434

[CONSRV]
- Fix some files' info-header text.
- Move code of SrvSetConsoleWindowInfo to ConDrvSetConsoleWindowInfo.
- Introduce the helper function GetScreenBufferSizeUnits to retrieve screen-buffer width/height units.

svn path=/trunk/; revision=59433