| Message ID | 20180531052915.31171-10-rashmica.g@gmail.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | [01/10] libpdbg: Print the name of the instruction when erroring | expand |
On Thursday, 31 May 2018 3:29:15 PM AEST Rashmica Gupta wrote: > --- > src/main.c | 2 +- > src/reg.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/main.c b/src/main.c > index e714377..3df9a4e 100644 > --- a/src/main.c > +++ b/src/main.c > @@ -739,7 +739,7 @@ int main(int argc, char *argv[]) > return 1; > > found_action: > - if (rc <= 0) { > + if (rc != 0) { Sadly this isn't so straight forward. Currently a lot of the actions (eg. getscom) return the number of targets they were able to successfully execute an action on. The idea being that the below message could be printed in cases when rc == 0 which indicates nothing was actually executed. Admitedly the message might not be very useful, but it is better than exiting without printing anything. However I agree our inconsistent return codes are a pretty nasty hack that should die, just that it will require changes to all the actions. - Alistair > printf("No valid targets found or specified. Try adding -p/-c/-t options to specify a target.\n"); > printf("Alternatively run '%s -a probe' to get a list of all valid targets\n", > basename(argv[0])); > diff --git a/src/reg.c b/src/reg.c > index 0a87006..e58275f 100644 > --- a/src/reg.c > +++ b/src/reg.c > @@ -103,7 +103,7 @@ static int getprocreg(struct pdbg_target *target, uint32_t index, uint64_t *reg, > > print_proc_reg(target, *reg, value, rc); > > - return !rc; > + return rc; > } > > int handle_gpr(int optind, int argc, char *argv[]) >
diff --git a/src/main.c b/src/main.c index e714377..3df9a4e 100644 --- a/src/main.c +++ b/src/main.c @@ -739,7 +739,7 @@ int main(int argc, char *argv[]) return 1; found_action: - if (rc <= 0) { + if (rc != 0) { printf("No valid targets found or specified. Try adding -p/-c/-t options to specify a target.\n"); printf("Alternatively run '%s -a probe' to get a list of all valid targets\n", basename(argv[0])); diff --git a/src/reg.c b/src/reg.c index 0a87006..e58275f 100644 --- a/src/reg.c +++ b/src/reg.c @@ -103,7 +103,7 @@ static int getprocreg(struct pdbg_target *target, uint32_t index, uint64_t *reg, print_proc_reg(target, *reg, value, rc); - return !rc; + return rc; } int handle_gpr(int optind, int argc, char *argv[])