CRhinoDisplayPipeline::MeasureString - can we get sensible defaults?

papercut

(Menno Deij - van Rijswijk) #1

CRhinoDisplayPipeline::MeasureString has no defaults for middle, height and fontface, but its sibling CRhinoDisplayPipeline::DrawString has these defaults. I propose the following:

CRhinoDisplayPipeline::MeasureString(CRect &rect, const wchar_t* text, ON_2dPoint point, bool middle = false, int heightPixels = 12, const wchar_t* fontface = L"Arial")

(that is, the same defaults).


(Dale Fugier) #2

@jeff, any comments about this?


(Jeff Lasor) #3

If this really helps developers, then I say do it. I have no arguments against doing it.


(Dale Fugier) #4

https://mcneel.myjetbrains.com/youtrack/issue/RH-37026


(Brian Gillespie) #5

RH-37026 is fixed in the latest WIP