[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

[SAMLIB]
- Implement SamChangePasswordUser.
- Remove useless casts.

svn path=/trunk/; revision=59432

[INCLUDE]
- Properly declare and use DECLSPEC_CACHEALIGN. Patch by Samuel Serapión.
CORE-7139 #resolve

svn path=/trunk/; revision=59431

[RTL8139]
- Switch back to a 32K receive window, because WRAP doesn't work with a 64K window and our receive code depends on that functionality

svn path=/trunk/; revision=59430

[RTL8139]
- Fix receive stalling if more than 10 packets are in the receive window
- Tune configuration to use larger DMA transfers and begin DMA earlier

svn path=/trunk/; revision=59429

[RTL8139]
- Fix PCH in CMake file
- Fix broken failure path in MiniportInitialize

svn path=/trunk/; revision=59428

[RTL8139]
- Add an RTL8139 driver based on documentation on OSDev, our existing drivers, and some prior work by Z98
- It should fully work on emulated and real RTL8139 hardware (I developed against QEMU's RTL8139)
- It's not 100% complete (some reset and halt paths aren't done) but everything I tested worked (ping, dwnl, rapps, Firefox)

svn path=/trunk/; revision=59427

[NDIS]
- Callers to NdisMInitializeScatterGatherDma always support 32-bit DMA addresses

svn path=/trunk/; revision=59426

Fix some comments

svn path=/trunk/; revision=59423

[CMD]
Features-only commit:
- Fix version presentation at startup and when using the "ver" command (in French)
- Add to the prompt's command help, the fact that the $S specifier can be used to add a space.
- Implement the $I prompt specifier (see http://www.robvanderwoude.com/prompt.php), which sets
  an information line on top of the screen (à la OS/2).
  Use this new setting by default (I'm doing a poll on who loves it / hates it, and how we can improve it).
  To disable it, use the command: "prompt $p$g" (the default windows' prompt).
  To enable it, add the $s specifier to the prompt string (e.g. "prompt $i$p$g").
  For your information, OS/2 uses "prompt $i[$p]" :)

svn path=/trunk/; revision=59422

[CONSRV]
- Move SrvSetConsoleWindowInfo to a better place.
- When enlarging the console buffer, use the attributes of the last cell of each line to fill the new cells for each line.
- Remove usage of HAVE_WMEMSET / wmemset for resizing the buffer (NOTE: maybe it will be needed again later on...)

svn path=/trunk/; revision=59420

[LWIP]
- Don't leak PCBs for connections that were not accepted

svn path=/trunk/; revision=59414

[TCPIP]
- Add missing reference for lwIP on listeners

svn path=/trunk/; revision=59413

[TCPIP]
- Add asserts to ReferenceObject and DereferenceObject to catch reference misuse

svn path=/trunk/; revision=59412

[CMD]
Remove extra \r put by hand in some console output functions, so that we use there only \n.
But modify the ConWrite function so that, if we redirect output to something else than a console
(e.g. redirect to a file or to a serial console via ... > AUX), newline characters \n get converted
into \r\n automatically.

What you get, for instance, is: https://imageshack.com/a/img853/5834/l34.png

svn path=/trunk/; revision=59411

[MSV1_0]
Add dispatcher code to LsaApCallPackage and start working on password change code.

svn path=/trunk/; revision=59408

[CMD]
Fix the default input & output console modes. Now, you can use the command interpreter via serial port redirection, by:
1- Enabling a serial port on the machine on which ReactOS runs.
2- Launching:
   cmd > AUX
   at a command prompt.

svn path=/trunk/; revision=59405

[CMD]
Fix whitespace

svn path=/trunk/; revision=59404

[NETAPI32]
NetUserSetInfo: Implement level 1018 and ignore the usriX_max_storage in levels 2-4.

svn path=/trunk/; revision=59402

[CMD]
Add a helper function which checks whether we are going to output text to a console handle or to a file handle.
Use it in ConWrite and in ConPrintfPaging.

svn path=/trunk/; revision=59401

[CONSRV]
Improve debug prints.

svn path=/trunk/; revision=59400

[KERNEL32]
Code formatting only.

svn path=/trunk/; revision=59399

[CMD]
Yet again some whitespace fixes.

svn path=/trunk/; revision=59396

[CMD]
Fix the "TYPE" command (now behaves as on Windows)

svn path=/trunk/; revision=59395

Add Alter's DVD Write Now software to rapps. It includes a UDF filesystem driver. Working: Access to UDF disks with software (Windows DVDs) and Video DVDs. Not working: Video BluRays (UDF 2.50 not supported by the driver), USB sticks. Untested: Rest

svn path=/trunk/; revision=59394

[CMD]
Give help for the "help" command.

svn path=/trunk/; revision=59393

[QUARTZ]
Use winetest_interactive to disable tests.

svn path=/trunk/; revision=59392

[CMD]
- Remove erratic tabs in language files.
- Add the fact that one can use the "help" command with: "help command" to obtain help for "command".

svn path=/trunk/; revision=59391

[CMD]
Fix some people's names presentation

svn path=/trunk/; revision=59390

[CMD]
And finally...
...
... fix whitespace, part 6/6 !!!! (convert tabs into 4-space indent).

Really, CMD is, together with win32k, the worst indented project I've ever seen.
You have lots of non-sensical extra newlines put in the middle of the code,
weird indentation made by a mix of tabs / spaces which take either 4 or 8 or even (!!)
2 spaces per indentation level; sometimes some guy mixed his space key with his tab key
to add a space between a word and another one; a mix of some sh*tty code styles (K&R and personal ones),
many many times no space between a logical keyword and the operand on which it applies, etc... etc...

I tried to put some sense in it, but I may have forgotten some insidious tabs somewhere...

svn path=/trunk/; revision=59389