[U-Boot] [PATCH 3/3] sandbox: add getopt support

Mike Frysinger vapier at gentoo.org
Mon Feb 27 05:35:10 CET 2012


On Sunday 26 February 2012 23:26:52 Simon Glass wrote:
> On Sun, Feb 26, 2012 at 8:20 PM, Mike Frysinger wrote:
> > On Sunday 26 February 2012 21:46:23 Simon Glass wrote:
> >> On Sun, Feb 26, 2012 at 2:38 PM, Mike Frysinger wrote:
> >> >  int main(int argc, char *argv[])
> >> >  {
> >> > ...
> >> > +       state = state_get_current();
> >> > +       os_parse_args(state, argc, argv);
> >> 
> >> We don't check the return value. Perhaps add a comment as to why.
> > 
> > since the code takes care of setting parse_err itself now, i'm not sure
> > what to do with the return value
> 
> I agree it is right, just asking for a comment. Same with most of my
> other things - more comments as I suggested is nice for people that
> come into the code fresh.

my plan was to clean this up a bit more before submitting (such as adding 
comments).  i was looking more for feedback on the general approach here and 
any fundamental sticking points since this is a semi-radical departure from 
what either of us posted earlier.

in this case, i'm asking: should we just toss the return value and have it be 
void ?
-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/20120226/07ccc8b1/attachment.pgp>


More information about the U-Boot mailing list