ntdll: fix read_directory_getdirentries() under FreeBSD.

André Hentschel nerv at dawncrow.de
Mon Dec 29 07:32:20 CST 2014


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am 28.12.2014 um 21:09 schrieb David Naylor:
> 
> Change the handling of getdirentries(2) by: - tracking the data pointer instead of subtracting 'res' - simplify rewind by skipping re-appending entries onto the list - try make the logic easier to follow --- dlls/ntdll/directory.c | 73 +++++++++++++++++++------------------------------- 1 file changed, 28 insertions(+), 45 deletions(-)

Hi and welcome to Wine development,
maybe it's just me, but the patch is not easy to review.
Is it possible to split the patch into multiple atomic patches (maybe three, as in the change list you provided)?
Also, how does the patch fix Wine on FreeBSD?
A quick look at the patch itself:
You missed a whitespace at the last lseek call.
You're adding a FIXME comment, but you should rather clarify your question, use wine-devel and/or IRC and/or "git blame" for that.


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQGcBAEBAgAGBQJUoVfbAAoJEGm5GZTakYssGkoMAIfOM3ulOIh28PKA3RS0IByC
sEk8AerEkKyN2nS4d2hNRU/kHIFlW+wHfi8Ah7FhBGepZ0HIHrWqZhuMmZsYx1W/
MeYyW+bROa+uCGWYvz7w9aaXW+xOIKDaJ7mIQrJKnkMgOQ3FFI8yKa6tRSSAm6iA
Xg2porB96Ov+lLcl6zq0UATuVgqENhzSRrMxHh/YhVwK3xO7hSQ4ysgMJj2cBg7A
XX7cEdxys1X5kSAO10I/gucmtKe3GrQi++DWx6VUd1O4WIEtm86c1gO2UtbUxYNf
p9q5feajTJLscWOKpaC032fLrfFcnkhSS1ZwyIl7TfICaWW6u+8zfrcrVG8xfPkR
MfbCRrJpkCFXsa5bYJLTPdD+0LETPlJ48zkyRyny56XYwgkBlgPirVlCeYnSZs9t
CCOaHYYrMePAyB1DLZeJIdCCmyy8gLlY9cu28SgbN/oaHuYtwfrPbXKt30fwhZyM
cOcTZnZZuYSeb5oVt72O+j0GbkSW2x+juFuVhyBvlw==
=RACU
-----END PGP SIGNATURE-----



More information about the wine-devel mailing list