On Tue, May 09, 2000 at 12:42:48AM +1000, Bruce Evans wrote:
> On Fri, 5 May 2000, Joe Karthauser wrote:
>
> > I only received one reply to this so I'm guessing that no-one else minds.
> > [I know that we've all been preoccupied with restructuring documents ;)]
> >
> > It's been suggested the {g|s}etflagsbyname is a better set of
> > function names, and I'm inclined to agree. I'm going to commit
> > this next week, unless there are serious objections in advance.
>
> getflagsbyname() is a much worse name. getflags() gets a string
> representing the flags from the flags, i.e., it gets non-flags by number.
>
> "flags" is too generic. I would prefer names like fileflagstostr()
> and strtofileflags(), but shorter.
Hmm, hard isn't it. How much shorter would make sense. fflagstostr?
fflagstos fftos?
> If the names are changed, then the broken interfaces should be fixed:
> 1) getflags() returns the string in a static buffer.
> 2) The "def" arg for getflags() is too specialized for ls. The caller
> can easily translate from "" to whatever it wants to print if no
> flags are set. (ls prints "-" and mtree prints "none".)
I've already fixed these, but not committed yet.
Joe
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message