Merge pull request #7901 from deepikabhavnani/warning_fix

0xc0170 · web-flow · commit 9abcbf6acfcc · 2018-09-17T14:29:15.000+02:00
Compiler Warning fixes
diff --git a/hal/mbed_ticker_api.c b/hal/mbed_ticker_api.c
@@ -47,7 +47,7 @@ static void initialize(const ticker_data_t *ticker)
 
     uint8_t frequency_shifts = 0;
     for (uint8_t i = 31; i > 0; --i) {
-        if ((1 << i) == frequency) {
+        if ((1U << i) == frequency) {
             frequency_shifts = i;
             break;
         }
diff --git a/rtos/TARGET_CORTEX/mbed_rtx_handlers.c b/rtos/TARGET_CORTEX/mbed_rtx_handlers.c
@@ -55,8 +55,6 @@ __NO_RETURN void osRtxIdleThread(void *argument)
 
 __NO_RETURN uint32_t osRtxErrorNotify(uint32_t code, void *object_id)
 {
-    osThreadId_t tid = osThreadGetId();
-
     switch (code) {
         case osRtxErrorStackUnderflow:
             // Stack underflow detected for thread (thread_id=object_id)

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ static void initialize(const ticker_data_t *ticker)`
`47`	`47`
`48`	`48`	`uint8_t frequency_shifts = 0;`
`49`	`49`	`for (uint8_t i = 31; i > 0; --i) {`
`50`		`- if ((1 << i) == frequency) {`
	`50`	`+ if ((1U << i) == frequency) {`
`51`	`51`	`frequency_shifts = i;`
`52`	`52`	`break;`
`53`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,6 @@ __NO_RETURN void osRtxIdleThread(void *argument)`
`55`	`55`
`56`	`56`	`__NO_RETURN uint32_t osRtxErrorNotify(uint32_t code, void *object_id)`
`57`	`57`	`{`
`58`		`- osThreadId_t tid = osThreadGetId();`
`59`		`-`
`60`	`58`	`switch (code) {`
`61`	`59`	`case osRtxErrorStackUnderflow:`
`62`	`60`	`// Stack underflow detected for thread (thread_id=object_id)`