[PATCH] Add devd config backend for FreeBSD (and GNU/kFreeBSD)

Guillem Jover guillem at debian.org
Wed Feb 26 22:13:31 PST 2014


On Mon, 2014-02-24 at 22:27:54 +0000, Robert Millan wrote:
> On 22/02/2014 16:12, Guillem Jover wrote:
> >> +++ b/config/devd.c
> >> @@ -0,0 +1,365 @@
> >> +/*
> >> + * Copyright ?? 2012 Baptiste Daroussin
> > 
> > Encoding issue? Should be ©, I guess.
> 
> It's © in the actual file - just checked.

It seems the attachment does not contain a charset attribute in its
Content-Type: field header. ISTR «git send-email» adds one for me.

> > realloc() can do the initial alloc if buf is NULL.
> 
> What would be the point in that?

It makes sure the code path leading to the realloc() is always exercised
and tested, because otherwise going over 1 KiB might not happen too
often. And reduces logic duplication.

> This change pessimizes on the common case by adding
> two extra in-loop branches, without actually saving any cost since
> the allocation needs to happen anyway.

You mean on a loop that is performing a syscall per byte, ok.

Anyway, I'm not the one who's going to merge such changes, so…

Thanks,
Guillem


More information about the xorg-devel mailing list