xQueueGive() ordering of increment

feilipu · feilipu · commit 15fc40a754ea · 2016-03-14T03:05:34.000+11:00
diff --git a/src/queue.c b/src/queue.c
@@ -833,8 +833,6 @@ Queue_t * const pxQueue = ( Queue_t * ) xQueue;
 			{
 				traceQUEUE_SEND( pxQueue );
 
-				++( pxQueue->uxMessagesWaiting );
-
 				#if ( configUSE_MUTEXES == 1 )
 				{
 					if( pxQueue->uxQueueType == queueQUEUE_IS_MUTEX )
@@ -849,6 +847,8 @@ Queue_t * const pxQueue = ( Queue_t * ) xQueue;
 					}
 				}
 				#endif /* configUSE_MUTEXES */
+				
+				++( pxQueue->uxMessagesWaiting );
 
 				#if ( configUSE_QUEUE_SETS == 1 )
 				{

Original file line number	Diff line number	Diff line change
`@@ -833,8 +833,6 @@ Queue_t * const pxQueue = ( Queue_t * ) xQueue;`
`833`	`833`	`{`
`834`	`834`	`traceQUEUE_SEND( pxQueue );`
`835`	`835`
`836`		`- ++( pxQueue->uxMessagesWaiting );`
`837`		`-`
`838`	`836`	`#if ( configUSE_MUTEXES == 1 )`
`839`	`837`	`{`
`840`	`838`	`if( pxQueue->uxQueueType == queueQUEUE_IS_MUTEX )`
`@@ -849,6 +847,8 @@ Queue_t * const pxQueue = ( Queue_t * ) xQueue;`
`849`	`847`	`}`
`850`	`848`	`}`
`851`	`849`	`#endif /* configUSE_MUTEXES */`
	`850`	`+`
	`851`	`+ ++( pxQueue->uxMessagesWaiting );`
`852`	`852`
`853`	`853`	`#if ( configUSE_QUEUE_SETS == 1 )`
`854`	`854`	`{`