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
F136870467: D1805.diff
Thu, Nov 20, 6:45 AM
F136854287: D1805.diff
Thu, Nov 20, 3:11 AM
Unknown Object (File)
Tue, Nov 18, 3:38 PM
Unknown Object (File)
Thu, Nov 13, 10:44 PM
Unknown Object (File)
Thu, Nov 13, 9:32 AM
Unknown Object (File)
Wed, Nov 5, 1:50 AM
Unknown Object (File)
Sat, Nov 1, 10:55 PM
Unknown Object (File)
Sat, Nov 1, 10:55 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.