Re: Linux loader for the PPC

[Jeroen Dekkers]

At Tue, 13 Jul 2004 21:03:09 +0200,
Marco Gerards wrote:
> 
> Jeroen Dekkers <address@hidden> writes:
> 
> > At Tue, 13 Jul 2004 19:01:22 +0200,
> > Marco Gerards wrote:
> >> +static void
> >> +grub_rescue_cmd_linux (int argc, char *argv[])
> >> +{
> >> +  grub_load_linux (argc, argv);
> >> +}
> >> +
> >> +GRUB_MOD_INIT
> >> +{
> >> +  grub_rescue_register_command ("linux", grub_rescue_cmd_linux,
> >> +                          "load a linux kernel");
> >> +  my_mod = mod;
> >> +}
> >
> > Why don't you just use grub_load_linux as argument to
> > grub_rescue_register_command()?
> 
> Because that will make it harder to change the interface of
> grub_load_linux afterwards.

Why? It's in the same file as the register command. If it gets a
different interface, you can update the register call with it to
point to some new function that wraps is.

Jeroen Dekkers