Refactor CMSIS Includes

[Devilbinder]

I came across redundant else statements and some if statements that can be reworked by Inverting them, in the CMSIS include folder. I believe this will increase the readability and reduce the overall footprint of the code with regards to the amount of lines.

Here is a examples one of the many functions that do this.

CMSIS_5/CMSIS/Core/Include/core_cm1.h

Line 669 in e336766

__STATIC_INLINE uint32_t __NVIC_GetEnableIRQ(IRQn_Type IRQn)

This can be refactored too:

__STATIC_INLINE uint32_t __NVIC_GetEnableIRQ(IRQn_Type IRQn)
{
  if ((int32_t)(IRQn) >= 0)
  {
    return ((uint32_t)(((NVIC->ISER[0U] & (1UL << (((uint32_t)IRQn) & 0x1FUL))) != 0UL) ? 1UL : 0UL));
  }
  return (0U);
}

• Removed redundant else statement.
• Inverted some if statement.
• Changed some return statement formatting.

[ghost]

No specific error is fixed.
It's a personal opinion if the changes make the code more readable.
The Changes should cause no harm.

[ghost]

No specific error is fixed.
It's a personal opinion if the changes make the code more readable.
The Changes should cause no harm.

[JonatanAntoni]

ok to test