( end of the message) , and its more elegant than many (str[i] !=
prefix[i] || ) instructions.
Maybe youre criticizing this change because of performances reasons ?
Cheers
Hermès
De : Hermès BÉLUSCA - MAÏTO [mailto:hermes.belusca@sfr.fr]
Envoyé : lundi 3 juin 2013 11:10
À : 'ReactOS Development List'
Objet : RE: [ros-dev] [ros-diffs] [hbelusca] 59165: [RTL] Use
RtlPrefixUnicodeString.
It was suggested by Thomas Faber, during some fixes of path.c
De : ros-dev-bounces(a)reactos.org [mailto:ros-dev-bounces@reactos.org] De la
part de Alex Ionescu
Envoyé : lundi 3 juin 2013 04:33
À : ReactOS Development List
Cc : Linda Wang
Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 59165: [RTL] Use
RtlPrefixUnicodeString.
You should realize there's sometimes a reason why code doesn't use an
existing helper function. Among some very good bug fixes and patches, you
really seem to love changing code just for the sake of changing code.
Best regards,
Alex Ionescu
On Sun, Jun 2, 2013 at 3:51 PM, <hbelusca(a)svn.reactos.org> wrote:
Author: hbelusca
Date: Sun Jun 2 22:51:24 2013
New Revision: 59165
URL: http://svn.reactos.org/svn/reactos?rev=59165
<http://svn.reactos.org/svn/reactos?rev=59165&view=rev> &view=rev
Log:
[RTL]
Use RtlPrefixUnicodeString.
Modified:
trunk/reactos/lib/rtl/path.c
Modified: trunk/reactos/lib/rtl/path.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/path.c?rev=59165
<http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/path.c?rev=59165&r
1=59164&r2=59165&view=diff> &r1=59164&r2=59165&view=diff
============================================================================
==
--- trunk/reactos/lib/rtl/path.c [iso-8859-1] (original)
+++ trunk/reactos/lib/rtl/path.c [iso-8859-1] Sun Jun 2 22:51:24
2013
@@ -479,13 +479,19 @@
/* Capture input string */
CapturedDosName = *DosName;
- /* Check for \\?\ form */
- if ((CapturedDosName.Length <= RtlpWin32NtRootSlash.Length) ||
- (CapturedDosName.Buffer[0] != RtlpWin32NtRootSlash.Buffer[0]) ||
- (CapturedDosName.Buffer[1] != RtlpWin32NtRootSlash.Buffer[1]) ||
- (CapturedDosName.Buffer[2] != RtlpWin32NtRootSlash.Buffer[2]) ||
- (CapturedDosName.Buffer[3] != RtlpWin32NtRootSlash.Buffer[3]))
- {
+ /* Check for the presence of the NT prefix "\\?\" form */
+ if (RtlPrefixUnicodeString(&RtlpWin32NtRootSlash, &CapturedDosName,
FALSE))
+ {
+ /* NT prefix is contained in the path */
+
+ /* Use the optimized path after acquiring the lock */
+ QuickPath = TRUE;
+ NewBuffer = NULL;
+ }
+ else
+ {
+ /* NT prefix is not contained in the path */
+
/* Quick path won't be used */
QuickPath = FALSE;
@@ -497,12 +503,6 @@
NewBuffer = RtlAllocateHeap(RtlGetProcessHeap(), 0, MaxLength);
DPRINT("MaxLength: %lx\n", MaxLength);
if (!NewBuffer) return STATUS_NO_MEMORY;
- }
- else
- {
- /* Use the optimized path after acquiring the lock */
- QuickPath = TRUE;
- NewBuffer = NULL;
}
/* Lock the PEB and check if the quick path can be used */
Revisions >= 59161 are fine (before starting with 59152 they are broken
- known).
This is only due to a VMware testbot internal failure. This is now fixed.
Sorry for the flood.
On 03/06/2013 00:28, buildbot(a)reactos.org wrote:
> The Buildbot has detected a failed build on builder Linux_AMD64_2 VMWPlayer-Test while building ReactOS.
> Full details are available at:
> http://build.reactos.org/builders/Linux_AMD64_2%20VMWPlayer-Test/builds/985
>
> Buildbot URL: http://build.reactos.org/
>
> Buildslave for this Build: Linux_AMD64_2
>
> Build Reason: Triggerable(Linux_AMD64_1 KVM-Test Trigger)
> Build Source Stamp: 59161
> Blamelist:
>
> BUILD FAILED: failed test
>
> sincerely,
> -The Buildbot
>
>
>
>
--
Pierre Schweitzer<pierre at reactos.org>
System Administrator
ReactOS Foundation
Hi,
the Linux builder RosBE has been upgraded to latest 2.1. It keeps using ninja for builds.
With my best regards,
--
Pierre Schweitzer <pierre(a)reactos.org>
System Administrator
ReactOS Foundation
Hello,
Let me invite you to the monthly status meeting taking place last
Thursday of this month, 30th of May, 19:00 UTC. And that's today!
The meeting will be at irc://dev.reactos.org (Port 6667, no SSL) in the
channel #meeting, or as Pierre says. Note that the IRC service will only
be started shortly before the meeting. Your participation passwords will
be emailed to you shortly before the meeting starts, and they are going
to be different once again as they are not stored in any database.
Hopefully it's not much of inconvenience.
If someone still is not getting passwords sent before a meeting - please
email Colin or Pierre before the meeting started to get one.
The agenda will be posted shortly before the meeting, suggestions are
welcome (send them to me shortly before the meeting starts).
Regards,
Aleksey Bragin
Hi,
Please note that r59047 broke VMware testbot. It times out in
setupapi:install and fails to restart afterwards.
Regards,
Pierre
On 20/05/2013 15:01, buildbot(a)reactos.org wrote:
> The Buildbot has detected a failed build on builder Linux_AMD64_2 VMWPlayer-Test while building ReactOS.
> Full details are available at:
> http://build.reactos.org/builders/Linux_AMD64_2%20VMWPlayer-Test/builds/869
>
> Buildbot URL: http://build.reactos.org/
>
> Buildslave for this Build: Linux_AMD64_2
>
> Build Reason: Triggerable(Linux_AMD64_1 KVM-Test Trigger)
> Build Source Stamp: 59050
> Blamelist:
>
> BUILD FAILED: failed test
>
> sincerely,
> -The Buildbot
>
>
>
>
--
Pierre Schweitzer<pierre at reactos.org>
System Administrator
ReactOS Foundation
The following tests are introducing several bugs/regressions:
[ros-diffs]
[ekohl] 59021: [SYSSETUP] Enable the restore privilege before the call
to NtUnloadKey. Disable it immediately after the call to NtUnloadKey.
ekohl at svn.reactos.org
^This one introduces a potential regression at:
user32:clipboard From 49216 tests executed, aftwerdars 49208 (-8)shell32:appbar. From 61 test executed, afterwards 58
[ros-diffs]
[ekohl] 59022: [FASTFAT] Allow only readonly, hidden, system and
archive files attributes to be set for existing files. Other file
attributes must be ignored. This fixes several tests.
ekohl at svn.reactos.org
^This introduces a whole bunch of regressions easily spoted.
ole32:storage32oleaut32:typelibregedit:regeditsetupapi:miscsetupapi:parserEtc,Etc
[ros-diffs]
[ekohl] 59023: [FASTFAT] Implement a special case for existing files
which are opened using the FILE_OVERWRITE or FILE_OVERWRITE_IF file
disposition: New file attributes must be ORed with the exist...
ekohl at svn.reactos.org
^The following regresses:
zz_notepad++_6.1.2:1.installzz_notepad++_6.1.2:2.SaveAsDesktopzz_notepad++_6.1.2:3.OpenDocumentzz_notepad++_6.1.2:4.Ctrl-F.FindDialogzz_notepad++_6.1.2:5.Send_InputAmong others
Comparison chart: http://www.reactos.org/sites/all/modules/reactos/testman/compare.php?ids=17…
Hi,
due to critical load issues on reactos.org, the webserver has been taken
down now for an extra maintenance to address the issues.
Affected services are the following: reactos.org, doxygen.reactos.org
(still reachable but slow), jira.reactos.org (no auth), code.reactos.org
(no auth).
We cannot provide any ETA for the end of the maintenance.
We are sorry for the caused inconveniences.
With my best regards,
--
Pierre Schweitzer<pierre at reactos.org>
System Administrator
ReactOS Foundation