> Please go ahead (I assume it raises no warnings and it's well tested). I tested the latest version with gfxterm and xnu loader. I use previous version in my daily bootloader. I'll run another series of tests before comitting. > > One small detail: > >> + grub_free (mode_list); >> + mode_list = 0; > > I would prefer if we used NULL for pointers. I know we don't do that > everywhere, but it makes code easier to understand. >From GCS: "Zero without a cast is perfectly fine as a null pointer constant, except when calling a varargs function. " > > -- > Robert Millan > > The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and > how) you may access your data; but nobody's threatening your freedom: we > still allow you to remove your data and not access it at all." > > > _______________________________________________ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel >
-- Regards Vladimir 'phcoder' Serbinenko Personal git repository: http://repo.or.cz/w/grub2/phcoder.git _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel
