diff options
author | Karl Berry <karl@freefriends.org> | 2009-06-17 23:10:43 +0000 |
---|---|---|
committer | Karl Berry <karl@freefriends.org> | 2009-06-17 23:10:43 +0000 |
commit | 0391d3c497fbcb5a141be72868fa43918aed6a41 (patch) | |
tree | fe71401b8058d21bc22d8fb13238545b05ac385b /Build/source/texk/makeindexk | |
parent | 0e8507b3ae5ce3950803002c45ba2ebfbde976a8 (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/ChangeLog | 2 | ||||
-rw-r--r-- | Build/source/texk/makeindexk/genind.c | 7 |
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 |