Re: [Patch] Scripting engine

2005-08-28 Thread Vladimir Serbinenko
>I'd like to see a kind of "design document", since it is a bit hard for others >to understand. > > Ok. I hope to write it this week (my studies begin tomorrow) > >We must wait until your assignment is finished. How is it going? > > > > No problem. I meant after assignment. I sent it back to

Re: backtrace support

2005-08-28 Thread Marco Gerards
Vincent Guffens <[EMAIL PROTECTED]> writes: Hi Vincent, > Here is a new version of the backtrace support. I have written the > ChangeLog and modified the code according to your advices. Thanks a lot! Here are still some comments and questions about the patch. > To compile with the backtrace su

Re: [Patch] Scripting engine

2005-08-28 Thread Yoshinori K. Okuji
On Wednesday 24 August 2005 11:36, Vladimir Serbinenko wrote: > I wrote the preciew version of scripting engine. I'll add some comments > soonly. I'd like to see a kind of "design document", since it is a bit hard for others to understand. > It would be good if then it > could be incorporated >

Re: backtrace support

2005-08-28 Thread Yoshinori K. Okuji
On Monday 22 August 2005 23:23, Vincent Guffens wrote: > +# For kern_debug.mod > +kern_debug_mod_OTHERDEP = kern/grub_debug_kern.sym > +kern_debug_mod_SOURCES = kern/kern_debug.c > +kern_debug_mod_CFLAGS = $(COMMON_CFLAGS) I think it is a really good idea to make a variable for dependencies, but

Re: font in open firmware

2005-08-28 Thread Yoshinori K. Okuji
On Wednesday 24 August 2005 20:30, Marco Gerards wrote: > Can you please explain why this is important and what you mean? AFAIK > the console font on the new world apple is fixed width. On the > pegasos a text screen just like the PC is used. Maybe this is the second time, but I talk about it.

Re: Completion for arguments

2005-08-28 Thread Yoshinori K. Okuji
On Saturday 27 August 2005 00:11, Marco Gerards wrote: > + /* Add the short arguments. */ > + for (option = cmd->options; option->doc; option++) > +{ > + if (!option->shortarg) > + continue; This should be: if (! grub_isgraph (option->shortarg)) continue; because we sometim