Page MenuHomeFreeBSD

sys: Nuke double-semicolons
ClosedPublic

Authored by ehem_freebsd_m5p.com on Aug 29 2021, 10:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 1:45 PM
Unknown Object (File)
Mon, Mar 4, 9:27 PM
Unknown Object (File)
Mon, Mar 4, 9:27 PM
Unknown Object (File)
Jan 22 2024, 11:17 AM
Unknown Object (File)
Dec 27 2023, 10:46 AM
Unknown Object (File)
Dec 27 2023, 10:46 AM
Unknown Object (File)
Dec 12 2023, 4:29 PM
Unknown Object (File)
Nov 8 2023, 6:41 AM

Details

Reviewers
manu
emaste
rrs
Group Reviewers
transport
Restricted Owners Package(Owns No Changed Paths)
Commits
rG21cc0918c779: sys: Nuke double-semicolons
Summary

A distinct number of double-semicolons have ended up in FreeBSD. Take a
pass at getting rid of many of these harmless typos.

Diff Detail

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

Event Timeline

Please don't do the pms ones. That's vendor code that needs to be moved to contrib/sys.

The rest look good to me.

Pulling the hunks from sys/dev/pms/RefTisa. Those did look kind of odd.

Pulling one hunk in light of 0637070b5bc, which seems a bit odd.

Pulling one hunk in light of 0637070b5bc, which seems a bit odd.

That one was sitting in a local tree for some time, waiting on replies from maintainers or upstreams.
(same case for 8fa63f44e64ebac444a4ac6451ac5e150cdcf8b1, b1a217a369186dc45c7df7bde13a87868fd5b2c8, 11224884f285481ccecb61f57381b51661d77c6d)

rrs added inline comments.
sys/netinet/sctp_timer.c
332 ↗(On Diff #94397)

Make sure to make Michael Tuexen aware of this change since you are changing here not
the code of SCTP, thats in a different git repo and has a bunch of undef scrubbers that
run on it before submitting to freebsd... so he will need to pull in these changes to the
sctp git repo

tuexen added inline comments.
sys/netinet/sctp_timer.c
332 ↗(On Diff #94397)

I committed this already to the FreeBSD tree and also to the upstream repositories in 454216468583481746845a1c8a45a8c18898ee72

This revision is now accepted and ready to land.Feb 10 2022, 3:22 PM
This revision was automatically updated to reflect the committed changes.
Owners added a reviewer: Restricted Owners Package.Nov 2 2022, 3:36 PM