I'm trying to familiarize myself with clang and did a
quick 'clang-cl --analyze <MSVC-CFLAGS> <SOURCES>'. This is what
it revealed:
init.c(569,13) : warning: Value stored to 'home' during its initialization is
never read
char *home = home_dir ();
^~~~ ~~~~~~~~~~~
Makes sense since since:
char *home = home_dir (); << line 569
xfree (file);
home = ws_mypath ();
Seems this last line should be dropped since home_dir() already
calls ws_mypath(). So I think this patch is in order:
--- a/init.c 2015-05-25 18:36:26 +0000
+++ b/init.c 2015-06-17 22:26:34 +0000
@@ -568,7 +568,6 @@
{
char *home = home_dir ();
xfree (file);
- home = ws_mypath ();
if (home)
{
file = aprintf ("%s/wget.ini", home);
--
--gv