Cleanup #2344
closed
convert Monitor maps to use ENCODE_START
Added by Greg Farnum about 12 years ago.
Updated almost 12 years ago.
Description
Looks like the Monitor stuff doesn't use the ENCODE_START idiom at all, and it should.
- Status changed from New to Fix Under Review
The PGMap changes there are fine, but there are several others:
AuthMonitor::Incremental
MonCaps
MonMap
OSDMap
Are the ones I can see in a quick check.
- Status changed from Fix Under Review to In Progress
- Translation missing: en.field_position set to 1228
- Target version changed from v0.47 to v0.48
MonCaps look good; I'm a bit concerned about the "all features" default used for the MonMap (left a note on GitHub) but the rest is good.
Leaves AuthMonitor::Incremental and OSDMap. :)
- Status changed from In Progress to Fix Under Review
Looks good. Reviewed-by: Greg Farnum. :)
- Status changed from Fix Under Review to Resolved
Also available in: Atom
PDF