From 0391d3c497fbcb5a141be72868fa43918aed6a41 Mon Sep 17 00:00:00 2001 From: Karl Berry Date: Wed, 17 Jun 2009 23:10:43 +0000 Subject: more commentary on uninitialized let git-svn-id: svn://tug.org/texlive/trunk@13795 c570f23f-e606-0410-a88d-b1316a301751 --- Build/source/texk/makeindexk/ChangeLog | 2 +- Build/source/texk/makeindexk/genind.c | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) (limited to 'Build/source') 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 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 -- cgit v1.2.3