Mark EnumerateMetafileProc's callbackData parameter as nullable

[jkoritzinsky]

Follow up to #71225

[dotnet-issue-labeler]

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

[ghost]

Tagging subscribers to this area: @dotnet/area-system-drawing
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Follow up to #71225

Author:	jkoritzinsky
Assignees:	jkoritzinsky
Labels:	area-System.Drawing, new-api-needs-documentation
Milestone:	-

[stephentoub]

Any idea why Graphics.cs is showing as every line having been edited?

[jkoritzinsky]

No clue. I edited it on the online editor. I'll retry the change from a local clone tomorrow.

[jkotas]

Any idea why Graphics.cs is showing as every line having been edited?

Unix newlines got converted to Windows newlines. You can tell by looking at the file size on Unix or by opening the file on Unix in editor that is able to show the difference. You won't see any difference on Windows if you use default git settings. git converts Unix new lines to Windows new lines behind the scenes on Windows by default.

jkotas@jkotas11:~/runtime/src/libraries/System.Drawing.Common/src/System/Drawing$ ls -la Graphics.cs
-rw-r--r-- 1 jkotas jkotas 128939 Jun 23 22:01 Graphics.cs
jkotas@jkotas11:~/runtime/src/libraries/System.Drawing.Common/src/System/Drawing$ git checkout nullable-callbackdata
Switched to branch 'nullable-callbackdata'
Your branch is up to date with 'upstream/nullable-callbackdata'.
jkotas@jkotas11:~/runtime/src/libraries/System.Drawing.Common/src/System/Drawing$ ls -la Graphics.cs
-rw-r--r-- 1 jkotas jkotas 132393 Jun 23 22:02 Graphics.cs

[stephentoub]

I fixed it.

[jkoritzinsky]

Failures are #70969 and #70549 (comment)