Project

General

Profile

Actions
Copy link

Cleanup #2344

closed

convert Monitor maps to use ENCODE_START

Added by Greg Farnum almost 12 years ago. Updated almost 12 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Monitor
Target version:
v0.48
% Done:

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

Looks like the Monitor stuff doesn't use the ENCODE_START idiom at all, and it should.

Actions
Copy link
 #1

Updated by Sage Weil almost 12 years ago

  • Status changed from New to Fix Under Review

wip-mon-encoding

Actions
Copy link
 #2

Updated by Greg Farnum almost 12 years ago

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.

Actions
Copy link
 #3

Updated by Sage Weil almost 12 years ago

  • Status changed from Fix Under Review to In Progress
  • Translation missing: en.field_position set to 1228
Actions
Copy link
 #4

Updated by Sage Weil almost 12 years ago

  • Target version changed from v0.47 to v0.48
Actions
Copy link
 #5

Updated by Greg Farnum almost 12 years ago

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. :)

Actions
Copy link
 #6

Updated by Sage Weil almost 12 years ago

  • Status changed from In Progress to Fix Under Review
Actions
Copy link
 #7

Updated by Greg Farnum almost 12 years ago

Looks good. Reviewed-by: Greg Farnum. :)

Actions
Copy link
 #8

Updated by Sage Weil almost 12 years ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link

Also available in: Atom PDF