Re: [PATCH 2/2] list.h doc: change "counter" to "control"

[Randy.Dunlap]

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 <rdunlap@xxxxxxxxxxxx> wrote:
> > From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> >
> > 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 <rdunlap@xxxxxxxxxxxx>
> > ---
> >  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 majordomo@xxxxxxxxxxxxxxx
> > 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/