<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Feb 16, 2020 at 12:30 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">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">
  
    
  
  <div>
    <p><br>
    </p>
    <div>On 16/02/2020 05:31, Gedare Bloom
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr" class="gmail_attr">On Sat, Feb 15, 2020 at 4:36 PM
        Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@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
        2020-02-15 03:02, Sebastian Huber wrote:<br>
        > Move the idle thread body configuration constant out of the<br>
        > configuration table.<br>
        > <br>
        > Provide a default definition of the idle thread body
        constant.<br>
        <br>
        May be I am missing something simple here. How would I provide
        an custom <br>
        IDLE task? Is it by providing something like ...<br>
        <br>
        #include <rtems/score/threadidledata.h><br>
        <br>
        const Thread_Idle_body _Thread_Idle_body = My_Idle_body;<br>
        <br>
      </blockquote>
      <div><br>
      </div>
      <div>I think you do it by a configure macro that confdefs picks up
        by this line added:</div>
      <div>+const Thread_Idle_body _Thread_Idle_body =
        CONFIGURE_IDLE_TASK_BODY;</div>
      <div><br>
      </div>
      <div>It needs some doco ;)</div>
    </blockquote>
    <p>Sorry, my problem is that I am so familiar with this stuff that I
      don't need documentation. What do you want to have documented and
      were should it be placed?</p>
    <p>When I see a variable/function definition and want to see the
      Doxygen I just hit an editor key and get it. I don't want to
      duplicate documentation at the variable/function definition level.<br></p></div></blockquote><div> Well, <a href="https://docs.rtems.org/branches/master/c-user/configuring_a_system.html#configure-idle-task-body" target="_blank">https://docs.rtems.org/branches/master/c-user/configuring_a_system.html#configure-idle-task-body</a> hasn't seemed to change, so maybe that is just the point to make in the commit. When we touch things that manipulate configuration, it is important to understand whether or not it impacts the API. Sometimes that is hard for us to tell in patch review just by looking quickly.</div><div><br></div><div>In the broader sense, the method used to initialize the idle task should be explained at <a href="https://docs.rtems.org/branches/master/c-user/initialization.html#the-idle-task">https://docs.rtems.org/branches/master/c-user/initialization.html#the-idle-task</a></div><div><br></div><div><br></div><div><br></div></div></div></div>