[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