Bug 192266 - np_temp omitted from {SLIST,STAILQ,TAILQ} section examples in queue(3)
Summary: np_temp omitted from {SLIST,STAILQ,TAILQ} section examples in queue(3)
Status: Open
Alias: None
Product: Documentation
Classification: Unclassified
Component: Manual Pages (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-bugs (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2014-07-30 04:58 UTC by Enji Cooper
Modified: 2021-10-11 19:23 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Enji Cooper freebsd_committer freebsd_triage 2014-07-30 04:58:49 UTC
The LIST section example notes np_temp, but implies np_temp's existence in the {SLIST,STAILQ,TAILQ} sections.
Comment 1 Diego Casati 2016-12-14 22:50:10 UTC
Hi --

Is this ticket still valid? I've compared what's on sys/queue.h with queue.3 and did not find the discrepancies noted by the user.

queue.h defines these functions:

#define SLIST_FOREACH_SAFE(var, head, field, tvar)                      \
#define SLIST_FOREACH_FROM_SAFE(var, head, field, tvar)                 \
#define STAILQ_FOREACH_SAFE(var, head, field, tvar)                     \
#define STAILQ_FOREACH_FROM_SAFE(var, head, field, tvar)                \
#define LIST_FOREACH_SAFE(var, head, field, tvar)                       \
#define LIST_FOREACH_FROM_SAFE(var, head, field, tvar)                  \
#define TAILQ_FOREACH_SAFE(var, head, field, tvar)                      \
#define TAILQ_FOREACH_FROM_SAFE(var, head, field, tvar)                 \
#define TAILQ_FOREACH_REVERSE_SAFE(var, head, headname, field, tvar)    \
#define TAILQ_FOREACH_REVERSE_FROM_SAFE(var, head, headname, field, tvar) \

Every example on queue.3 that uses these functions do make a reference to tvar (declared as np_temp on the man page).

SLIST_FOREACH_SAFE(np, &head, entries, np_temp) {
STAILQ_FOREACH_SAFE(np, &head, entries, np_temp) {
LIST_FOREACH_SAFE(np, &head, entries, np_temp) {
TAILQ_FOREACH_SAFE(np, &head, entries, np_temp) {


Did I miss anything ? 

thanks,

-dcasati
Comment 2 Felix Johnson 2021-10-11 19:20:58 UTC
See bug #192267 and attachment #227599 [details].
The attachment there changes np_temp to tvar to match the macro argument name.
Comment 3 Felix Johnson 2021-10-11 19:23:18 UTC
(In reply to Felix Johnson from comment #2)
CORRECTION: See attachment #228599 [details]