summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordkf <donal.k.fellows@manchester.ac.uk>2012-06-29 08:47:49 (GMT)
committerdkf <donal.k.fellows@manchester.ac.uk>2012-06-29 08:47:49 (GMT)
commit18e116ab4e97129f7b4d4e36792035acf039a3ff (patch)
tree8abae8da3745d19293c8f8072bdb2e6ddb5d83e9
parent374cbfedc276dc200fd23f713711ffc43115e13a (diff)
parentfe280c4de5f432f6790900dfe1323a0a935d342c (diff)
downloadtcl-18e116ab4e97129f7b4d4e36792035acf039a3ff.zip
tcl-18e116ab4e97129f7b4d4e36792035acf039a3ff.tar.gz
tcl-18e116ab4e97129f7b4d4e36792035acf039a3ff.tar.bz2
Reinforced the description of the requirement for the tables of names for Tcl_GetIndexFromObj to index over to be static, following posting to tcl-core by Brian Griffin about a bug caused by not obeying this rule correctly.
-rw-r--r--ChangeLog8
-rw-r--r--doc/GetIndex.310
2 files changed, 16 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 60c74ef..b8d7638 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2012-06-29 Donal K. Fellows <dkf@users.sf.net>
+
+ * doc/GetIndex.3: Reinforced the description of the requirement for
+ the tables of names to index over to be static, following posting to
+ tcl-core by Brian Griffin about a bug caused by Tktreectrl not obeying
+ this rule correctly. This does not represent a functionality change,
+ merely a clearer documentation of a long-standing constraint.
+
2012-06-25 Don Porter <dgp@users.sourceforge.net>
* generic/tclFileSystem.h: [Bug 3024359] Make sure that the
diff --git a/doc/GetIndex.3 b/doc/GetIndex.3
index 82590fb..45d4e9c 100644
--- a/doc/GetIndex.3
+++ b/doc/GetIndex.3
@@ -32,10 +32,16 @@ table entry.
.AP "const char" **tablePtr in
An array of null-terminated strings. The end of the array is marked
by a NULL string pointer.
+Note that references to the \fItablePtr\fR may be retained in the
+internal representation of \fIobjPtr\fR, so this should represent the
+address of a statically-allocated array.
.AP "const void" *structTablePtr in
An array of arbitrary type, typically some \fBstruct\fR type.
The first member of the structure must be a null-terminated string.
The size of the structure is given by \fIoffset\fR.
+Note that references to the \fIstructTablePtr\fR may be retained in the
+internal representation of \fIobjPtr\fR, so this should represent the
+address of a statically-allocated array of structures.
.AP int offset in
The offset to add to structTablePtr to get to the next entry.
The end of the array is marked by a NULL string pointer.
@@ -52,10 +58,10 @@ The index of the string in \fItablePtr\fR that matches the value of
.SH DESCRIPTION
.PP
-This procedure provides an efficient way for looking up keywords,
+These procedures provide an efficient way for looking up keywords,
switch names, option names, and similar things where the value of
an object must be one of a predefined set of values.
-\fIObjPtr\fR is compared against each of
+\fBTcl_GetIndexFromObj\fR compares \fIobjPtr\fR against each of
the strings in \fItablePtr\fR to find a match. A match occurs if
\fIobjPtr\fR's string value is identical to one of the strings in
\fItablePtr\fR, or if it is a non-empty unique abbreviation