<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 25, 2023 at 5:53 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Jul 25, 2023 at 4:48 PM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
><br>
> I may have missed something. Commented in one place.<br>
><br>
> It looks like mostly spaces inside () and variable/parameter declaration changes.<br>
><br>
Yes, for the most part those are the least consistent so far.<br>
<br>
> On Tue, Jul 25, 2023 at 4:38 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>> diff --git a/cpukit/score/cpu/arm/armv7m-isr-dispatch.c b/cpukit/score/cpu/arm/armv7m-isr-dispatch.c<br>
>> index ea168969ba..dfc125d545 100644<br>
>> --- a/cpukit/score/cpu/arm/armv7m-isr-dispatch.c<br>
>> +++ b/cpukit/score/cpu/arm/armv7m-isr-dispatch.c<br>
>> @@ -43,7 +43,7 @@<br>
>><br>
>>  #ifdef ARM_MULTILIB_ARCH_V7M<br>
>><br>
>> -static void __attribute__((naked)) _ARMV7M_Thread_dispatch( void )<br>
>> +static void __attribute__((naked)) _ARMV7M_Thread_dispatch(void)<br>
>>  {<br>
>>    __asm__ volatile (<br>
>>      "bl _Thread_Dispatch\n"<br>
>> @@ -53,7 +53,7 @@ static void __attribute__((naked)) _ARMV7M_Thread_dispatch( void )<br>
>>    );<br>
>>  }<br>
>><br>
>> -static void _ARMV7M_Trigger_lazy_floating_point_context_save( void )<br>
>> +static void _ARMV7M_Trigger_lazy_floating_point_context_save(void)<br>
>>  {<br>
>>  #ifdef ARM_MULTILIB_VFP<br>
>>    __asm__ volatile (<br>
>> @@ -62,7 +62,7 @@ static void _ARMV7M_Trigger_lazy_floating_point_context_save( void )<br>
>>  #endif<br>
>>  }<br>
>><br>
>> -void _ARMV7M_Pendable_service_call( void )<br>
>> +void _ARMV7M_Pendable_service_call(void)<br>
>>  {<br>
>>    Per_CPU_Control *cpu_self = _Per_CPU_Get();<br>
>><br>
>> @@ -73,7 +73,7 @@ void _ARMV7M_Pendable_service_call( void )<br>
>>     * this interrupt service may be delayed until interrupts are enable again.<br>
>>     */<br>
>>    if (<br>
>> -    ( cpu_self->isr_nest_level | cpu_self->thread_dispatch_disable_level ) == 0<br>
>> +    (cpu_self->isr_nest_level | cpu_self->thread_dispatch_disable_level) == 0<br>
>>    ) {<br>
><br>
><br>
> Does this fit on a single line?<br>
><br>
No. it's like two characters short. In fact, i had to do this one<br>
manually. otherwise, it breaks as<br>
if ( (...<br>
    ) == 0 ) {<br></blockquote><div><br></div><div>! instead of == 0? :) </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Ignoring the fact it is using bitwise operations on two integer counters. Perhaps<br>
> it should be a +?<br>
><br>
separate problem I suppose. That is a little bit of a suspicious bit of logic.<br>
</blockquote></div></div>