Code cleanup

This commit is contained in:
Pierre Pronchery 2017-11-09 03:24:45 +01:00
parent 3578933a2c
commit 11980fa1b5
3 changed files with 43 additions and 31 deletions

View File

@ -471,6 +471,13 @@ HostOS configure_get_os(Configure * configure)
} }
/* configure_get_prefs */
ConfigurePrefs const * configure_get_prefs(Configure * configure)
{
return configure->prefs;
}
/* configure_get_soext */ /* configure_get_soext */
String const * configure_get_soext(Configure * configure) String const * configure_get_soext(Configure * configure)
{ {

View File

@ -181,6 +181,7 @@ String const * configure_get_config(Configure * configure,
String const * section, String const * variable); String const * section, String const * variable);
String const * configure_get_exeext(Configure * configure); String const * configure_get_exeext(Configure * configure);
HostOS configure_get_os(Configure * configure); HostOS configure_get_os(Configure * configure);
ConfigurePrefs const * configure_get_prefs(Configure * configure);
String const * configure_get_soext(Configure * configure); String const * configure_get_soext(Configure * configure);
unsigned int configure_is_flag_set(Configure * configure, unsigned int flags); unsigned int configure_is_flag_set(Configure * configure, unsigned int flags);

View File

@ -201,8 +201,8 @@ static int _variables_package(Configure * configure, FILE * fp,
_makefile_output_variable(fp, "PACKAGE", package); _makefile_output_variable(fp, "PACKAGE", package);
_makefile_output_variable(fp, "VERSION", version); _makefile_output_variable(fp, "VERSION", version);
if((p = _makefile_get_config(configure, NULL, "config")) != NULL) if((p = _makefile_get_config(configure, NULL, "config")) != NULL)
return settings(configure->prefs, configure->config, directory, return settings(configure_get_prefs(configure),
package, version); configure->config, directory, package, version);
return 0; return 0;
} }
@ -243,6 +243,7 @@ static int _variables_print(Configure * configure, FILE * fp,
static int _variables_dist(Configure * configure, FILE * fp) static int _variables_dist(Configure * configure, FILE * fp)
{ {
ConfigurePrefs const * prefs;
String const * p; String const * p;
String * dist; String * dist;
String * q; String * q;
@ -253,6 +254,7 @@ static int _variables_dist(Configure * configure, FILE * fp)
return 0; return 0;
if((dist = string_new(p)) == NULL) if((dist = string_new(p)) == NULL)
return 1; return 1;
prefs = configure_get_prefs(configure);
q = dist; q = dist;
for(i = 0;; i++) for(i = 0;; i++)
{ {
@ -268,9 +270,9 @@ static int _variables_dist(Configure * configure, FILE * fp)
{ {
_makefile_output_variable(fp, "OBJDIR", ""); _makefile_output_variable(fp, "OBJDIR", "");
_makefile_output_variable(fp, "PREFIX", _makefile_output_variable(fp, "PREFIX",
configure->prefs->prefix); prefs->prefix);
_makefile_output_variable(fp, "DESTDIR", _makefile_output_variable(fp, "DESTDIR",
configure->prefs->destdir); prefs->destdir);
} }
_makefile_output_variable(fp, "MKDIR", _makefile_output_variable(fp, "MKDIR",
configure->programs.mkdir); configure->programs.mkdir);
@ -398,6 +400,7 @@ static void _executables_variables(Configure * configure, FILE * fp,
String const * target, char * done); String const * target, char * done);
static int _variables_executables(Configure * configure, FILE * fp) static int _variables_executables(Configure * configure, FILE * fp)
{ {
ConfigurePrefs const * prefs;
char done[TT_LAST]; /* FIXME even better if'd be variable by variable */ char done[TT_LAST]; /* FIXME even better if'd be variable by variable */
String const * targets; String const * targets;
String const * includes; String const * includes;
@ -407,6 +410,7 @@ static int _variables_executables(Configure * configure, FILE * fp)
size_t i; size_t i;
char c; char c;
prefs = configure_get_prefs(configure);
memset(&done, 0, sizeof(done)); memset(&done, 0, sizeof(done));
targets = _makefile_get_config(configure, NULL, "targets"); targets = _makefile_get_config(configure, NULL, "targets");
includes = _makefile_get_config(configure, NULL, "includes"); includes = _makefile_get_config(configure, NULL, "includes");
@ -433,10 +437,8 @@ static int _variables_executables(Configure * configure, FILE * fp)
else if(includes != NULL) else if(includes != NULL)
{ {
_makefile_output_variable(fp, "OBJDIR", ""); _makefile_output_variable(fp, "OBJDIR", "");
_makefile_output_variable(fp, "PREFIX", _makefile_output_variable(fp, "PREFIX", prefs->prefix);
configure->prefs->prefix); _makefile_output_variable(fp, "DESTDIR", prefs->destdir);
_makefile_output_variable(fp, "DESTDIR",
configure->prefs->destdir);
} }
if(targets != NULL || includes != NULL || package != NULL) if(targets != NULL || includes != NULL || package != NULL)
{ {
@ -516,31 +518,31 @@ static void _binary_ldflags(Configure * configure, FILE * fp,
String const * ldflags); String const * ldflags);
static void _variables_binary(Configure * configure, FILE * fp, char * done) static void _variables_binary(Configure * configure, FILE * fp, char * done)
{ {
ConfigurePrefs const * prefs;
String * p; String * p;
prefs = configure_get_prefs(configure);
if(!done[TT_LIBRARY] && !done[TT_SCRIPT]) if(!done[TT_LIBRARY] && !done[TT_SCRIPT])
{ {
_makefile_output_variable(fp, "OBJDIR", ""); _makefile_output_variable(fp, "OBJDIR", "");
_makefile_output_variable(fp, "PREFIX", _makefile_output_variable(fp, "PREFIX", prefs->prefix);
configure->prefs->prefix); _makefile_output_variable(fp, "DESTDIR", prefs->destdir);
_makefile_output_variable(fp, "DESTDIR",
configure->prefs->destdir);
} }
/* BINDIR */ /* BINDIR */
if(configure->prefs->bindir[0] == '/') if(prefs->bindir[0] == '/')
_makefile_output_variable(fp, "BINDIR", _makefile_output_variable(fp, "BINDIR",
configure->prefs->bindir); prefs->bindir);
else if((p = string_new_append("$(PREFIX)/", configure->prefs->bindir, else if((p = string_new_append("$(PREFIX)/", prefs->bindir,
NULL)) != NULL) NULL)) != NULL)
{ {
_makefile_output_variable(fp, "BINDIR", p); _makefile_output_variable(fp, "BINDIR", p);
string_delete(p); string_delete(p);
} }
/* SBINDIR */ /* SBINDIR */
if(configure->prefs->sbindir[0] == '/') if(prefs->sbindir[0] == '/')
_makefile_output_variable(fp, "SBINDIR", _makefile_output_variable(fp, "SBINDIR",
configure->prefs->sbindir); prefs->sbindir);
else if((p = string_new_append("$(PREFIX)/", configure->prefs->sbindir, else if((p = string_new_append("$(PREFIX)/", prefs->sbindir,
NULL)) != NULL) NULL)) != NULL)
{ {
_makefile_output_variable(fp, "SBINDIR", p); _makefile_output_variable(fp, "SBINDIR", p);
@ -759,19 +761,19 @@ static void _binary_ldflags(Configure * configure, FILE * fp,
static void _variables_library(Configure * configure, FILE * fp, char * done) static void _variables_library(Configure * configure, FILE * fp, char * done)
{ {
ConfigurePrefs const * prefs;
String const * libdir; String const * libdir;
String const * p; String const * p;
prefs = configure_get_prefs(configure);
if(!done[TT_LIBRARY] && !done[TT_SCRIPT]) if(!done[TT_LIBRARY] && !done[TT_SCRIPT])
{ {
_makefile_output_variable(fp, "OBJDIR", ""); _makefile_output_variable(fp, "OBJDIR", "");
_makefile_output_variable(fp, "PREFIX", _makefile_output_variable(fp, "PREFIX", prefs->prefix);
configure->prefs->prefix); _makefile_output_variable(fp, "DESTDIR", prefs->destdir);
_makefile_output_variable(fp, "DESTDIR",
configure->prefs->destdir);
} }
if((libdir = _makefile_get_config(configure, NULL, "libdir")) == NULL) if((libdir = _makefile_get_config(configure, NULL, "libdir")) == NULL)
libdir = configure->prefs->libdir; libdir = prefs->libdir;
if(libdir[0] == '/') if(libdir[0] == '/')
_makefile_output_variable(fp, "LIBDIR", libdir); _makefile_output_variable(fp, "LIBDIR", libdir);
else else
@ -829,18 +831,20 @@ static void _variables_libtool(Configure * configure, FILE * fp, char * done)
static void _variables_script(Configure * configure, FILE * fp, char * done) static void _variables_script(Configure * configure, FILE * fp, char * done)
{ {
ConfigurePrefs const * prefs;
if(!done[TT_BINARY] && !done[TT_LIBRARY] && !done[TT_SCRIPT]) if(!done[TT_BINARY] && !done[TT_LIBRARY] && !done[TT_SCRIPT])
{ {
prefs = configure_get_prefs(configure);
_makefile_output_variable(fp, "OBJDIR", ""); _makefile_output_variable(fp, "OBJDIR", "");
_makefile_output_variable(fp, "PREFIX", _makefile_output_variable(fp, "PREFIX", prefs->prefix);
configure->prefs->prefix); _makefile_output_variable(fp, "DESTDIR", prefs->destdir);
_makefile_output_variable(fp, "DESTDIR",
configure->prefs->destdir);
} }
} }
static int _variables_includes(Configure * configure, FILE * fp) static int _variables_includes(Configure * configure, FILE * fp)
{ {
ConfigurePrefs const * prefs;
String const * includes; String const * includes;
if((includes = _makefile_get_config(configure, NULL, "includes")) if((includes = _makefile_get_config(configure, NULL, "includes"))
@ -848,12 +852,12 @@ static int _variables_includes(Configure * configure, FILE * fp)
return 0; return 0;
if(fp == NULL) if(fp == NULL)
return 0; return 0;
if(configure->prefs->includedir[0] == '/') prefs = configure_get_prefs(configure);
_makefile_output_variable(fp, "INCLUDEDIR", if(prefs->includedir[0] == '/')
configure->prefs->includedir); _makefile_output_variable(fp, "INCLUDEDIR", prefs->includedir);
else else
_makefile_print(fp, "%s%s\n", "INCLUDEDIR= $(PREFIX)/", _makefile_print(fp, "%s%s\n", "INCLUDEDIR= $(PREFIX)/",
configure->prefs->includedir); prefs->includedir);
return 0; return 0;
} }