[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] File access library for lua
From: |
Bean |
Subject: |
Re: [PATCH] File access library for lua |
Date: |
Mon, 22 Jun 2009 11:38:35 +0800 |
Hi,
On Mon, Jun 22, 2009 at 8:42 AM, Pavel Roskin<address@hidden> wrote:
> On Sun, 2009-06-21 at 19:33 +0800, Bean wrote:
>
>> I've added a few function in the lua grub library to access files:
> ...
>> I also enable the string library from lua.
>
> The patch doesn't introduce any compiler warnings, and that's good.
>
> It makes lua.mod bigger by about 10%, but it's very little compared to
> the added functionality.
This is probably due to the enable of string library, the file access
function itself should be quite small.
>
> Please don't add trailing whitespace. STGit detects it in two places.
>
> There are several cases of missing spaces before parentheses.
>
> We probably need NESTED_FUNC_ATTR in the dir definition in struct
> grub_fs, as it takes two arguments. But is a separate issue.
>
> I'm feeling uneasy about having a parser in GRUB that is not used by
> default, but it's not related to the quality of the patch.
Actually, this is about to change. The file function here can be used
in a script osdetect.lua to generate menu items at runtime, and I also
plans to integrate lua with menu viewer.
>
> --
> Regards,
> Pavel Roskin
>
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>
--
Bean
- [PATCH] File access library for lua, Bean, 2009/06/21
- Re: [PATCH] File access library for lua, Pavel Roskin, 2009/06/21
- Re: [PATCH] File access library for lua,
Bean <=
- Re: [PATCH] File access library for lua, Pavel Roskin, 2009/06/21
- Re: [PATCH] File access library for lua, Robert Millan, 2009/06/22
- Re: [PATCH] File access library for lua, Bean, 2009/06/22
- Re: [PATCH] File access library for lua, Robert Millan, 2009/06/22
- Re: [PATCH] File access library for lua, Bean, 2009/06/22
- Re: [PATCH] File access library for lua, Pavel Roskin, 2009/06/22
- Re: [PATCH] File access library for lua, Bean, 2009/06/22
- Re: [PATCH] File access library for lua, Pavel Roskin, 2009/06/22
- Re: [PATCH] File access library for lua, Bean, 2009/06/22
- Re: [PATCH] File access library for lua, Pavel Roskin, 2009/06/22