Page MenuHomeFreeBSD

[sockbuf] Don't access fields directly, use accessor functions
ClosedPublic

Authored by davide on Feb 8 2015, 10:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 14, 6:09 AM
Unknown Object (File)
Sat, May 11, 3:32 PM
Unknown Object (File)
Thu, May 9, 2:06 AM
Unknown Object (File)
Sun, Apr 28, 6:33 AM
Unknown Object (File)
Sat, Apr 27, 4:14 AM
Unknown Object (File)
Fri, Apr 26, 11:35 PM
Unknown Object (File)
Apr 13 2024, 1:21 AM
Unknown Object (File)
Apr 10 2024, 8:24 PM

Details

Summary

Small step towards a saner sockbuf abstraction

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

davide updated this revision to Diff 3695.
davide retitled this revision from to [sockbuf] Don't access fields directly, use accessor functions.
davide updated this object.
davide edited the test plan for this revision. (Show Details)
davide added reviewers: kmacy, np, lstewart, rrs, rwatson.
davide set the repository for this revision to rS FreeBSD src repository - subversion.
davide added a subscriber: Unknown Object (MLST).

Updated removing the last place where this was used. I was not certain about it before so I previously left it as-is, but it looks like in this case order doesn't matter considering we're holding the socket buffer lock across the two calls.

I plan to commit this in two days or such, so if there are objections, please raise them.

julian added a reviewer: julian.
This revision is now accepted and ready to land.Feb 11 2015, 8:01 AM
adrian added a reviewer: adrian.
rwatson edited edge metadata.

Seems sensible.