[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fix usability problem in FreeBSD loader
From: |
Vladimir 'phcoder' Serbinenko |
Subject: |
Re: [PATCH] fix usability problem in FreeBSD loader |
Date: |
Sat, 1 Aug 2009 15:49:15 +0200 |
On Sat, Aug 1, 2009 at 3:40 PM, Robert Millan<address@hidden> wrote:
>
> There's a minor usability problem with FreeBSD loader. E.g. if user runs
> freebsd_module first, the error message is confusing.
>
-static int kernel_type;
+static int kernel_type; /* 0 == KERNEL_TYPE_NONE */
I would prefer either
static int kernel_type = KERNEL_TYPE_NONE;
And it would be better to use an enum here rather than defines.
+ "You need to load the kernel first.");
+
if (kernel_type != KERNEL_TYPE_FREEBSD)
return grub_error (GRUB_ERR_BAD_ARGUMENT,
"only freebsd support module");
It seems that we have a bitrot in error message styles. We should
agree on one style and use it everywhere
Other than these two remarks patch is good.
> --
> Robert Millan
>
> The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
> how) you may access your data; but nobody's threatening your freedom: we
> still allow you to remove your data and not access it at all."
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>
>
--
Regards
Vladimir 'phcoder' Serbinenko
Personal git repository: http://repo.or.cz/w/grub2/phcoder.git