On 10 December 2010 20:26, Brian C. Lane <b...@redhat.com> wrote:
> +#define DEFAULT_ALIGNMENT       1048576

Could we instead have this as

#define DEFAULT_ALIGNMENT       (1024 * 1024)

because to me 1048576 appears to be a magic number
and while I understand that this number is highly unlikely
to be anything other than 1MB, it is not obvious by just
reading the code and I would have to check with a
calculator to be 100% sure, whereas "1024 * 1024"
is obvious and without possibilities of misunderstanding.

BR Håkon Løvdal

_______________________________________________
bug-parted mailing list
bug-parted@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-parted

Reply via email to