From 321e5eb7cba8a475149352addc6e03b1b88b2256 Mon Sep 17 00:00:00 2001 From: Jared Parsons Date: Sat, 15 Aug 2020 02:44:43 -0700 Subject: [PATCH] Fix annotation in DataTableReaderListnener (#40855) When running the new nullable constructor analysis on runtime we discovered what appears to be a missing annotation in `DataTableReaderListener`. Note: the second `if` check in the constructor seems like dead code. It's checking for the field to be non-null before ever setting the field to a value. Probably could be deleted. --- .../src/System/Data/DataTableReaderListener.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libraries/System.Data.Common/src/System/Data/DataTableReaderListener.cs b/src/libraries/System.Data.Common/src/System/Data/DataTableReaderListener.cs index 8d74e38644849..5008247f62d62 100644 --- a/src/libraries/System.Data.Common/src/System/Data/DataTableReaderListener.cs +++ b/src/libraries/System.Data.Common/src/System/Data/DataTableReaderListener.cs @@ -7,7 +7,7 @@ namespace System.Data { internal sealed class DataTableReaderListener { - private DataTable _currentDataTable; + private DataTable? _currentDataTable; private bool _isSubscribed; private readonly WeakReference _readerWeak;