This is the mail archive of the libc-hacker@sources.redhat.com mailing list for the glibc project.
Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
On Fri, Sep 29, Ulrich Drepper wrote:
> Jakub Jelinek <jakub@redhat.com> writes:
>
> > --- libc/nscd/nscd_gethst_r.c.jj Wed May 3 10:19:59 2000
> > +++ libc/nscd/nscd_gethst_r.c Fri Sep 29 11:30:57 2000
> > @@ -165,7 +165,7 @@ nscd_gethst_r (const char *key, size_t k
> > if (hst_resp.found == 1)
> > {
> > struct iovec vec[4];
> > - size_t *aliases_len;
> > + nscd_size_t *aliases_len;
> > char *cp = buffer;
> > uintptr_t align1;
> > uintptr_t align2;
>
> This is perfectly fine. There is no real need for using size_t. I
> would perhaps just using uint32_t and int32_t instead of introducing
> another type. I'll probably make this change in your patch.
> Otherwise people will ask why such a named type is necessry.
It is necessary to be binary compatible on Alpha. Or we need to
bump the nscd version number.
Thorsten
--
Thorsten Kukuk http://www.suse.de/~kukuk/ kukuk@suse.de
SuSE GmbH Schanzaeckerstr. 10 90443 Nuernberg
Linux is like a Vorlon. It is incredibly powerful, gives terse,
cryptic answers and has a lot of things going on in the background.
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |