18.07.2017 09:09, Philippe Mathieu-Daudé wrote:
> @@ -151,7 +151,7 @@ static int print_insn_objdump(bfd_vma pc,
> disassemble_info *info,
> info->read_memory_func(pc, buf, n, info);
>
> for (i = 0; i < n; ++i) {
> - if (i % 32 == 0) {
> + if (QEMU_IS_ALIGNED(i, 32)) {
> info->fprintf_func(info->stream, "\n%s: ", prefix);
> }
> info->fprintf_func(info->stream, "%02x", buf[i]);
This does not seem to be related to _alignment_ per se, it is
just formatting, 32 entries per line, so that output looks
more-or less compact and stays readable. To me it is a
strange move, not so much logical.
It doesn't matter much either way, but with IS_ALIGNED, to me
at least, it looks a bit less logical than now.
Thanks,
/mjt
