Robert Millan wrote:
> This makes it possible to use relative paths in the "file" component of
> gfxmenu images.  I've tested that it works, but I'd appreciate if someone
> more familiar with gfxmenu can review it.
>   

+  static const char *theme_dir = NULL;
Add a member to grub_gui_image_t rather than using a static variable

+      theme_dir = value;
Use appropriate grub_strdup and free in _destroy function.
+      return load_image (self, absvalue);
absvalue isn't freeed

>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> http://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to