<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 23, 2023 at 8:51 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">On 23.01.23 15:01, Joel Sherrill wrote:<br>
> Following that comment precisely means we should remove them if there <br>
> are no functions. That ignores data items and the simple maintenance <br>
> issue of not having an extern C, adding something in the future and <br>
> forgetting to add extern C.<br>
> <br>
> IMO it's just part of the header file template we should always use. <br>
> Deciding when it is absolutely not needed is going to introduce latent <br>
> issues.<br>
So, you prefer to use the template (including header guards) also for <br>
files with just comments?<br></blockquote><div><br></div><div>Are you referring to something like a Doxygen @mainpage which is never</div><div>intended to have code? I'm ok leaving it out of that case.</div><div><br></div><div>If the file has code (defines, etc) or if there is any chance the  file will have it</div><div>added, then yes. I think it is best to follow the pattern. We all have come back</div><div>and added something without realizing it needed to be added.</div><div><br></div><div>I would like to just future proof the header.</div><div><br></div><div>--joel</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
embedded brains GmbH<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></div>