len0rd/FreeRTOS-Kernel
1
0
Fork 0
mirror of https://github.com/FreeRTOS/FreeRTOS-Kernel.git synced 2025-09-12 09:07:46 -04:00
Code Issues Projects Releases Packages Wiki Activity

The following line was used in a couple of placed to force

Browse source 
an assert, but caused warnings on 64-bit architectures because
it compares a uint32_t with a 64-bit number.
configASSERT( pxTCB->ulNotifiedValue[ uxIndexToNotify ] == ~0UL );

The changed to configASSERT( xTickCount == ( TickType_t ) 0 );
which has the same effect of forcing an assert, but without
the warning.
This commit is contained in:
RichardBarry 2020-09-06 20:34:39 -07:00
parent ce3134bba9
commit 82b2079456
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
tasks.c
View file

@ -4887,7 +4887,7 @@ TickType_t uxTaskResetEventItemValue( void )
/* Should not get here if all enums are handled. /* Should not get here if all enums are handled.
* Artificially force an assert by testing a value the * Artificially force an assert by testing a value the
* compiler can't assume is const. */ * compiler can't assume is const. */
configASSERT( pxTCB->ulNotifiedValue[ uxIndexToNotify ] == ~0UL ); configASSERT( xTickCount == ( TickType_t ) 0 );
break; break;
} }
@ -5030,7 +5030,7 @@ TickType_t uxTaskResetEventItemValue( void )
/* Should not get here if all enums are handled. /* Should not get here if all enums are handled.
* Artificially force an assert by testing a value the * Artificially force an assert by testing a value the
* compiler can't assume is const. */ * compiler can't assume is const. */
configASSERT( pxTCB->ulNotifiedValue[ uxIndexToNotify ] == ~0UL ); configASSERT( xTickCount == ( TickType_t ) 0 );
break; break;
} }