Fixes for list.h and poison.h to be used in C++ code.

- Add list.h and poison.h to Makefile.am;
- list.h cannot use the C++ reserved keyword new;
- Cast void pointers to the proper type;
- Proper C++ pointer arithmetic on poison.h.
This commit is contained in:
Marcelo Roberto Jimenez 2010-11-15 00:07:14 -02:00
parent eb5db65692
commit 5c8d118899
3 changed files with 32 additions and 30 deletions

View File

@ -25,6 +25,8 @@ upnpinclude_HEADERS = \
inc/Event.h \ inc/Event.h \
inc/EventSubscribe.h \ inc/EventSubscribe.h \
inc/FileInfo.h \ inc/FileInfo.h \
inc/list.h \
inc/poison.h \
inc/StateVarComplete.h \ inc/StateVarComplete.h \
inc/StateVarRequest.h \ inc/StateVarRequest.h \
inc/SubscriptionRequest.h \ inc/SubscriptionRequest.h \

View File

@ -41,46 +41,46 @@ static inline void INIT_LIST_HEAD(struct list_head *list)
* the prev/next entries already! * the prev/next entries already!
*/ */
#ifndef CONFIG_DEBUG_LIST #ifndef CONFIG_DEBUG_LIST
static inline void __list_add(struct list_head *new, static inline void __list_add(struct list_head *new_,
struct list_head *prev, struct list_head *prev,
struct list_head *next) struct list_head *next)
{ {
next->prev = new; next->prev = new_;
new->next = next; new_->next = next;
new->prev = prev; new_->prev = prev;
prev->next = new; prev->next = new_;
} }
#else #else
extern void __list_add(struct list_head *new, extern void __list_add(struct list_head *new_,
struct list_head *prev, struct list_head *prev,
struct list_head *next); struct list_head *next);
#endif #endif
/** /**
* list_add - add a new entry * list_add - add a new entry
* @new: new entry to be added * @new_: new entry to be added
* @head: list head to add it after * @head: list head to add it after
* *
* Insert a new entry after the specified head. * Insert a new entry after the specified head.
* This is good for implementing stacks. * This is good for implementing stacks.
*/ */
static inline void list_add(struct list_head *new, struct list_head *head) static inline void list_add(struct list_head *new_, struct list_head *head)
{ {
__list_add(new, head, head->next); __list_add(new_, head, head->next);
} }
/** /**
* list_add_tail - add a new entry * list_add_tail - add a new entry
* @new: new entry to be added * @new_: new entry to be added
* @head: list head to add it before * @head: list head to add it before
* *
* Insert a new entry before the specified head. * Insert a new entry before the specified head.
* This is useful for implementing queues. * This is useful for implementing queues.
*/ */
static inline void list_add_tail(struct list_head *new, struct list_head *head) static inline void list_add_tail(struct list_head *new_, struct list_head *head)
{ {
__list_add(new, head->prev, head); __list_add(new_, head->prev, head);
} }
/* /*
@ -106,8 +106,8 @@ static inline void __list_del(struct list_head * prev, struct list_head * next)
static inline void list_del(struct list_head *entry) static inline void list_del(struct list_head *entry)
{ {
__list_del(entry->prev, entry->next); __list_del(entry->prev, entry->next);
entry->next = LIST_POISON1; entry->next = (struct list_head *)LIST_POISON1;
entry->prev = LIST_POISON2; entry->prev = (struct list_head *)LIST_POISON2;
} }
#else #else
extern void list_del(struct list_head *entry); extern void list_del(struct list_head *entry);
@ -116,23 +116,23 @@ extern void list_del(struct list_head *entry);
/** /**
* list_replace - replace old entry by new one * list_replace - replace old entry by new one
* @old : the element to be replaced * @old : the element to be replaced
* @new : the new element to insert * @new_ : the new element to insert
* *
* If @old was empty, it will be overwritten. * If @old was empty, it will be overwritten.
*/ */
static inline void list_replace(struct list_head *old, static inline void list_replace(struct list_head *old,
struct list_head *new) struct list_head *new_)
{ {
new->next = old->next; new_->next = old->next;
new->next->prev = new; new_->next->prev = new_;
new->prev = old->prev; new_->prev = old->prev;
new->prev->next = new; new_->prev->next = new_;
} }
static inline void list_replace_init(struct list_head *old, static inline void list_replace_init(struct list_head *old,
struct list_head *new) struct list_head *new_)
{ {
list_replace(old, new); list_replace(old, new_);
INIT_LIST_HEAD(old); INIT_LIST_HEAD(old);
} }
@ -593,8 +593,8 @@ static inline void __hlist_del(struct hlist_node *n)
static inline void hlist_del(struct hlist_node *n) static inline void hlist_del(struct hlist_node *n)
{ {
__hlist_del(n); __hlist_del(n);
n->next = LIST_POISON1; n->next = (struct hlist_node *)LIST_POISON1;
n->pprev = LIST_POISON2; n->pprev = (struct hlist_node **)LIST_POISON2;
} }
static inline void hlist_del_init(struct hlist_node *n) static inline void hlist_del_init(struct hlist_node *n)
@ -641,11 +641,11 @@ static inline void hlist_add_after(struct hlist_node *n,
* reference of the first entry if it exists. * reference of the first entry if it exists.
*/ */
static inline void hlist_move_list(struct hlist_head *old, static inline void hlist_move_list(struct hlist_head *old,
struct hlist_head *new) struct hlist_head *new_)
{ {
new->first = old->first; new_->first = old->first;
if (new->first) if (new_->first)
new->first->pprev = &new->first; new_->first->pprev = &new_->first;
old->first = NULL; old->first = NULL;
} }

View File

@ -19,8 +19,8 @@
* under normal circumstances, used to verify that nobody uses * under normal circumstances, used to verify that nobody uses
* non-initialized list entries. * non-initialized list entries.
*/ */
#define LIST_POISON1 ((void *) 0x00100100 + POISON_POINTER_DELTA) #define LIST_POISON1 ((void *) (0x00100100 + POISON_POINTER_DELTA))
#define LIST_POISON2 ((void *) 0x00200200 + POISON_POINTER_DELTA) #define LIST_POISON2 ((void *) (0x00200200 + POISON_POINTER_DELTA))
/********** include/linux/timer.h **********/ /********** include/linux/timer.h **********/
/* /*