Skip to content

Enable nullable for symreader#401

Open
agocke wants to merge 1 commit intodotnet:mainfrom
agocke:nullable
Open

Enable nullable for symreader#401
agocke wants to merge 1 commit intodotnet:mainfrom
agocke:nullable

Conversation

@agocke
Copy link
Member

@agocke agocke commented Mar 13, 2026

No description provided.

}

throw new DllNotFoundException(loadException.Message, loadException);
throw new DllNotFoundException(loadException!.Message, loadException);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

! is unfortunate here since we already asserted Debug.Assert(loadException != null); above.
Can we configure the project not to report nullable warnings for netstadnard2.0?

public static System.Runtime.InteropServices.ComTypes.IStream ConvertToManaged(IntPtr native)
{
IUnsafeComStream marshalledStream = ComInterfaceMarshaller<IUnsafeComStream>.ConvertToManaged((void*)native);
IUnsafeComStream? marshalledStream = ComInterfaceMarshaller<IUnsafeComStream>.ConvertToManaged((void*)native);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

var instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants