On Fri, 19 May 2006 10:37:28 -0500 Scott Preece wrote:
> I agree that "counter" is wrong, but "control" is still a little
> vague. How about:
> "the &struct list_head that tracks iteration over the list" or
> "the &struct list_head that tracks current location in list."
I considered "iterator" (but that's the macro itself I think),
"control", "variable", "pointer", and "position".
"current location" would be OK with me.
> scott
>
> On 5/18/06, Randy.Dunlap <[email protected]> wrote:
> > From: Randy Dunlap <[email protected]>
> >
> > Use loop "control" instead of loop "counter" for list iterator
> > descriptions. They are not counters, they are controls or
> > pointers or positions.
> >
> > Signed-off-by: Randy Dunlap <[email protected]>
> > ---
> > include/linux/list.h | 52 +++++++++++++++++++++++++--------------------------
> > 1 files changed, 26 insertions(+), 26 deletions(-)
> >
> > --- linux-2617-rc4g6.orig/include/linux/list.h
> > +++ linux-2617-rc4g6/include/linux/list.h
> > @@ -328,7 +328,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each - iterate over a list
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @head: the head for your list.
> > */
> > #define list_for_each(pos, head) \
> > @@ -337,7 +337,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * __list_for_each - iterate over a list
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @head: the head for your list.
> > *
> > * This variant differs from list_for_each() in that it's the
> > @@ -350,7 +350,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_prev - iterate over a list backwards
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @head: the head for your list.
> > */
> > #define list_for_each_prev(pos, head) \
> > @@ -359,7 +359,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_safe - iterate over a list safe against removal of list entry
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @n: another &struct list_head to use as temporary storage
> > * @head: the head for your list.
> > */
> > @@ -369,7 +369,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry - iterate over list of given type
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > */
> > @@ -380,7 +380,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_reverse - iterate backwards over list of given type.
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > */
> > @@ -402,7 +402,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_continue - continue iteration over list of given type
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > *
> > @@ -416,7 +416,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_from - iterate over list of given type from the current point
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > *
> > @@ -428,7 +428,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_safe - iterate over list of given type safe against removal of list entry
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @n: another type * to use as temporary storage
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > @@ -441,7 +441,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_safe_continue
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @n: another type * to use as temporary storage
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > @@ -457,7 +457,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_safe_from
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @n: another type * to use as temporary storage
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > @@ -472,7 +472,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_safe_reverse
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @n: another type * to use as temporary storage
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > @@ -488,7 +488,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_rcu - iterate over an rcu-protected list
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @head: the head for your list.
> > *
> > * This list-traversal primitive may safely run concurrently with
> > @@ -507,7 +507,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_safe_rcu
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @n: another &struct list_head to use as temporary storage
> > * @head: the head for your list.
> > *
> > @@ -524,7 +524,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_entry_rcu - iterate over rcu list of given type
> > - * @pos: the type * to use as a loop counter.
> > + * @pos: the type * to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the list_struct within the struct.
> > *
> > @@ -541,7 +541,7 @@ static inline void list_splice_init(stru
> >
> > /**
> > * list_for_each_continue_rcu
> > - * @pos: the &struct list_head to use as a loop counter.
> > + * @pos: the &struct list_head to use as a loop control.
> > * @head: the head for your list.
> > *
> > * Iterate over an rcu-protected list, continuing after current point.
> > @@ -791,8 +791,8 @@ static inline void hlist_add_after_rcu(s
> >
> > /**
> > * hlist_for_each_entry - iterate over list of given type
> > - * @tpos: the type * to use as a loop counter.
> > - * @pos: the &struct hlist_node to use as a loop counter.
> > + * @tpos: the type * to use as a loop control.
> > + * @pos: the &struct hlist_node to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the hlist_node within the struct.
> > */
> > @@ -804,8 +804,8 @@ static inline void hlist_add_after_rcu(s
> >
> > /**
> > * hlist_for_each_entry_continue - iterate over a hlist continuing after current point
> > - * @tpos: the type * to use as a loop counter.
> > - * @pos: the &struct hlist_node to use as a loop counter.
> > + * @tpos: the type * to use as a loop control.
> > + * @pos: the &struct hlist_node to use as a loop control.
> > * @member: the name of the hlist_node within the struct.
> > */
> > #define hlist_for_each_entry_continue(tpos, pos, member) \
> > @@ -816,8 +816,8 @@ static inline void hlist_add_after_rcu(s
> >
> > /**
> > * hlist_for_each_entry_from - iterate over a hlist continuing from current point
> > - * @tpos: the type * to use as a loop counter.
> > - * @pos: the &struct hlist_node to use as a loop counter.
> > + * @tpos: the type * to use as a loop control.
> > + * @pos: the &struct hlist_node to use as a loop control.
> > * @member: the name of the hlist_node within the struct.
> > */
> > #define hlist_for_each_entry_from(tpos, pos, member) \
> > @@ -827,8 +827,8 @@ static inline void hlist_add_after_rcu(s
> >
> > /**
> > * hlist_for_each_entry_safe - iterate over list of given type safe against removal of list entry
> > - * @tpos: the type * to use as a loop counter.
> > - * @pos: the &struct hlist_node to use as a loop counter.
> > + * @tpos: the type * to use as a loop control.
> > + * @pos: the &struct hlist_node to use as a loop control.
> > * @n: another &struct hlist_node to use as temporary storage
> > * @head: the head for your list.
> > * @member: the name of the hlist_node within the struct.
> > @@ -841,8 +841,8 @@ static inline void hlist_add_after_rcu(s
> >
> > /**
> > * hlist_for_each_entry_rcu - iterate over rcu list of given type
> > - * @tpos: the type * to use as a loop counter.
> > - * @pos: the &struct hlist_node to use as a loop counter.
> > + * @tpos: the type * to use as a loop control.
> > + * @pos: the &struct hlist_node to use as a loop control.
> > * @head: the head for your list.
> > * @member: the name of the hlist_node within the struct.
> > *
> >
> >
> > ---
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to [email protected]
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
> >
>
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
