[so] [Tema1][General] Checkpatch si const struct

Mihai Barbulescu b12mihai at gmail.com
Tue Mar 3 22:31:34 EET 2020


Folosește butonul reply all

Nu știam că ai rulat checkpatch în loc de wrapper. El e făcut de echipa de
so tocmai pentru scopurile de la noi.

E foarte interesant ca cel original da astfel de erori, ia să văd de ce
vrea așa ceva.

În general nu recomand typedef pentru că atunci când citești cod mare vrei
sa distingi struct de tip de date simplu cum ar fi uint8_t


On Tue, Mar 3, 2020, 22:27 Alin-Andrei GEORGESCU (94722) <
alin.georgescu at stud.acs.upb.ro> wrote:

> Ba uite ca dădea warningul acela, cerând să fie const, pe structura „kkmk”,
> așa simplă cum e ea. Mai dădea și pentru node, că nu prea îi plăceau
> typedef-urile, dar asta nu era o problemă așa mare, având alternativă. Am
> rezolvat chestiunile prin rularea wrapperului, care dă mult mai puține
> warninguri și erori decât rularea directă a checkpatch.pl, am specificat
> pe forum.
>
>
>
> Seară bună!
>
>
>
> *From: *Mihai Barbulescu <b12mihai at gmail.com>
> *Sent: *marți, 3 martie 2020 21:37
> *To: *Paul Olaru <olarupaulstelian97 at gmail.com>; Sisteme de Operare
> <so at cursuri.cs.pub.ro>
> *Cc: *Alin-Andrei GEORGESCU (94722) <alin.georgescu at stud.acs.upb.ro>
> *Subject: *Re: [so] [Tema1][General] Checkpatch si const struct
>
>
>
> Ce inseamna "map-ul se modifica si nu ramane constant"? Ca asta e
> tautologie.
>
> Poti arata aici cam cum arata structura sa imi dau si eu seama despre
> ce e vorba. Ca idee daca ai
>
> struct kkmk {
>     int data;
>     struct kkmk *next;
> }
>
> typedef struct node {
>     int val;
>     struct node *next;
> } node_t
>
> Nu ar trebui sa dea erorarea respectiva deci ....
>
> Asta daca nu cumva te-ai apucat sa imi declari structuri const precum
> obiectele prin C++ motiv pentru care urla warningul. Aici nu avem
> referinte "imutabile" - in C ai doar pass by value sau pass by
> pointer.
>
> On Mon, 2 Mar 2020 at 22:39, Paul Olaru via so <so at cursuri.cs.pub.ro>
> wrote:
> >
> > Această eroare în particular poate fi ignorată. Vezi de restul dacă ai.
> >
> >
> >
> > From: Alin-Andrei GEORGESCU (94722) via so
> > Sent: Monday, March 2, 2020 10:38 PM
> > To: so at cursuri.cs.pub.ro
> > Subject: [so] [Tema1][General] Checkpatch si const struct
> >
> >
> >
> > Salut!
> >
> >
> >
> > În implementarea hashmapului am folosit structuri, însă checkpatch are
> un warning atunci când folosesc structuri care nu sunt constante (WARNING:
> struct  should normally be const). Ce ar trebui să fac în cazul acesta,
> ținând cont că mapul se modifică și nu rămâne constant?
> >
> >
> >
> > Mulțumesc,
> >
> > Georgescu Alin
> >
> > 335CA
> >
> >
> >
> > _______________________________________________
> > http://ocw.cs.pub.ro/courses/so/info/lista-discutii
>
>
>
> --
> Cu stimă,
> Mihai Bărbulescu
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so/attachments/20200303/32567f30/attachment.html>


More information about the so mailing list