On 6 July 2012 03:00, Peter Crosthwaite <peter.crosthwa...@petalogix.com> wrote: > On Fri, Jul 6, 2012 at 3:00 AM, Peter Maydell <peter.mayd...@linaro.org> > wrote: >> @@ -371,7 +371,8 @@ void arm_load_kernel(ARMCPU *cpu, struct arm_boot_info >> *info) >> initrd_size = load_image_targphys(info->initrd_filename, >> info->loader_start >> + INITRD_LOAD_ADDR, >> - ram_size - INITRD_LOAD_ADDR); >> + info->ram_size >> + - INITRD_LOAD_ADDR); > > No blocker, but can we de-indent one or two tabs, to not have to have > this arg split this across so many lines?
The general style at least in ARM related QEMU code is that function arguments always line up (ie indent to just after the opening paren). This is what the older code written by Paul does and it's also what my editor does. I prefor not to break that convention. (I agree this looks a little ugly, but that's what 80-column restrictions get you.) -- PMM
