<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-im">> Can you explain this one:<br>
> <br>
>     3. end...() tags should always be empty <br>
<br></span>
They are not really used by cmake and have to be updated every time the <br>
corresponding term (...) is changed, which means additional work and is <br>
often forgotten</blockquote><div><br></div><div>Sorry for being unclear, can you point out which end..() tags are being talked about? Are those endif() and endfunction()?</div><div>I checked, there were no contents in between the parentheses. If I missed somewhere, can you please point out?</div><div><br></div><div>Thanks.</div><div>Prasun<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 11 Jun 2020 at 20:19, Ralf Habacker <<a href="mailto:ralf.habacker@freenet.de">ralf.habacker@freenet.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">Am 11.06.20 um 05:14 schrieb Prasun Kumar:<br>
> Thanks for the review, sir. I will incorporate your suggestions in my <br>
> coding style.<br>
> <br>
> Can you explain this one:<br>
> <br>
>     3. end...() tags should always be empty <br>
<br>
They are not really used by cmake and have to be updated every time the <br>
corresponding term (...) is changed, which means additional work and is <br>
often forgotten<br>
<br>
> And, is this one necessary?:<br>
> <br>
>     4. use 4 spaces, no tabs<br>
 > Is there any specific reason for this in this project?<br>
<br>
The display of tabs depends on the settings of the used editor and will <br>
often lead to formatting issues and are often not visible as required in <br>
review tools.<br>
<br>
> I have always use tabs and I know there are different views on this matter,<br>
> but I could use spaces if it is required for this project.<br>
<br>
yes, please<br>
<br>
Regards<br>
Ralf<br>
</blockquote></div>