[PATCH 1/3] Cygwin: dll_list: drop FILE_BASIC_INFORMATION
Corinna Vinschen
corinna-cygwin@cygwin.com
Tue Apr 30 16:38:00 GMT 2019
On Apr 30 18:13, Corinna Vinschen wrote:
> On Apr 30 16:14, Michael Haubenwallner wrote:
> > Querying FILE_BASIC_INFORMATION is needless since using win pid+threadid
> > for forkables dirname rather than newest last write time.
> > ---
> > winsup/cygwin/dll_init.cc | 1 -
> > winsup/cygwin/dll_init.h | 1 -
> > winsup/cygwin/forkable.cc | 7 +++----
> > 3 files changed, 3 insertions(+), 6 deletions(-)
> > [...]
> > - if (!dll_list::read_fii (fhandle, &d->fii) ||
> > - !dll_list::read_fbi (fhandle, &d->fbi))
>
> Given this is the only place calling dll_list::read_fbi, the method
> and declaration should be removed, too.
>
> > + if (!dll_list::read_fii (fhandle, &d->fii))
> > system_printf ("WARNING: Unable to read real file attributes for %W",
> > pmsi1->SectionFileName.Buffer);
> >
> > NtClose (fhandle);
> > - return d->fbi.FileAttributes != INVALID_FILE_ATTRIBUTES;
> > + return d->fii.IndexNumber.QuadPart != -1LL;
> > }
> >
> > /* easy use of NtOpenFile */
> > --
> > 2.19.2
I pushed this patch series as is now, but I have two requests:
- Please send a followup patch to remove the unused method.
- When sending patchsets, would you mind to send them with cover letter
per `git format-patch --cover-letter'? It's not a hard requirement,
but it makes discussing single patches vs. patchsets clearer.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20190430/ae20090e/attachment.sig>
More information about the Cygwin-patches
mailing list