Tom Lane wrote:
> Magnus Hagander <magnus@hagander.net> writes:
>> See attached patch which takes out the parts of fnmatch that we're not
>> interested in, and puts it directly in fe-secure.c. Obviously, if we go
>> down that way, we can remove fnmatch.c from port again :-)
>
>> Thoughts?
>
> Generally +1, but a couple of comments:
>
> * This seems to be still mostly NetBSD code, so I think you need to do
> more than just credit them in an aside. Should we repeat the full
> NetBSD copyright notice for this one function?
Do you mean the* Copyright (c) 1989, 1993, 1994* The Regents of the University of California. All rights
reserved.**This code is derived from software contributed to Berkeley by* Guido van Rossum.
part, or the whole licence? Since the licence is the same as ours, doing
that seems like overkill.
> * This is still making unjustified assumptions about the behavior of
> tolower/toupper. I think you probably want ASCII-only case folding,
> ie use pg_toupper/pg_tolower. If it actually should be locale aware
> then it's still wrong because it won't work in multibyte encodings.
> Also you forgot the de rigueur (unsigned char) casts for ctype.h calls.
Will fix.
//Magnus
