[systemd-devel] [PATCH] continuation support in envinronment files
Lennart Poettering
lennart at poettering.net
Tue Jan 15 19:45:11 PST 2013
On Fri, 11.01.13 15:03, Michal Vyskocil (mvyskocil at suse.cz) wrote:
Minor nitpick:
> + if (endswith(p, "\\")) {
> + *(p+strlen(p)-1) = '\0';
> +
> + if (!(c = strappend(b, p))) {
For new code we prefer
...
c = strappend(b, p);
if (!c) {
...
Rather then the on-line syntax.
Looks good otherwise (modulo the issues Zbigniew's raised).
Lennart
--
Lennart Poettering - Red Hat, Inc.
More information about the systemd-devel
mailing list