Robert Millan <[EMAIL PROTECTED]> writes: > On Tue, Jan 15, 2008 at 12:42:39PM -0500, Pavel Roskin wrote: >> >> On Tue, 2008-01-15 at 18:17 +0100, Robert Millan wrote: >> > - grub_dprintf ("mem", "Using memory for heap: addr=%p, size=%u\n", >> > addr, (unsigned int) size); >> > + grub_dprintf ("mem", "Using memory for heap: start=%p, end=%p\n", >> > addr, addr + size); >> >> Maybe addr+size-1 would be better? Inclusive boundaries are more >> intuitive. Just run "cat /proc/iomem" to see what I mean. > > Sounds better to me. > > Does this change (with addr + size -1) sound fine to everyone else?
Why not showing both the range and the size. Isn't the size useful sometimes? I hate calculating such things by hand... -- Marco _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel