Or do they serve as useful hints to human readers of the code?
You are of course welcome to keep those you find useful to readers of the
code. My goals with this suggestion was to (a) stop passing garbage in
struct rmid_read fields, (b) use struct rmid_read consistently.
Reinette,
I dug through the code and found only two existing redundant assignments:
rr->val = 0; (in mon_event_read())
and:
rr.first = false; (in mbm_update())
plus a third added by my patch 14:
rr.ci = NULL; (in mbm_update())
None of them seem particularly helpful hints, so I'm dropping
all three in the next rev. of the series.