[U-Boot] [PATCH] MAKEALL: Add -l option to only list build targets
Mike Frysinger
vapier at gentoo.org
Sat Dec 3 10:19:53 CET 2011
On Saturday 03 December 2011 03:23:28 Marek Vasut wrote:
> > On Saturday 03 December 2011 02:32:03 Marek Vasut wrote:
> > > + if [ "$ONLY_LIST" == 'y' ] ; then return ; fi
> >
> > [ "$ONLY_LIST" = "y" ] && return
>
> I prefer to be explicit.
my version is "explicit" as well
the point was more that if statements shouldn't be one-liners like that. if
you want to keep it, then unwrap the block.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20111203/2d6d2cc1/attachment.pgp>
More information about the U-Boot
mailing list