PIC32MX: compiler xc32 warns about sign-conversion in portmacro.h

iweindesmedt wrote on Saturday, March 25, 2017:

Hello Engineers!

Maybe this is already known, but xc32 warns about the signed 31 in macro portGET_HIGHEST_PRIORITY

...portmacro.h:201:92: warning: conversion to 'UBaseType_t' from 'int' may change the sign of the result [-Wsign-conversion]
  #define portGET_HIGHEST_PRIORITY( uxTopPriority, uxReadyPriorities ) uxTopPriority = ( 31 - _clz( ( uxReadyPriorities ) ) )

I think this should be 31UL

FreeRTOS v9.0.0
compiler XC32 v1.42 (with -Wconversion)

rtel wrote on Saturday, March 25, 2017:

Thanks for the information. It’s not a warning I have seen myself, but
the code has been updated (not checked in yet though).