[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] more scripting fixes
From: |
Vladimir 'phcoder' Serbinenko |
Subject: |
Re: [PATCH] more scripting fixes |
Date: |
Mon, 15 Jun 2009 17:22:21 +0200 |
On Sat, Jun 13, 2009 at 10:15 PM, Pavel Roskin<address@hidden> wrote:
> On Sat, 2009-06-13 at 00:34 +0200, Vladimir 'phcoder' Serbinenko wrote:
>
>> > It looks like \" cancels quoting for the subsequent characters, but it
>> > shouldn't.
>> Actually the problem is different: escaping in quoting isn't parsed at
>> all. For first step lexer uses rescue mode parser which isn't that
>> easy to extend compactly. I think that we should leave rescue mode
>> parser as it is now and put a better first-step parser in sh.mod. I'll
>> do it when my todo list scrols to this point
>
> I see.
>
> I have no objections against your patch, but it would be better if you
> split it into logical pieces. Parsing is hard to get right and easy to
> break, and we don't have a testsuite to catch breakage.
Good idea will do it when comitting
>
> Some changes are trivial, such as replacing
>
> while (!state->script || ! grub_strlen (state->script))
>
> with
>
> while (! state->script || ! *state->script)
>
> Other changes actually change the behavior. If you can split the fix
> for escaped quotes from the fix for quoted newlines, it would be good.
> This way, if any breakage is found, it could be traced to a smaller
> change.
>
> --
> Regards,
> Pavel Roskin
>
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>
--
Regards
Vladimir 'phcoder' Serbinenko