This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] sim: Be sure of calling freeargv() after successfully call buildargv().
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Chen Gang S <gang dot chen at sunrus dot com dot cn>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 29 Jan 2015 08:49:11 +0400
- Subject: Re: [PATCH] sim: Be sure of calling freeargv() after successfully call buildargv().
- Authentication-results: sourceware.org; auth=none
- References: <54C8CBC8 dot 90102 at sunrus dot com dot cn>
[binutils does not need to be copied in this case, as you're only
touching files in the simulator, which is part of the GDB project]
> 2015-01-28 Chen Gang <gang.chen.5i5j@gmail.com>
>
> * common/sim-options.c (sim_args_command): Call freeargv() when
> failure occurs.
> * mcore/interp.c (sim_do_command): Call freeargv() before return.
> * microblaze/interp.c (sim_do_command): Call freeargv() before
> return.
Small procedural request, Chen. Those 3 changes are pretty much
independent, so it's highly preferable to submit them separately.
This has a number of advantages: We can review each one of them
individually, with possibly different reviewers, and that makes
tracking of which part has been reviewed a lot easier. Also, by
having them submitted separately, you can have one patch per piece,
which means that if one patch turns out to be incorrect, we can
easily revert just that patch using git, rather than doing a semi-
revert by hand.
--
Joel