This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
Re: Patch to allow trailing dots on managed mounts
On Thu, Dec 16, 2004 at 04:53:39PM +0100, Corinna Vinschen wrote:
>On Dec 16 10:00, Christopher Faylor wrote:
>> On Thu, Dec 16, 2004 at 09:55:48AM -0500, Pierre A. Humblet wrote:
>> >Here is an untested patch.
>> >I hope Mark can test it (on managed and unmanaged mounts,
>> >including basenames consisting entirely of dots and spaces)
>> >and possibly make adjustments, without having to file the
>> >paperwork.
>> >
>> >Pierre
>> >
>> > * path.cc (path_conv::check): Do not strip trailing dots and spaces.
>> > * fhandler.cc (fhandler_base::open): Strip trailing dots and spaces.
>>
>> Is it correct to assume that only fhandler_base::open cares about
>> trailing dots?
>
>So far, yes. But somehow moving this code into open() looks a bit like
>a step backwards to me. The general direction is to do more stuff using
>NT functions, isn't it? So in the long run we would get the same problem
>in other code paths as well.
That was my thought as well.
>Since the mount code is called from path_conv anyway, wouldn't it be
>better to pass the information "managed mount or not" up to path_conv?
How about just doing the pathname munging in `conv_to_win32_path' if/when
it's needed?
cgf