On Fri, Nov 09, 2007 at 10:17:19PM +0100, Marco Gerards wrote:
> Robert Millan <[EMAIL PROTECTED]> writes:
>
> > On Thu, Oct 25, 2007 at 09:51:38PM +0200, Christian Franke wrote:
> >> static grub_err_t
> >> -grub_host_open (const char *name __attribute((unused)), grub_disk_t disk)
> >> +grub_host_open (const char *name, grub_disk_t disk)
> >> {
> >> + if (grub_strcmp(name, "host"))
> >> + return grub_error (GRUB_ERR_UNKNOWN_DEVICE, "not a host disk");
> >> +
> >
> > I would test for (disk->id == GRUB_DISK_DEVICE_HOST_ID) instead. Faster,
> > and also cleaner/simpler IMHO.
>
> It's not possible unfortunately :-(. This information is about to be
> filled in in this same function.
Still seems like an ugly hack to me. Oh well :-/
> Besides that, you have grub_disk_dev
> in mind.
Ouch ;-)
--
Robert Millan
<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call, if you are unable to speak?
(as seen on /.)
_______________________________________________
Grub-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/grub-devel
