[LIBWINE]
- Fix broken indentation (-Wmisleading-indentation)

svn path=/trunk/; revision=73641

[ACPI]
- Fix some broken indentation (and GCC6 -Wmisleading-indentation warnings with it)

svn path=/trunk/; revision=73637

[FREELDR]
- Correctly check for vertical retrace in PcVideoSync. CID 513066.

svn path=/trunk/; revision=73636

[DHCPCSVC]
- Addendum to r69652: Avoid some more "Operands don't affect result" warnings. CID 515151, 515152, 515153

svn path=/trunk/; revision=73635

[PSDK]
- Remove incorrect bit masking from IsRecognizedPartition & IsContainerPartition. CID 1341188.

svn path=/trunk/; revision=73634

[FRAMEDYN]
- Fix integer overflow checks. CID 1101981, 1248380, 1248381

svn path=/trunk/; revision=73633

[NDK]
- Add missing lpctypes.h include in lpcfuncs.h
- Fix some file header comments

svn path=/trunk/; revision=73632

[PING]: Convert ping to use ConStreams. Also fix file redirection.
CORE-12122 #resolve

svn path=/trunk/; revision=73626

[CMD]: The 'type' command can open & display files opened with write access.

svn path=/trunk/; revision=73623

[TREE]: Rework the utility (use explicit UNICODE, use WIN32 types, remove useless dependency to user32...), and add support for console ConUtils streams.
CORE-12677 #resolve

svn path=/trunk/; revision=73622

[FREELDR]
- Simplify the implementation of the 'DiskGetBootPath' function;
- Cache the retrived freeldr boot path (corresponding to a given 'FrldrBootDrive' number);
- Introduce a 'DiskIsCdRomDrive' function to attempt to fix CORE-12692 .

svn path=/trunk/; revision=73621

[CALC]
- Do not push memory stores onto the operation stack since they can't be acted upon. Based on a patch by Katayama Hirofumi MZ.
CORE-12325 #resolve

svn path=/trunk/; revision=73620

[SHELL32]: Fix build on *sane* compilers (wth, is that an assignment without the equal sign, or a called constructor, missing its parentheses?!).

svn path=/trunk/; revision=73619

[FREELDR]: Part 3 of my local changes merging:
- Remove the DiskVtbl functions (machxbox.c) of the XBOX and share them instead with those of the PC, since they are basically the same.
  They just however differ by which 'DiskGetPartitionEntry' function they use. To cope with this, 'DiskGetPartitionEntry' becomes a pointer
  to either the default function 'DiskGetMbrPartitionEntry' (which indeed assumes a MBR-layout, see partition.c), or the special function
  'XboxDiskGetPartitionEntry' in the case of partitionless XBOX disks.
  Note that, by the way, partition.c should in the future support GPT-layout disks as well...
- The boot devices pre-initialization code can also be shared as well, thus getting rid of 'XboxInitializeBootDevices', because we note that
  this is exactly what the (old) XBOX version of 'DetectBiosDisks' did at its beginning.
- But then, we can also share 'DetectBiosDisks' XBOX code with its PC version, and also *fix* the PC code version as well!
  'DetectSystem' and 'DetectBiosDisks' are therefore merged *as they should be*. Thanks to the boot devices pre-initialization code that has
  run before the hardware detection, 'DetectBiosDisks' can just use the cached information.
- Now we correctly initialize the PC root HW system configuration key: we first create it, then, after the bios disks (floppies + HDDs) are enumerated
  we set the BIOS INT13h disk data to the system key's configuration data.

svn path=/trunk/; revision=73618

[FREELDR]
- Correctly check for buffer overflow in DetectPnpBios. Patch by Serge Gautherie.
CORE-12623 #resolve

svn path=/trunk/; revision=73617

[FREELDR]: More local changes merging, part 2/x:
- Move the HW detection code specific to the PC architecture from hardware.c to machpc.c, keeping in hardware.c the code that is shared between PC and XBOX.
- Move what remained of i386disk.c into pcdisk.c (specific to PC architecture) + code formatting.
- Move what remained of xboxhw.c into machxbox.c (specific to XBOX architecture); we discover that some code related to disk management is actually shared with PC architecture (in hwdisk.c): hwdisk.c therefore contains disk-management routines common to both PC & XBOX.
- xboxdisk.c: Code formatting only.
- Cleanup in disk.c/disk.h

svn path=/trunk/; revision=73616

[EXPLORER] Re-save the start menu banner so that we can display it again (and touch the .rc so it will be used).

svn path=/trunk/; revision=73615

[SHELL32] - CDefView: When dragging an item it shouldn't interact with itself. So you can move a folder in itself. Highlight the item that we are dragging over if it can accept the item that is being dragged.

svn path=/trunk/; revision=73614

[CMAKE]
- Add missing target-level dependency when passing a target to add_rostests_file

svn path=/trunk/; revision=73613

[EXPLORER]
* Recreated the start menu banner as a SVG file in Inkscape.
* Deleted old PSD (Photoshop) files.
* Replaced the explorer banner BMP with one exported by Inkscape from the SVG file.
The end result is slightly sharper text, but otherwise identical.

svn path=/trunk/; revision=73612

Fix build.

svn path=/trunk/; revision=73611

[FREELDR]: Gently merge my local changes, part 1/x:
- Adjust the prototype of DiskGetBootPath;
- Add back 1 HW helper function.

svn path=/trunk/; revision=73610

[FREELDR]
- Add missing checks for allocation failure in DetectSerialPointerPeripheral and DetectPS2Mouse. Patch by Serge Gautherie.
CORE-12623

svn path=/trunk/; revision=73608

[NTOS:MM]
- Support MmSpecialPoolTag == '*' to mean all tags. By Serge Gautherie.
CORE-12711 #resolve

svn path=/trunk/; revision=73607

[FREELDR]
- Use 'DiskReadBufferSize' instead of hardcoded values in hwdisk.c
- Transform reactos_arc_disk_info into a ARC_DISK_SIGNATURE_EX structure, so that its conversion into a ARC list in winldr.c becomes a bit simpler.
- In winldr.c:
  * In case we fail to either open the SYSTEM hive or scan it, bail out with an error message, instead of trying to continue loading ROS.
  * Fix an old comment related to ACPI;
- In memory.c/h: Do some cleanup (in particular, just use the MachXXX macros instead of also defining functions with the same names as the macros, and which just redirect into the virtual table).

svn path=/trunk/; revision=73606

[HIDPARSE][HIDPARSER][LIBUSB][USBHUB] Merge USB stack improvements by Vardan Mikayelyan in GSoC.

svn path=/trunk/; revision=73605

[STORAHCI] Merge Storport Miniport driver by Aman Priyadarshi in GSoC.

svn path=/trunk/; revision=73604

[SHELL32]: Replace some animations with RLE-8 compressed ones; courtesy Jared Smudde. Thanks!
CORE-11643

svn path=/trunk/; revision=73601

[SDK] -rosctrls.h: Add GetItemPosition and SetItemPosition methods in CListView.

svn path=/trunk/; revision=73600

[SHELL32] -CDefView: Implement moving icons around. Note that the positions are never stored so a refresh returns them to their initial position.

svn path=/trunk/; revision=73599

[FREELDR]: Fix GCCLin compilation (#if'd 0 an unused static function; I keep it there still in case it can become useful later).

svn path=/trunk/; revision=73597

[XDK]: Do not always redefine the ASSERT(MSG) macros with the NT_xxx ones, especially for the FreeLdr/... bootloader (where we don't use WinDbg for debugging, yet), because in this case the int 0x2c is hardly informative. We fall back to the good old RtlAssert which displays an assertion failure string in the default debug output port (e.g. serial), and then does an int 0x03.

[FREELDR]
- "static"-ify some printing helper functions;
- hide the textmode cursor before blue-screening;
- disable interrupts before hanging forever.

svn path=/trunk/; revision=73596

[ISOHYBRID]
Import the isohybrid tool from http://repo.or.cz/syslinux.git/tree/138e850fab106b5235178848b3e0d33e25f4d3a2:/utils

I had to add a lot of glue code (getopt, err/errx/warn/warnx) to get it to compile under all our build platforms Windows, Linux and macOS.
GPT/Mac partition support is disabled at the moment (look for REACTOS_ISOHYBRID_EFI_MAC_SUPPORT), that would need a portable libuuid.
A fix for correctly opening ISOs as binary and not text-mode has been implemented and already sent upstream.

ISOs are not being patched automatically yet due to CORE-12692

CORE-12648

svn path=/trunk/; revision=73595

[BOOTSECT]
Overhaul our ISO boot sector:
- Add another entry point for hybrid booting (called by isombr) and the hybrid signature needed for isohybrid.
- Import latest isolinux.asm "API" from http://repo.or.cz/syslinux.git/blob/8bbb10b70905339b7ca6b58d6833b6a95ab03dae:/core/isolinux.asm (latest version to date) and http://repo.or.cz/syslinux.git/blob/9141c603930ef23da1e08e487a0c26750fbb4fbe:/core/isolinux.asm (latest version that came with all functions in ASM).
  Among other things, this brings us getlinsec_ebios (for reading setupldr.sys in hybrid/MBR mode) and BrokenAwardHack (for working around broken Award BIOSes)
- Fix upstream getlinsec_ebios to load files >64K.
- Load the HDD boot sector directly to 0x7C00 instead of trackbuf. This way, we don't have to load it again when the user decides to boot from HDD.
- Check for the 0xAA55 signature to decide if the HDD contains a valid MBR. This was done differently and inconsistently for isoboot and isobtrt.
- Adapt the file and comment style to ReactOS guidelines. Add meaningful comments to our main code. Remove stuff that was just taken from isolinux.asm, but makes no sense for our version.
- Remove DEBUG_MESSAGES ifdef. There is no chance they can ever fit into the 2K sector with all the added features. Debugging can still happen using Bochs or adding specific messages.

Tested with Bochs, QEMU, VMware, VirtualBox and four real computers.
Many thanks to Hermès for all the helpful suggestions and additional investigations! :)

CORE-12648

svn path=/trunk/; revision=73594

[SUBST] improve/fix German translation. Patch by "Joefish". CORE-12654 #resolve

svn path=/trunk/; revision=73593

[HELP] Small grammar correction. Based on parch by Doug Lyons. CORE-12669 #resolve

svn path=/trunk/; revision=73592

[SHELL32]
- CDefaultContextMenu: Fix the implementation of the QueryContextMenu to respect how IContextMenu should function. Don't ignore idCmdFirst and idCmdLast parameters. Return a correct HRESULT. For now I'm not sure how 0 cmd ids should be handled and if the standard menu items should also be affected by these parameters.

svn path=/trunk/; revision=73591

[WELCOME]
- Start implementing the possibility of customizing the Welcome.exe utility using configuration INI files (e.g. what we would do for FOSDEM, CLT, etc...)
- Resources cleanup.

svn path=/trunk/; revision=73590

[i8042prt]: Yet another hack for Dell Latitude E4310 for touchpad.
CORE-12434

svn path=/trunk/; revision=73589

[FREELDR]
Add VESA DDC detection and EDID read code. Work in progress.

svn path=/trunk/; revision=73587

[CDFS]
 - Reduce stack usage by manipulating UNICODE_STRINGS we are given instead of copying it a gazillion times
 - Do not assume said strings are null-terminated

svn path=/trunk/; revision=73586

[ADVAPI32]
Add I_ScIsSecurityProcess() stub.

svn path=/trunk/; revision=73585

[PSDK]: Add the declarations for IsNetDrive, DriveType and RealDriveType functions (from shell32).

svn path=/trunk/; revision=73584

[SHELL32]: Follow-ups to r73579 and r73580: use PathYetAnotherMakeUniqueName.
CORE-12684

svn path=/trunk/; revision=73583

[SHELL32]: Code formatting only.

svn path=/trunk/; revision=73582

[SHELL32]: Remove trailing whitespace.

svn path=/trunk/; revision=73581

Fix a bug introduced in r73579.
CORE-12684

svn path=/trunk/; revision=73580

[SHELL32]: Fix the "CNewMenu.cpp:432: Unexpected failure 80070002." debug line that appears when one tries to create a new file using the explorer (note: did not happen for new directories).
CORE-12684 #resolve #comment Fixed in r73579.

svn path=/trunk/; revision=73579

[SHELL32]
- Now that we correctly handle the shell link data block list with APIs, no need to hack an extra zeroed DWORD at the end of shortcut files!
- Fix the display of the shortcut target location: this typically displays the (last) directory where the target resides. Addendum for r53624, r54957 and CORE-5730 CORE-6104

svn path=/trunk/; revision=73578

[SYSSETUP][SHORTCUTS.INF]: Follow-up of r71049 and CORE-11020 :
- Make more shortcuts start from the user directory;
- The desktop shortcut of the ReadMe file should not use an icon location, so that the shell correctly retrieves the icon to display from the type of the file (in our case, .txt).
- Make the necessary adjustments in syssetup/install.c :
  * opt-out setting the icon location;
  * in addition, since we now support shell link targets containing environment variables, don't always expand the target path before setting up the shortcut, but do it just in the case we have to compute a suitable working directory.

svn path=/trunk/; revision=73577

[SHELL32]: CShellLink fixups Part 2:
- Add COM inheritance for interfaces IExtractIconA/W, and add in comment the missing other ones, the ordering of which is given by the apitests/com/shell32.
- Add proper support for the extra data block list (which is added at the end of the .lnk files), by using the API from shlwapi.dll: SH[Read|Write|Free]DataBlockList, SH[Add|Find|Remove]DataBlock.
- Using this support, getting/setting the MSI data block becomes as simple as child's play, and opens the possibility for implementing support for the other types of blocks.
- This in particular enables support for paths with environment variables for the link's target and icon: CORE-9236 #resolve
- Fix all the "shell32_winetest shelllink" tests: CORE-7158 #resolve
  Some of the fixes are inspired from a patch by Katayama Hirofumi MZ.
- Fix all the "shell32_apitest CShellLink" tests *but* those calling IExtractIcon::GetIconLocation().
- Implement a hackish substitute to the shell32!PathResolve API until someone writes a correct one (see the code & the FIXMEs for some ideas), possibly using the SHELL_xxx helpers in Wine's shellpath.c.
- In CFSExtractIcon_CreateInstance: Because IShellLink::GetIconLocation can return no icon location, in case none is specified in the .lnk (proved by apitests), we have to call the shell link's IExtractIcon::GetIconLocation in order to retrieve the icon of its target (yes, some shortcuts are made like that, e.g. Notepad++ 6.9 one...).

- More fixes...
- ... and a lot of documentation added in the code for you!

CORE-12682

svn path=/trunk/; revision=73576

[KERNEL32_APITEST] -Make FindActCtxSectionStringW test less strict.

svn path=/trunk/; revision=73575

[KERNEL32_APITEST] -Try to avoid crashes in DefaultActCtx test.

svn path=/trunk/; revision=73574

[SHELL32]: CShellLink fixups Part 1:
- Use STDMETHODCALLTYPE for the interface methods calling convention;
- Use the naming convention "m_Member" for the class members (note that I haven't applied the convention for some members: because they will go away in the next commits);
- Use _countof() where needed;
- Make the ANSI CShellLink::GetIconLocation call the UNICODE version;
- Move some functions around to put them closer to the functions that call them;
- Very minor code formatting.
CORE-12682

svn path=/trunk/; revision=73573

[KERNEL32_APITEST] -Add redir2dep.manifest to the testdata.

svn path=/trunk/; revision=73572

[KERNEL32_APITEST] -Fix gcc build

svn path=/trunk/; revision=73571

[SHELL32]
- Use explicit unicode SHFILEINFOW structure (where needed);
- In CFSExtractIcon_CreateInstance, use _countof instead of hardcoding the MAX_PATHs; add some FIXMEs about the 'flags' parameter;
- Code formatting in CExtractIcon.cpp.

svn path=/trunk/; revision=73570

[regtest.cmd] -RENAME parameter in add_rostests_file doesn't have any effect on the name of the installed file so rename sine files manually before starting tests.

svn path=/trunk/; revision=73569

[KERNEL32_APITEST]
- Fix and enable tests for redirection support in LoadLibraryExW.
- Add tests for the default activation context that is active for every process.

svn path=/trunk/; revision=73568

[PSDK]: Update shlobj.h:
- Define Logo3-related symbols only when NTDDI_VERSION < NTDDI_LONGHORN (Vista);
- In the SHELL_LINK_DATA_FLAGS enumeration, some symbols are defined for Vista+ only;
- Update the comment for EXP_LOGO3_ID_SIG;
- The PRF_TRYPROGRAMEXTENSIONS define is actually the combination of PRF_EXECUTABLE and PRF_VERIFYEXISTS;
- Add PRF_REQUIREABSOLUTE and fix PRF_DONTFINDLNK typo.

svn path=/trunk/; revision=73567

[INCLUDES]: Add in shlwapi_undoc.h the exported-by-ordinal-only shlwapi functions SH[Add|Remove|Find]DataBlock, SH[Write|Read|Free]DataBlockList, and SHExpandEnvironmentStrings[ForUser]A/W . Will be used later.
CORE-12682

svn path=/trunk/; revision=73566

[SHLWAPI]
- Fix SHAddDataBlock function signature & return value.
- Back this up by updating the corresponding wine test.
Send upstream, I will remove the __REACTOS__ markups once this is merged in Wine.

svn path=/trunk/; revision=73565

[SHLWAPI]: Follow-up on r73527 and r73529: Use WINVER conditional to separate old-Win2k3 and new-Vista+ behaviours of PathIsUNCA/W.
CORE-12653

svn path=/trunk/; revision=73564

[SHELL32_APITEST]
- Use smart COM pointers (thanks Mark ;)
- Add other tests for testing non-existing files in existing directories.

svn path=/trunk/; revision=73563

[SHELL32_APITEST]: Add more tests to see how IShellLink::Get/SetIconLocation interacts with IExtractIcon::GetIconLocation.

svn path=/trunk/; revision=73562

Comply to GCCLin rules.

svn path=/trunk/; revision=73561

[SHELL32_APITEST]: Add some tests for GetIconLocation. Show also that CShellLink inherits from IExtractIcon(A/W).

svn path=/trunk/; revision=73560

[SHELL32_APITEST]
- Fix the names of some expected return values get from the APIs;
- Move a bit the TestShellLink() function.
- When displaying hexadecimal, prepend the "0x" string (otherwise, when you see: "hr = 10", is it 10 in decimal or in hex??)
- Call CoUninitialize() at the end of the tests.

svn path=/trunk/; revision=73559

[SHELL32]
Fix truncated French text in File Properties dialog.
Patch by Katayama Hirofumi MZ.

CORE-12554

svn path=/trunk/; revision=73558

[INTL]
Fix truncated French text in Languages dialog.
Patch by Katayama Hirofumi MZ.

CORE-12366

svn path=/trunk/; revision=73557

[SHELL32]
Fix truncated French text in Open With dialog.
Patch by Katayama Hirofumi MZ.

CORE-12556

svn path=/trunk/; revision=73556

[BOOTSECT]
Add a simple MBR boot sector that chainloads the El Torito BIOS Boot Sector of the ReactOS ISO.
This is our equivalent to syslinux's isohdpfx.S and will be used as input for the soon to be imported "isohybrid" patching tool.

Largely untested Work In Progress towards CORE-12648

svn path=/trunk/; revision=73555

[NTDLL_APITEST] -Create a copy of the test in testdata and add along three empty files: ntdll_apitest.exe.local, shell32.dll and test.dll. Make the test start the subprocess from testdata to avoid any kind of interference of the .local to all the other tests. The empty dlls are needed there because there is no way to rename the files while adding them to the cab file like we do with NAME_ON_CD and creating them at runtime is not reliable as caching can cause random failures.

svn path=/trunk/; revision=73554

[UBTRFS]
[BTRFSLIB]
As suggested by Peter, bring back the btrfslib and make use of it in ubtrfs.
This will allow code reuse especially for people having plans with btrfs and usetup ;-)

CORE-12617

svn path=/trunk/; revision=73553

Fix a line that I forgot to finish editing.

svn path=/trunk/; revision=73552

[NTDLL_APITEST]
- Add more tests for RtlDosApplyFileIsolationRedirection_Ustr. Unfortunately creating .local files and making it have instant effect seems to be impossible. So I added tests for .local than can be enabled to show that this is indeed the behaviour but I have disabled them to avoid random failed tests.

svn path=/trunk/; revision=73551

[PRINTUI] Properly stub bPrinterSetup. Patch by Jared Smudde. CORE-12600 #resolve #comment Thanks!

svn path=/trunk/; revision=73550

SHLWAPI_APITEST: Add tests for StrFormatByteSizeW. Patch by Doug Lyons. CORE-12661 #resolve

svn path=/trunk/; revision=73549

[ADVAPI32]
Implement LsaQueryInfoTrustedDomain() and LsaQueryTrustedDomainInfo().

svn path=/trunk/; revision=73548

[ADVAPI32]
Implement I_ScPnPGetServiceName().

svn path=/trunk/; revision=73547

[KERNEL32]: Fix (old-style) annotations.

svn path=/trunk/; revision=73546

[SHLWAPI_APITEST]: Use the 'add_rostests_file' command from Colin ;)

svn path=/trunk/; revision=73545

[SHLWAPI_APITEST]
- Add tests for PathIsUNCServer and PathIsUNCServerShare, by Jared Smudde. Thanks! ROSTESTS-258 #resolve
- Simplify the code: Since those functions (as well as PathIsUNC) are public and exported by name, we can directly load the apitest with shlwapi and call the APIs directly (no need for explicit LoadLibrary & GetProcAddress).

svn path=/trunk/; revision=73544

- Fix leak.

svn path=/trunk/; revision=73543

FTSU GCCLin.

svn path=/trunk/; revision=73542

[SHELL32_APITEST]: Improve a bit the output of the CShellLink test.

svn path=/trunk/; revision=73541

[CMAKE]
Call the NAME_ON_CD parameter RENAME (because it's not tied to a CD/ISO) and simplify the implementation.

svn path=/trunk/; revision=73540

[LOCALSPL_APITEST]
Disable fpGetPrintProcessorDirectory tests that use PSEH.
For some reason, they can only be run once and corrupt something inside spoolsv, so that spoolsv needs to be restarted afterwards.

svn path=/trunk/; revision=73539

[KERNEL32_APITEST] - Add tests for LoadLibraryExW to test its ability to load a redirected dll. Do not include it in the build until a way is found for it to be compiled properly.

svn path=/trunk/; revision=73538

[CMakeMacros.cmake] - Add NAME_ON_CD parameter to add_rostests_file to make it possible to select a different name for a file.

svn path=/trunk/; revision=73537

[SHLWAPI_APITEST]: Commit few tests for PathUnExpandEnvStrings.

svn path=/trunk/; revision=73536

[KMTESTS]
Temporarily disable kmtests for "rostests_install" (and subsequently the WHS-Testbot) as long as kmtest:ExPools crashes the machine and I have to reboot it manually.

svn path=/trunk/; revision=73535

Only generate installation rules for rostests if the ROSTESTS_INSTALL environment variable is defined.

svn path=/trunk/; revision=73534

[ROSTESTS]
- Add a function "add_rostests_file" that calls both add_cd_file() and install() accordingly for regression tests.
  Use this function everywhere in rostests where add_cd_file() was used previously. This puts tests always in the right place on CD and also generates proper installation rules for them.
- Add a custom target "rostests_install" that performs an "install" on all rostests components.

This finally gives us a convenient and reliable way to copy all regression tests and their dependencies into a single directory (specified by the ROSTESTS_INSTALL environment variable).
Will be used on the WHS-Testbot to let it finally run each and every test.

CORE-12401
ONLINE-441

svn path=/trunk/; revision=73533

[NtUser]
- Fix menu surfing while using MS Office 2003. See CORE-12085 and CORE-12071.

svn path=/trunk/; revision=73532

Use wine_dbgstr_w as Jared did originally (I hate this macro because it displays strings in 'C'-language form instead of in natural way).

svn path=/trunk/; revision=73531

/me slaps GCCLin.

svn path=/trunk/; revision=73530

[SHLWAPI]: On Windows 2003, tests show that strings starting with "\\?" are considered UNC, while on Windows Vista+ this is not the case anymore.
Select Win2k3 behaviour: revert r54542 (and add a note).
CORE-12653 #resolve

svn path=/trunk/; revision=73529

Make GCC happy.

svn path=/trunk/; revision=73528

[SHLWAPI_APITEST]: New test for PathIsUNC function, by Jared Smudde. Thanks!
ROSTESTS-256 #resolve

svn path=/trunk/; revision=73527