<div dir="ltr">Hey i've replied with another patch :) <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 5 Mar 2024 at 10:42, Kinsey Moore <<a href="mailto:kinsey.moore@oarcorp.com">kinsey.moore@oarcorp.com</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"><div dir="ltr"><div>Zack,</div><div>It looks like there's been a little confusion here. Sebastian was requesting that you condense the pair of #ifs into a single #if with && and the defined check still coming first. It is mostly trivial in this instance, but virtually every other example in the related files is of a multi-conditional #if versus nested #ifs where possible. Technically, I think what you have is fine as the RTEMS coding conventions don't directly express a preference on split #ifs versus compound #ifs but it doesn't match the style of the surrounding code.<br></div><div><br></div><div>Kinsey<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 4, 2024 at 7:39 PM zack leung <<a href="mailto:zakthertemsdev@gmail.com" target="_blank">zakthertemsdev@gmail.com</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"><div dir="ltr"><div>ping</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Feb 2024 at 21:33, zack leung <<a href="mailto:zakthertemsdev@gmail.com" target="_blank">zakthertemsdev@gmail.com</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"><div dir="ltr"><div><br></div><div>joel said: <br></div><div><span>   +#if CONFIGURE_TICKS_PER_TIMESLICE <= 0 &&<br>
>     defined(CONFIGURE_TICKS_PER_TIMESLICE)<br>
>     +  #error "CONFIGURE_TICKS_PER_TIMESLICE shall be greater than zero"<br>
>     +#endif<br>
>     +<br>
> <br>
> <br>
> This is modifying the right file but I think it is safer to check that <br>
> it is defined<br>
> before checking its value.<br>
<br></span>
Yes, the defined() check should be first. - sebestian <br><br></div><div>You said this in relation to joel's comment in my previous thread. You can check in either one.  The  previous one used the "and" statement. <br></div><div><br></div><div>Zack <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Feb 2024 at 02:08, Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</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 19.02.24 03:13, <a href="mailto:zakthertemsdev@gmail.com" target="_blank">zakthertemsdev@gmail.com</a> wrote:<br>
> +#if defined(CONFIGURE_TICKS_PER_TIMESLICE)<br>
> +  #if CONFIGURE_TICKS_PER_TIMESLICE <= 0<br>
> +    #error "CONFIGURE_TICKS_PER_TIMESLICE shall be greater than zero"<br>
> +  #endif<br>
> +#endif<br>
<br>
This should be<br>
<br>
#if defined(CONFIGURE_TICKS_PER_TIMESLICE) && <br>
CONFIGURE_TICKS_PER_TIMESLICE <= 0<br>
<br>
from my point of view. Just check the style of the related header files <br>
for a reference.<br>
<br>
-- <br>
embedded brains GmbH & Co. KG<br>
Herr Sebastian HUBER<br>
Dornierstr. 4<br>
82178 Puchheim<br>
Germany<br>
email: <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
phone: +49-89-18 94 741 - 16<br>
fax:   +49-89-18 94 741 - 08<br>
<br>
Registergericht: Amtsgericht München<br>
Registernummer: HRB 157899<br>
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler<br>
Unsere Datenschutzerklärung finden Sie hier:<br>
<a href="https://embedded-brains.de/datenschutzerklaerung/" rel="noreferrer" target="_blank">https://embedded-brains.de/datenschutzerklaerung/</a><br>
</blockquote></div>
</blockquote></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>
</blockquote></div>