This commit is contained in:
bradleysmith23 2024-02-06 16:16:07 -08:00
parent 86cf5f32f2
commit efa9286d6f
2 changed files with 7 additions and 4 deletions

View file

@ -357,7 +357,7 @@ typedef struct xLIST
#define listINSERT_END( pxList, pxNewListItem ) \ #define listINSERT_END( pxList, pxNewListItem ) \
do { \ do { \
ListItem_t * const pxIndex = ( pxList )->pxIndex; \ ListItem_t * const pxIndex = ( pxList )->pxIndex; \
UBaseType_t uxNumberOfitems; \ UBaseType_t uxNumberOfItems; \
\ \
/* Only effective when configASSERT() is also defined, these tests may catch \ /* Only effective when configASSERT() is also defined, these tests may catch \
* the list data structures being overwritten in memory. They will not catch \ * the list data structures being overwritten in memory. They will not catch \

9
list.c
View file

@ -106,6 +106,7 @@ void vListInsertEnd( List_t * const pxList,
ListItem_t * const pxNewListItem ) ListItem_t * const pxNewListItem )
{ {
ListItem_t * const pxIndex = pxList->pxIndex; ListItem_t * const pxIndex = pxList->pxIndex;
UBaseType_t uxNumberOfItems;
traceENTER_vListInsertEnd( pxList, pxNewListItem ); traceENTER_vListInsertEnd( pxList, pxNewListItem );
@ -130,7 +131,7 @@ void vListInsertEnd( List_t * const pxList,
/* Remember which list the item is in. */ /* Remember which list the item is in. */
pxNewListItem->pxContainer = pxList; pxNewListItem->pxContainer = pxList;
UBaseType_t uxNumberOfItems = ( pxList->uxNumberOfItems ); uxNumberOfItems = ( pxList->uxNumberOfItems );
uxNumberOfItems++; uxNumberOfItems++;
( pxList->uxNumberOfItems ) = uxNumberOfItems; ( pxList->uxNumberOfItems ) = uxNumberOfItems;
@ -143,6 +144,7 @@ void vListInsert( List_t * const pxList,
{ {
ListItem_t * pxIterator; ListItem_t * pxIterator;
const TickType_t xValueOfInsertion = pxNewListItem->xItemValue; const TickType_t xValueOfInsertion = pxNewListItem->xItemValue;
UBaseType_t uxNumberOfItems;
traceENTER_vListInsert( pxList, pxNewListItem ); traceENTER_vListInsert( pxList, pxNewListItem );
@ -207,7 +209,7 @@ void vListInsert( List_t * const pxList,
* item later. */ * item later. */
pxNewListItem->pxContainer = pxList; pxNewListItem->pxContainer = pxList;
UBaseType_t uxNumberOfItems = ( pxList->uxNumberOfItems ); uxNumberOfItems = ( pxList->uxNumberOfItems );
uxNumberOfItems++; uxNumberOfItems++;
( pxList->uxNumberOfItems ) = uxNumberOfItems; ( pxList->uxNumberOfItems ) = uxNumberOfItems;
@ -220,6 +222,7 @@ UBaseType_t uxListRemove( ListItem_t * const pxItemToRemove )
/* The list item knows which list it is in. Obtain the list from the list /* The list item knows which list it is in. Obtain the list from the list
* item. */ * item. */
List_t * const pxList = pxItemToRemove->pxContainer; List_t * const pxList = pxItemToRemove->pxContainer;
UBaseType_t uxNumberOfItems;
traceENTER_uxListRemove( pxItemToRemove ); traceENTER_uxListRemove( pxItemToRemove );
@ -242,7 +245,7 @@ UBaseType_t uxListRemove( ListItem_t * const pxItemToRemove )
} }
pxItemToRemove->pxContainer = NULL; pxItemToRemove->pxContainer = NULL;
UBaseType_t uxNumberOfItems = ( pxList->uxNumberOfItems ); uxNumberOfItems = ( pxList->uxNumberOfItems );
uxNumberOfItems--; uxNumberOfItems--;
( pxList->uxNumberOfItems ) = uxNumberOfItems; ( pxList->uxNumberOfItems ) = uxNumberOfItems;