On Wed, Dec 31, 2014 at 04:16:02PM +0200, Philippe De Swert wrote: > The variable r is already checked one time before and the same error returned. > So the second check is unneeded. > --- > src/machine/machinectl.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c > index 0abc251..951b3cc 100644 > --- a/src/machine/machinectl.c > +++ b/src/machine/machinectl.c > @@ -250,10 +250,6 @@ static int list_images(int argc, char *argv[], void > *userdata) { > (int) max_mtime, strna(format_timestamp(mtime_buf, > sizeof(mtime_buf), images[j].mtime))); > } > > - if (r < 0) > - return bus_log_parse_error(r); > - > - David pushed the same change yesterday.
Zbyszek _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel