whoops fields

Author: srawlins

lib/src/rules/prefer_void_to_null.dart

@@ -140,7 +140,9 @@ class _Visitor extends SimpleAstVisitor<void> {
       return;
     }
 
-    if (parent is VariableDeclarationList && node == parent.type) {
+    if (parent is VariableDeclarationList &&
+        node == parent.type &&
+        parent.parent is! FieldDeclaration) {
       // We should not recommend to use `void` for local or top-level variables.
       return;
     }

test_data/rules/prefer_void_to_null.dart

@@ -52,8 +52,6 @@ class F implements E {
 }
 
 void void_; // OK
-Null null_; // LINT
-core.Null core_null; // LINT
 Future<void>? future_void; // OK
 Future<Null>? future_null; // LINT
 Future<core.Null>? future_core_null; // LINT
@@ -72,8 +70,6 @@ void Function(Future<Null>)? voidFunctionFutureNull; // LINT
 
 usage() {
   void void_; // OK
-  Null null_; // LINT
-  core.Null core_null; // LINT
   Future<void>? future_void; // OK
   Future<Null> future_null; // LINT
   Future<core.Null> future_core_null; // LINT
@@ -152,8 +148,6 @@ class AsMembers {
 
   void usage() {
     void void_; // OK
-    Null null_; // LINT
-    core.Null core_null; // LINT
     Future<void>? future_void; // OK
     Future<Null> future_null; // LINT
     Future<core.Null> future_core_null; // LINT