[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: |
Tue, 16 Jun 2009 01:25:39 +0200 |
Comitted
On Mon, Jun 15, 2009 at 5:22 PM, Vladimir 'phcoder'
Serbinenko<address@hidden> wrote:
> 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
>
--
Regards
Vladimir 'phcoder' Serbinenko