Re: [PATCH] Cleanup i386-nat.c

Eli Zaretskii
2004-02-29T06:05:19Z

> Date: Sat, 28 Feb 2004 16:38:23 +0100 (CET)
> From: Mark Kettenis <kettenis@chello.nl>
>
> The coding style in this file had some quirks, so I committed the
> attached.

That's largely my code, so please explain the changes, so that I
never repeat any mistakes I've committed. I've read the entire
patch, and I must say that I don't understand even a single change
you made. In the comments reformatting department, I guess we have
different setting for fill-column (what is the canonical one, btw?),
but as for the rest, I don't have a clue. So please do explain.

> - /* Four(eigth on x86_64) is the maximum length an x86 debug register
> + int align = addr % max_wp_len;
> + /* Four (eigth on AMD64) is the maximum length a debug register

Here, there's a real typo in the comment, but it wasn't fixed.