[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Split #2: parser/reader separation
From: |
Bean |
Subject: |
Re: [PATCH] Split #2: parser/reader separation |
Date: |
Mon, 27 Apr 2009 13:51:44 +0800 |
On Sun, Apr 26, 2009 at 11:50 PM, Vladimir 'phcoder' Serbinenko
<address@hidden> wrote:
> Hello this patch breaksĀ grub-emu. Also some files include grub/rescue.h and
> this has to be fixed.
Hi,
For large patch, I normally add only the necessary changes to make it
compile in i386-pc. As commits are quite frequent, this would minimize
the effort to sync with svn head. I'd fix those small issue before
final commitment.
> Also there is a bug is that in case of syntax error reader continues to ask
> for more lines. E.g
> grub> if
>>;
> syntax error
>>
> Then it's impossible to exit from this "bug mode". It's not your fault but
> since you touch this code could you fix this?
I believe a proper way to solve this is to add an eol character, for
example ctrl-D. Then we have a way to break from console input. But
this fix is not trivial, perhaps it should be in a separate patch.
> This patch increases the size of kernel by 504 bytes and the size of
> core.img by 224 bytes. Is there a way to do the same thing in a more compact
> way?
Oh, I think it is quite compact already, have you got any suggestion
to reduce its size ?
> Also it would be good to write the current parser in "grub>" prompt (not
> necessary for rescue prompt)
Good point.
--
Bean