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. 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. -- Regards, Pavel Roskin _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel
