Re: [2/5] ntdll: Make NtWriteFile handle special offset -2 same way as -1. Resend.

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> dlls/ntdll/file.c | 31 +++++++++++++++----------------
>  1 file changed, 15 insertions(+), 16 deletions(-)

Why do you need that?  It doesn't look to me like an improvement.

[2/5] ntdll: Make NtWriteFile handle special offset -2 same way as -1. Resend.

[Dmitry Timoshkov]

Alexandre Julliard  wrote:
> Why do you need that?  It doesn't look to me like an improvement.

IMO it simplifies the code and avoids using two different code paths
for normal files.

[2/5] ntdll: Make NtWriteFile handle special offset -2 same way as -1. Resend.

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> Alexandre Julliard  wrote:
> > Why do you need that?  It doesn't look to me like an improvement.
>
> IMO it simplifies the code and avoids using two different code paths
> for normal files.

At the cost of two extra seeks, plus introducing race conditions.
I don't think it's a good idea.