[patch 5/6] Move linux_find_memory_regions_full & co.

Aleksandar Ristovski aristovski@qnx.com
Wed Mar 27 21:25:00 GMT 2013


Moves should now be clean, without mods.

Thanks,

Aleksandar


On 13-03-26 12:58 PM, Jan Kratochvil wrote:
> On Fri, 22 Mar 2013 14:04:16 +0100, Aleksandar Ristovski wrote:
>> --- a/gdb/target.c
>> +++ b/gdb/target.c
>> @@ -3477,69 +3477,6 @@ target_fileio_read_alloc_1_pread (int handle, gdb_byte *read_buf, int len,
> [...]
>> -read_alloc (gdb_byte **buf_p, int handle, read_alloc_pread_ftype *pread_func,
>> -	    int padding, void **memory_to_free_ptr)
> [...]
>> -	  buf = xrealloc (buf, buf_alloc);
>> -	}
>
> vs.
>
>> --- a/gdb/common/common-utils.c
>> +++ b/gdb/common/common-utils.c
>> @@ -348,3 +348,95 @@ skip_spaces_const (const char *chp)
> [...]
>> +read_alloc (gdb_byte **buf_p, int handle, read_alloc_pread_ftype *pread_func,
>> +	    int padding, void **memory_to_free_ptr)
> [...]
>> +	  buf = xrealloc (buf, buf_alloc);
>> +	  if (memory_to_free_ptr != NULL)
>> +	    *memory_to_free_ptr = buf;
>> +	}
>
> This is not a move, one does not see what has really changed in the diffs.
>
>
>
>> -read_stralloc (const char *filename, read_stralloc_func_ftype *func)
>> -{
>> -  gdb_byte *buffer;
>> -  char *bufstr;
>> -  LONGEST i, transferred;
>> -
>> -  transferred = func (filename, &buffer, 1);
>
> vs.
>
>> +read_stralloc (const char *filename, read_stralloc_func_ftype *func)
>> +{
>> +  char *buffer;
>> +  LONGEST i, transferred;
>> +
>> +  transferred = func (filename, (gdb_byte **) &buffer, 1);
>
> Likewise this is not just a move.
>
>
> One cannot review it before picking each function and comparing them by hand,
> this is why the patchset was split into the change + move parts, but you made
> changes in the 'move' parts without updating the 'change' parts.
>
> Please update the patchset.  Going to look at the real code changes of the
> last parts.
>
>
> Thanks,
> Jan
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-Move-linux_find_memory_regions_full-co.patch
Type: text/x-patch
Size: 19737 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20130327/e5be466f/attachment.bin>


More information about the Gdb-patches mailing list