Page 11 of 13
Re: Fix of the Week
Posted: Wed Dec 16, 2015 12:35 am
by Pi_User5
Revision 70350:
[WINSRV] Do not kill system processes when a user is logging off. Fixes user logoff. #CORE-2391 #resolve
Re: Fix of the Week
Posted: Wed Jan 27, 2016 9:07 pm
by Pi_User5
70646 by Aleksey Bragin
[LDR]
- Fix a typo in kernel32's manifest prober routine
- Don't RtlGetActiveActivationContext if it's already existing
- Improve actctx and ldr debug prints
- This fixes numerous LDR bugs, such as the famous bug with MSVCR90 CORE-7313, .NET installation issues CORE-7489 and, hopefully, many other. Thank you very much for putting your trust in me
Re: Fix of the Week
Posted: Fri Jan 29, 2016 1:21 pm
by ctasan
Pi_User5 wrote:70646 by Aleksey Bragin
[LDR]
- Fix a typo in kernel32's manifest prober routine
- Don't RtlGetActiveActivationContext if it's already existing
- Improve actctx and ldr debug prints
- This fixes numerous LDR bugs, such as the famous bug with MSVCR90 CORE-7313, .NET installation issues CORE-7489 and, hopefully, many other. Thank you very much for putting your trust in me
This commit led some regressions, though.
Re: Fix of the Week
Posted: Sun May 08, 2016 1:14 am
by middings
I nominate pschweitzer's
r71270 for Fix of the Week. This fix resolved
CORE-3875, a bug affecting FASTFAT. The log message says:
In case we're performing an in-place move (ie, in the same directory), if short name matches before and after, keep it.
This avoids that a file with long file name sees its short name changing on normally 'non-effect' renames.
For instance, in case of case-change only rename, short name will now be preserved.
See ya, you 7y old bug : - )
Re: Fix of the Week
Posted: Thu Jul 28, 2016 10:15 am
by Black_Fox
r72030 adds the ability to mouse scroll the console window:
[CONSRV]
- Do not hardcode the values of some flags used with GetKeyState.
- Rework a bit the 'OnScroll' function.
- Implement mouse scrolling the console.
CORE-9900 #resolve
CORE-8394
- In mouse selection mode, allow the user to enlarge the current selection area by pressing the SHIFT key together with clicking on some point.
Re: Fix of the Week
Posted: Thu Sep 01, 2016 12:16 pm
by middings
I nominate
r72521 for Fix of the Week.
CORE-1578 is a 10-year old bug. A patch by Sebastian Gasiorek was committed by Jim Tabor.
Log Message:
[NtGDI]
- Patch by Sebastian Gasiorek : Fix incorrect placement of the text in the Tab buttons.
Implement TA_UPDATECP.
See
CORE-1578 and
CORE-11415.
Re: Fix of the Week
Posted: Wed Sep 21, 2016 1:10 am
by EmuandCo
r72750 committed by AmineKhaldi:
[WS2_32] The current ws2_32 served us well, but it's time for ws2_32_new to shine. It took slightly more than 7 years since Ged brought-in Alex' impressive work on this module, and thanks to the great follow-up work by Peter Hater, Andreas Maier, Thomas Faber and everyone else involved in the effort, we now have a chance to retire ws2_32 in favor of ws2_32_new. The test results speak for the excellent achievements, and this serves as a platform to continue inspecting and fixing the remaining test failures. Thank you all for a job well done! Please see
CORE-10440 for a summary of the changes performed in this commit.
HELLO PROPER WINSOCK
Re: Fix of the Week
Posted: Sat Oct 22, 2016 6:00 am
by middings
revision 72867 is my nomination for Fix of the Week during the week of 25 September through 1 October 2016. A patch by Peter Hater was committed by Amine Khaldi. This patch improved DNS facilities in ReactOS and resolved 3-year old bug
CORE-7441 and
CORE-11394. These bugs affected browser programs.
The log message says:
[ADNS] [DNSAPI] [MSWSOCK] Several fixes and improvements to DnsQuery and co, detailed in CORE-11394. This also fixes CORE-7441, many tests in dnsapi:DnsQuery, ws2_32:getaddrinfo and more. Brought to you by Peter Hater.
Honorable mention goes to all the testers and developers who contributed to identifying the problem, investigating its cause, and testing its proposed fixes.
Re: Fix of the Week
Posted: Wed Oct 26, 2016 4:12 pm
by hbelusca
I nominate
revision 73039 (and the subsequent commits r73040 and 73041) as:
The Commit of The Month!!
Revision 73039
Author: cfinck
Date: Wed Oct 26 12:37:27 2016 UTC
Log Message:
[PRINTING]
Finally merge my branch "colins-printing-for-freedom", giving us an initial implementation of a Win32-compatible Printing Stack (localmon, localspl, spoolss, spoolsv, winspool)
You can now send raw data to parallel port printers using documented Win32 API. An example application is in my "winspool_print" commandline tool merged to rosapps.
ReactOS folks, thanks for your support during the development of this, making my bachelor's thesis a reality!
Documentation/Thesis:
https://svn.reactos.org/reactos/trunk/d ... thesis.pdf
Video:
https://www.youtube.com/watch?v=cNzePucTOLY
CORE-10489
Re: Fix of the Week
Posted: Wed Oct 26, 2016 5:01 pm
by EmuandCo
/me wets himself
Re: Fix of the Week
Posted: Wed Dec 07, 2016 3:18 am
by ROCKNROLLKID
I think this deserves fix of the week. Ultimately, it is part 2 of getting Word 2010 fixed. Congrats to hbelusca.
https://jira.reactos.org/browse/CORE-12414
Re: Fix of the Week
Posted: Fri Dec 23, 2016 12:37 am
by ROCKNROLLKID
I nominate Jimtabor for fixing this almost 10 year old bug:
https://jira.reactos.org/browse/CORE-2201
Re: Fix of the Week
Posted: Sat Dec 24, 2016 3:53 am
by manuel
Re: Fix of the Week
Posted: Sun Dec 25, 2016 4:57 am
by ROCKNROLLKID
manuel wrote:
was fixed this bug?
Yes, revision 73475 included the fix.
Re: Fix of the Week
Posted: Wed Jan 25, 2017 5:28 pm
by milon
I nominate both of cfinck's recent commits for fix of the week/
month! Hybrid live/install is on the way!!
https://svn.reactos.org/svn/reactos?vie ... sion=73594
r73594 wrote:
Author: cfinck
Date: Tue Jan 24 22:45:37 2017 UTC (16 hours, 39 minutes ago)
Log Message:
[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/8bb ... olinux.asm (latest version to date) and
http://repo.or.cz/syslinux.git/blob/914 ... olinux.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
https://svn.reactos.org/svn/reactos?vie ... sion=73595
r73595 wrote:
Author: cfinck
Date: Tue Jan 24 22:53:20 2017 UTC (16 hours, 31 minutes ago)
Log Message:
[ISOHYBRID]
Import the isohybrid tool from
http://repo.or.cz/syslinux.git/tree/138 ... 3a2:/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