[so] [Tema1] Erori Checkpatch

[Paul Olaru]

Îț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)

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...

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.


On Fri, Feb 28, 2020, 6:34 PM Teodor Popescu via so <so at cursuri.cs.pub.ro>
wrote:

> Bună seara,
>
> Am 4 întrebări / probleme legate de Checkpatch sau resursele oferite
> pe ocw.cs.pub.ro legate de Checkpatch:
>
> 1) În script-ul wrapper peste checkpatch[1], pentru LIN_IGNORE_FLAGS,
> nu se va ignora DOS_LINE_ENDINGS. În condițiile în care noi trimitem
> același cod pentru ambele sisteme, iar Windows are devoie de "\t\n",
> nu ar trebui ca ambele să ifnore DOS_LINE_ENDINGS?
>
> 2) Tot legat de IGNORE_FLAGS, nu cred că SPDX_LICENSE_TAG ne
> interesează în acest moment, dar poluează consola cu erori de pomană.
> Îl putem adăuga în listele de IGNORE_FLAGS?
>
> 3) Folosesc pentru a documenta anumite secțiuni de cod blocuri de
> comentarii după formatul definit aici[2]. Pe fiecare linie a acestor
> comentarii cu excepția celei de închidere (" */ "), și exclusiv pe
> aceste linii, îmi este semnalată o eroare TRAILING_WHITESPACE.
>     Nu am găsit nicio mențiune legată de această problemă pe Google.
>
> 4) Link-ul către exemple de utilizare a script-ului Checkpatch[3] de
> aici[4] nu pare să mai fie relevant. Nu sunt sigur dacă îmi scapă ceva
> sau dacă s-a modificat pur și simlplu resursa.
>
> Mulțumesc frumos anticipat
>
> [1]:
> https://raw.githubusercontent.com/systems-cs-pub-ro/so-assignments/master/checkpatch_wrapper.sh
> [2]:
> https://www.kernel.org/doc/html/v4.10/doc-guide/kernel-doc.html#function-documentation
> [3]: http://www.tuxradar.com/content/newbies-guide-hacking-linux-kernel
> [4]: https://ocw.cs.pub.ro/courses/so/teme/general#coding_style
>
> Seară frumoasă
> Teodor Popescu
> +40 770 498 496   |   teodor.popescu2005   |   335CB
> _______________________________________________
> http://ocw.cs.pub.ro/courses/so/info/lista-discutii
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so/attachments/20200228/f6d97c88/attachment.html>