[patch] cygcheck.cc update for cygpath()
Corinna Vinschen
corinna-cygwin@cygwin.com
Sun Mar 16 15:11:00 GMT 2008
On Mar 15 06:49, Brian Dessent wrote:
> Anyway, the attached is the result of what happened when I started with
> the Cygwin code and whittled it down. It fixes the bug in the
> grandparent of this email where it was reading the Win32 path out of a
> shortcut that didn't have an ITEMIDLIST, and it supports the new-style
> shortcut where the target > MAX_PATH gets stored at the end. It does
> not attempt to do anything with reparse points however.
That's ok. We won't use reparse points for our own dreadful purposes
anyway.
> * path.cc: Include malloc.h for alloca.
> (is_symlink): Rewrite. Just read the whole file in memory rather
> than by parts. Account for an ITEMIDLIST if present, as well as
> the new style of Cygwin shortcut supporting targets > MAX_PATH.
The patch looks ok to me. Please check it in.
Thanks,
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat
More information about the Cygwin-patches
mailing list