[U-Boot] [PATCH v3] mkimage: add "-V" option to print version information

[Kim Phillips]

On Sat, 12 Feb 2011 10:37:16 +0100
Wolfgang Denk <wd at denx.de> wrote:

> Dear Kim Phillips,
> 
> In message <20110211171117.b9b05b01.kim.phillips at freescale.com> you wrote:
> > On Fri, 11 Feb 2011 23:41:43 +0100
> > Wolfgang Denk <wd at denx.de> wrote:
> > 
> > > +			case 'V':
> > > +				/*
> > > +				 * Skip the "U-Boot " part in
> > > +				 * U_BOOT_VERSION by adding 7
> > > +				 */
> > > +				printf("mkimage version %s\n",
> > > +					U_BOOT_VERSION + 7);
> > 
> > I'd have done it without magic nor comments as
> > 
> > U_BOOT_VERSION[sizeof("U-Boot ") + 1]
> > 
> > or even
> > 
> > U_BOOT_VERSION[strlen("U-Boot ")]
> 
> Hm... I would have cleaned up such locations in U-Boot as part of my
> v4 patch that introduces PLAIN_VERSION - if I had found any such
> code. Am I missing something?

ok, apart from the missing & and +/- gaffes, what are you talking
about?  The code does exactly what the + 7 does, except it's more
self-explanatory.

Kim