Project

General

Profile

Feature #43

Make OSD use new connection-based

Added by Greg Farnum over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
OSD
Target version:
-
Start date:
04/15/2010
Due date:
% Done:

100%

Spent time:
Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

We have a new send_message(Message *m, Connection *c); let's use it in the OSD!

History

#1 Updated by Greg Farnum over 8 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

The OSD now uses this method to send_message whenever possible. There are a bunch of if-else blocks to allow for forwarded MOSDOps, which Sage says is actually disabled, so we may want to eliminate those to remove an if check on most sends, but that's not a huge deal. Still faster than a map lookup.

Also available in: Atom PDF