search for: new_background

I've traced the code of set_background quite a bit this w.e. and was wondering what would happen in the __unlikely case where there is no background for the main menu and one for a submenu. It looks to me that when the submenu is closed, set_background will be called with new_background == NULL, that current_background != NULL inside so that there will be a strcmp(current_background, NULL) inside the function. Thanks, Lo?c Greni?

The following changes since commit 8c576f1fe03e34879921311f46613a35c6530000: Merge remote-tracking branch 'mfleming/for-hpa/elflink/fix-compiler-warnings' into elflink (2011-03-16 12:53:58 -0700) are available in the git repository at: git://git.zytor.com/users/mfleming/syslinux.git for-hpa/elflink/ldlinux Matt Fleming (1): ldlinux: Perform auto-boot if NOESCAPE set in config

...> +#include <core-elf.h> #include <syslinux/vesacon.h> #include <sys/module.h> #include "menu.h" @@ -33,6 +34,8 @@ int draw_background(const char *what) else return vesacon_load_background(what); #endif + + return 0; } void set_background(const char *new_background) diff --git a/com32/elflink/modules/hello.c b/com32/elflink/modules/hello.c index cba63e2..b38f05c 100644 --- a/com32/elflink/modules/hello.c +++ b/com32/elflink/modules/hello.c @@ -17,10 +17,9 @@ static int hello_main(int argc, char **argv) { int *nums = NULL; - int i; nums = mall...

From: Matt Fleming <matt.fleming at linux.intel.com> This series fixes some bugs and switches the elflink branch to be entirely ELF modules. It applies on top of, http://syslinux.zytor.com/archives/2011-April/016369.html The deletions in the diff stat below are mainly from deleting com32/elflink/modules (finally!). Now there should be no duplicate code because we don't need COM32 and

From: Matt Fleming <matt.fleming at intel.com> Since we can't use __intcall() for EFI, and since we can now have the ELF module code resolve all our symbols at runtime, we should delete as many references to __intcall() as possible and just access the symbols directly. The most interesting patch is the support for weak symbols. We need to be able to reference derivative-specific

...*const kernel_types[]; - -extern enum kernel_type parse_kernel_type(const char *kernel); - /* Configurable integer parameters */ enum parameter_number { P_WIDTH, @@ -230,9 +226,6 @@ extern const int message_base_color; extern const char *current_background; void set_background(const char *new_background); -/* execute.c */ -void execute(const char *cmdline, enum kernel_type type); - /* drain.c */ void drain_keyboard(void); diff --git a/com32/include/sys/module.h b/com32/include/sys/module.h index eabc9e0..ea11a88 100644 --- a/com32/include/sys/module.h +++ b/com32/include/sys/module.h @@ -138...

From: Matt Fleming <matt.fleming at linux.intel.com> This is a series of patches that, * shrink the core by moving things into an ldlinux ELF module * begin wiring up some of the C versions of various functions The core now only contains essential code and loads the ldlinux module to do everything else, like providing a command line interface and loading kernels. The config file parsing