Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch



On May 19, 11:28pm, kleink%kleink.org@localhost (Klaus Klein) wrote:
-- Subject: Re: CVS commit: src/sys/arch

| What's the purpose of this?  ssize_t is meant to be "a count of
| bytes or an error indication", so having SSIZE_MIN doesn't make
| sense.  Nor does its only in-tree, broken use in libevent; searching
| outside our tree I wasn't able to come up with a use of EV_SSIZE_MIN
| except for the aforementioned regression test.

The purpose of this is to make it easier to query the range of the type.
(ssize_t)-1 would work too.

| (I'll be the first to admit I'm somewhat late in voicing my concerns,
| but I was ... busy.  *sigh*)

Heh. It would have been easier if that was a bit sooner, so I could
remember the exact situation. From what I can remember, I searched for
it, I saw that others had it too (linux, etc.) so I decided to add it.

christos


Home | Main Index | Thread Index | Old Index