[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH parted 1/5] dasd: Cleanup anchor handling
From: |
Jim Meyering |
Subject: |
Re: [PATCH parted 1/5] dasd: Cleanup anchor handling |
Date: |
Wed, 04 Nov 2009 10:32:50 +0100 |
Hans de Goede wrote:
> The current dasd label code keeps an fdasd anchor struct in the
> DasdDiskSpecific struct, and fills this during dasd_read. However this
> anchor does not get updated with any future mods, until dasd_write,
> at which points it gets completely re-initialized.
Thanks for these patches.
My first pass at reviewing them spotted only a few nits:
- it requires a test (outline is fine, as before)
- any fix like that deserves an entry in NEWS;
Please add an entry for your new alignment-related functions, while
you're at it.
- the new index, "int i" (in 2/5) should be declared "unsigned int i".
Also, please move the declaration so it is right before the first use:
i.e., declare it on the line right before the for-loop.
In general, please move declarations you add or touch "down"
to be as close to first use as possible.