Page MenuHomeFreeBSD

ng_sample: improve styling
ClosedPublic

Authored by p.mousavizadeh_protonmail.com on May 14 2025, 10:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 25, 1:49 PM
Unknown Object (File)
Wed, Nov 19, 5:52 PM
Unknown Object (File)
Wed, Nov 19, 2:57 AM
Unknown Object (File)
Tue, Nov 18, 11:28 PM
Unknown Object (File)
Mon, Nov 17, 10:08 AM
Unknown Object (File)
Mon, Nov 17, 9:29 AM
Unknown Object (File)
Sat, Nov 15, 7:53 AM
Unknown Object (File)
Fri, Nov 14, 2:28 AM

Details

Summary

Most of the developers used the (resp == NULL) style in their code. New developers will try to follow the ng_sample and may use the (!resp) style which is not consistent with other netgraph modules, also not recommended by MISRA-C.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ivy added a subscriber: ivy.
ivy added inline comments.
sys/netgraph/ng_sample.c
323

while you're here, i believe this extra space is a typo that could be removed.

This revision is now accepted and ready to land.May 14 2025, 2:45 PM
kp added a subscriber: kp.

We don't follow MISRA-C, but style(9) does also require this, so the change should be made anyway.

This revision now requires review to proceed.May 15 2025, 2:18 PM
des added a subscriber: des.

@ivy why don't you land this

This revision is now accepted and ready to land.May 16 2025, 12:26 PM
p.mousavizadeh_protonmail.com retitled this revision from ng_sample: As defined in MISRA-C, use resp == NULL to ng_sample: ng_sample: improve styling.May 16 2025, 8:11 PM
p.mousavizadeh_protonmail.com retitled this revision from ng_sample: ng_sample: improve styling to ng_sample: improve styling.May 16 2025, 8:18 PM
This revision was automatically updated to reflect the committed changes.