diff options
| author |   Mike Frysinger <vapier@gentoo.org> | 2016-01-03 21:40:34 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2016-01-03 22:07:39 -0500 |
| commit | 77cf2ef5dc9099501529151921a73be904757466 (patch) | |
| tree | afe55fc45fb53886d73ed81e984f83222ab759a7 /sim/avr | |
| parent | sim: TODO: move to wiki (diff) | |
| download | binutils-gdb-77cf2ef5dc9099501529151921a73be904757466.tar.gz binutils-gdb-77cf2ef5dc9099501529151921a73be904757466.tar.bz2 binutils-gdb-77cf2ef5dc9099501529151921a73be904757466.zip | |
sim: parse_args: display getopt error ourselves
Fix a long standing todo where we let getopt write directly to stderr
when an invalid option is passed. Use the sim io funcs instead as they
go through the filtered callbacks that gdb wants.
Diffstat (limited to 'sim/avr')
| -rw-r--r-- | sim/avr/ChangeLog | 4 | ||||
| -rw-r--r-- | sim/avr/interp.c | 4 |
2 files changed, 5 insertions, 3 deletions
diff --git a/sim/avr/ChangeLog b/sim/avr/ChangeLog index 71075540e2f..27d1e271608 100644 --- a/sim/avr/ChangeLog +++ b/sim/avr/ChangeLog @@ -1,5 +1,9 @@ 2016-01-03 Mike Frysinger <vapier@gentoo.org> + * interp.c (sim_open): Update sim_parse_args comment. + +2016-01-03 Mike Frysinger <vapier@gentoo.org> + * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete. * configure: Regenerate. diff --git a/sim/avr/interp.c b/sim/avr/interp.c index 91b97928b83..f33de1cfa70 100644 --- a/sim/avr/interp.c +++ b/sim/avr/interp.c @@ -1704,9 +1704,7 @@ sim_open (SIM_OPEN_KIND kind, host_callback *cb, struct bfd *abfd, char **argv) return 0; } - /* getopt will print the error message so we just have to exit if this fails. - FIXME: Hmmm... in the case of gdb we need getopt to call - print_filtered. */ + /* The parser will print an error message for us, so we silently return. */ if (sim_parse_args (sd, argv) != SIM_RC_OK) { free_state (sd); |