<div dir="auto"><div>Îți răspund la prima întrebare: compilerul de Windows poate compila (din câte știu, versiunea de 2010 era un pic dubioasă dar parcă nu avea probleme cu asta) fișiere cu newline Unix fără probleme, deci nu e nevoie să ai cu newline Windows. dos2unix is your friend (și e inclus în Busybox dacă nu vrei un tool separat)<div dir="auto"><br></div><div dir="auto">SPDX cred că e ignorabil în scopul temei but don't count on this. IRL licența este extrem de relevantă. Nu e nevoie să scrii cod GPL în userspace I mean...</div><div dir="auto"><br></div><div dir="auto">Pentru comentarii poți avea /**(newline) *(newline) * Comment(newline) */(newline), fără vreun trailing space. Nu cred că există nicio instanță în care să nu poți evita trailing spade și există chiar transformări mecanice prin care le poți elimina.</div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 28, 2020, 6:34 PM Teodor Popescu via so <<a href="mailto:so@cursuri.cs.pub.ro">so@cursuri.cs.pub.ro</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Bună seara,<br>
<br>
Am 4 întrebări / probleme legate de Checkpatch sau resursele oferite<br>
pe <a href="http://ocw.cs.pub.ro" rel="noreferrer noreferrer" target="_blank">ocw.cs.pub.ro</a> legate de Checkpatch:<br>
<br>
1) În script-ul wrapper peste checkpatch[1], pentru LIN_IGNORE_FLAGS,<br>
nu se va ignora DOS_LINE_ENDINGS. În condițiile în care noi trimitem<br>
același cod pentru ambele sisteme, iar Windows are devoie de "\t\n",<br>
nu ar trebui ca ambele să ifnore DOS_LINE_ENDINGS?<br>
<br>
2) Tot legat de IGNORE_FLAGS, nu cred că SPDX_LICENSE_TAG ne<br>
interesează în acest moment, dar poluează consola cu erori de pomană.<br>
Îl putem adăuga în listele de IGNORE_FLAGS?<br>
<br>
3) Folosesc pentru a documenta anumite secțiuni de cod blocuri de<br>
comentarii după formatul definit aici[2]. Pe fiecare linie a acestor<br>
comentarii cu excepția celei de închidere (" */ "), și exclusiv pe<br>
aceste linii, îmi este semnalată o eroare TRAILING_WHITESPACE.<br>
Nu am găsit nicio mențiune legată de această problemă pe Google.<br>
<br>
4) Link-ul către exemple de utilizare a script-ului Checkpatch[3] de<br>
aici[4] nu pare să mai fie relevant. Nu sunt sigur dacă îmi scapă ceva<br>
sau dacă s-a modificat pur și simlplu resursa.<br>
<br>
Mulțumesc frumos anticipat<br>
<br>
[1]: <a href="https://raw.githubusercontent.com/systems-cs-pub-ro/so-assignments/master/checkpatch_wrapper.sh" rel="noreferrer noreferrer" target="_blank">https://raw.githubusercontent.com/systems-cs-pub-ro/so-assignments/master/checkpatch_wrapper.sh</a><br>
[2]: <a href="https://www.kernel.org/doc/html/v4.10/doc-guide/kernel-doc.html#function-documentation" rel="noreferrer noreferrer" target="_blank">https://www.kernel.org/doc/html/v4.10/doc-guide/kernel-doc.html#function-documentation</a><br>
[3]: <a href="http://www.tuxradar.com/content/newbies-guide-hacking-linux-kernel" rel="noreferrer noreferrer" target="_blank">http://www.tuxradar.com/content/newbies-guide-hacking-linux-kernel</a><br>
[4]: <a href="https://ocw.cs.pub.ro/courses/so/teme/general#coding_style" rel="noreferrer noreferrer" target="_blank">https://ocw.cs.pub.ro/courses/so/teme/general#coding_style</a><br>
<br>
Seară frumoasă<br>
Teodor Popescu<br>
+40 770 498 496 | teodor.popescu2005 | 335CB<br>
_______________________________________________<br>
<a href="http://ocw.cs.pub.ro/courses/so/info/lista-discutii" rel="noreferrer noreferrer" target="_blank">http://ocw.cs.pub.ro/courses/so/info/lista-discutii</a></blockquote></div></div></div>