summaryrefslogtreecommitdiff
path: root/Build/source/texk/makeindexk
diff options
context:
space:
mode:
authorKarl Berry <karl@freefriends.org>2009-06-17 23:10:43 +0000
committerKarl Berry <karl@freefriends.org>2009-06-17 23:10:43 +0000
commit0391d3c497fbcb5a141be72868fa43918aed6a41 (patch)
treefe71401b8058d21bc22d8fb13238545b05ac385b /Build/source/texk/makeindexk
parent0e8507b3ae5ce3950803002c45ba2ebfbde976a8 (diff)
more commentary on uninitialized let
git-svn-id: svn://tug.org/texlive/trunk@13795 c570f23f-e606-0410-a88d-b1316a301751
Diffstat (limited to 'Build/source/texk/makeindexk')
-rw-r--r--Build/source/texk/makeindexk/ChangeLog2
-rw-r--r--Build/source/texk/makeindexk/genind.c7
2 files changed, 7 insertions, 2 deletions
diff --git a/Build/source/texk/makeindexk/ChangeLog b/Build/source/texk/makeindexk/ChangeLog
index 377baa8d108..36d3f40f523 100644
--- a/Build/source/texk/makeindexk/ChangeLog
+++ b/Build/source/texk/makeindexk/ChangeLog
@@ -5,7 +5,7 @@
* genind.c (new_entry): initialize let to -1. It will be
reassigned if it is actually used, as far as I can make out the
- overcomplex condition.
+ overcomplex condition and usage in put_header.
2009-06-12 Peter Breitenlohner <peb@mppmu.mpg.de>
diff --git a/Build/source/texk/makeindexk/genind.c b/Build/source/texk/makeindexk/genind.c
index 44e364871a0..185a8bb7a7d 100644
--- a/Build/source/texk/makeindexk/genind.c
+++ b/Build/source/texk/makeindexk/genind.c
@@ -221,7 +221,7 @@ char *term;
static void
new_entry(VOID_ARG)
{
- int let = -1;
+ int let = -1; /* see comment below */
FIELD_PTR ptr;
#ifdef HAVE_SETLOCALE
char *prev_locale;
@@ -251,6 +251,11 @@ new_entry(VOID_ARG)
PUT(group_skip);
ind_lc += skiplen;
/* beginning of a new letter? */
+ /* Although we may use let unassigned here, it doesn't matter,
+ because put_header will not use its arg except in case
+ ALPHA, when it is assigned in the midst of condition above.
+ Let's not perturb the logic, just initialize it (above) to
+ avoid the warning. */
put_header(let);
make_item(NIL);
} else