[API Proposal]: Trim friendly Font ToLogFont/FromLogFont

Background and motivation

Right now WinForm use System.Drawing.Common.Font.ToLogFont/FromLogFont with their own interop structures. That API produce trimming warning. This API would serve same purpose as existing API, but provide trim friendly alternative.

API Proposal

namespace System.Drawing;

public sealed partial class Font
{
    public void ToLogFont<T>(ref T logFont, Graphics graphics) where T: struct, unmanaged  { throw null; }
    public static Font FromLogFont<T>(in T logFont) where T: struct, unmanaged { throw null; }
    public static Font FromLogFont<T>(in T logFont, IntPtr hdc) where T: struct, unmanaged { throw null; }
    public void ToLogFont<T>(ref T logFont) where T: struct, unmanaged { throw null; }
}

API Usage

[StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
public unsafe struct MyLOGFONT
{
    public int lfHeight;
    public int lfWidth;
    public int lfEscapement;
    public int lfOrientation;
    public int lfWeight;
    public byte lfItalic;
    public byte lfUnderline;
    public byte lfStrikeOut;
    public byte lfCharSet;
    public byte lfOutPrecision;
    public byte lfClipPrecision;
    public byte lfQuality;
    public byte lfPitchAndFamily;
    private fixed char _lfFaceName[LF_FACESIZE];
}

// And similar 
var f = new Font();
MyLOGFONT interopStruct = default;
f.ToLogFont(ref interopStruct);

Alternative Designs

Ask users of this API, reimplement it itself within their application and make that code trim-friendly.

Risks

No response

Accompanying implementation dotnet/runtime#70224

[JeremyKuhne]

I’ve implemented this and will be putting up a PR tomorrow.

[bartonjs]

Video

• We made Interop.LOGFONT public and removed the genericness of the functions

namespace System.Drawing
{
    public sealed partial class Font
    {
        public void ToLogFont(out Interop.LOGFONT logFont, Graphics graphics)  { throw null; }
        public static Font FromLogFont(in Interop.LOGFONT logFont) { throw null; }
        public static Font FromLogFont(in Interop.LOGFONT logFont, IntPtr hdc) { throw null; }
        public void ToLogFont(out Interop.LOGFONT logFont) { throw null; }
    }
}

namespace System.Drawing.Interop
{
    public unsafe struct LOGFONT
    {
        public int lfHeight;
        public int lfWidth;
        public int lfEscapement;
        public int lfOrientation;
        public int lfWeight;
        public byte lfItalic;
        public byte lfUnderline;
        public byte lfStrikeOut;
        public byte lfCharSet;
        public byte lfOutPrecision;
        public byte lfClipPrecision;
        public byte lfQuality;
        public byte lfPitchAndFamily;
        private fixed char _lfFaceName[LF_FACESIZE];

        [UnscopedRef]
        public Span<char> lfFaceName => MemoryMarshal.CreateSpan(ref _lfFaceName[0], LF_FACESIZE);
    }
}