CRhinoDisplayPipeline::MeasureString - can we get sensible defaults?

Serengeti (Rhino WIP) Developer
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).

2

@jeff, any comments about this?

3

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

4

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

5

RH-37026 is fixed in the latest WIP

1 Like