On Tue, 16.07.13 20:39, Lennart Poettering ([email protected]) wrote:

> > +                                goto finish;
> > +                        }
> > +
> > +                        n = strappend("/dev/", unescaped_devname);
> 
> Good candidate for strappenda?

I take this back. Awful candidate for strappenda() since this is inside
a loop, and you cannot sanely use alloca() in a loop, since it keeps
appending to the stack.

So, please keep using strappend() for this one.

Lennart

-- 
Lennart Poettering - Red Hat, Inc.
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to