[ffigen] Remove FfiNativeConfig from Global and Func

pkgs/ffigen/lib/src/code_generator/func.dart

@@ -3,7 +3,6 @@
 // BSD-style license that can be found in the LICENSE file.
 
 import '../code_generator.dart';
-import '../config_provider/config_types.dart';
 import '../visitor/ast.dart';
 
 import 'binding_string.dart';
@@ -47,7 +46,7 @@ class Func extends LookUpBinding {
   final bool isLeaf;
   final bool objCReturnsRetained;
   final bool useNameForLookup;
-  final FfiNativeConfig ffiNativeConfig;
+  final bool loadFromNativeAsset;
   late final String funcPointerName;
 
   /// Contains typealias for function type if [exposeFunctionTypedefs] is true.
@@ -69,7 +68,7 @@ class Func extends LookUpBinding {
     this.objCReturnsRetained = false,
     this.useNameForLookup = false,
     super.isInternal,
-    this.ffiNativeConfig = const FfiNativeConfig(enabled: false),
+    this.loadFromNativeAsset = false,
   }) : functionType = FunctionType(
          returnType: returnType,
          parameters: parameters ?? const [],
@@ -158,7 +157,7 @@ class Func extends LookUpBinding {
       funcImplCall = '$funcVarName($argString)';
     }
 
-    if (ffiNativeConfig.enabled) {
+    if (loadFromNativeAsset) {
       final nativeFuncName = needsWrapper ? funcVarName : enclosingFuncName;
       s.write('''
 ${makeNativeAnnotation(w, nativeType: cType, dartName: nativeFuncName, nativeSymbolName: _lookupName, isLeaf: isLeaf)}
@@ -223,7 +222,7 @@ late final $funcVarName = $funcPointerName.asFunction<$dartType>($isLeafString);
     visitor.visit(functionType);
     visitor.visit(_exposedFunctionTypealias);
     visitor.visit(ffiImport);
-    if (ffiNativeConfig.enabled && exposeSymbolAddress) {
+    if (loadFromNativeAsset && exposeSymbolAddress) {
       visitor.visit(selfImport);
     }
   }

pkgs/ffigen/lib/src/code_generator/func_type.dart

@@ -5,7 +5,6 @@
 import '../code_generator.dart';
 import '../context.dart';
 import '../visitor/ast.dart';
-
 import 'unique_namer.dart';
 
 /// Represents a function type.

pkgs/ffigen/lib/src/code_generator/global.dart

@@ -2,9 +2,7 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-import '../config_provider/config_types.dart';
 import '../visitor/ast.dart';
-
 import 'binding.dart';
 import 'binding_string.dart';
 import 'compound.dart';
@@ -27,7 +25,7 @@ import 'writer.dart';
 class Global extends LookUpBinding {
   final Type type;
   final bool exposeSymbolAddress;
-  final FfiNativeConfig nativeConfig;
+  final bool loadFromNativeAsset;
   final bool constant;
 
   Global({
@@ -38,7 +36,7 @@ class Global extends LookUpBinding {
     super.dartDoc,
     this.exposeSymbolAddress = false,
     this.constant = false,
-    this.nativeConfig = const FfiNativeConfig(enabled: false),
+    this.loadFromNativeAsset = false,
   });
 
   @override
@@ -52,7 +50,7 @@ class Global extends LookUpBinding {
 
     // Removing pointer reference for ConstantArray cType since we always wrap
     // globals with pointer below.
-    final cType = (type is ConstantArray && !nativeConfig.enabled)
+    final cType = (type is ConstantArray && !loadFromNativeAsset)
         ? (type as ConstantArray).child.getCType(context)
         : type.getCType(context);
 
@@ -84,7 +82,7 @@ class Global extends LookUpBinding {
       }
     }
 
-    if (nativeConfig.enabled) {
+    if (loadFromNativeAsset) {
       if (type case final ConstantArray arr) {
         s.writeln(makeArrayAnnotation(w, arr));
       }
@@ -168,7 +166,7 @@ class Global extends LookUpBinding {
     super.visitChildren(visitor);
     visitor.visit(type);
     visitor.visit(ffiImport);
-    if (nativeConfig.enabled && exposeSymbolAddress) {
+    if (loadFromNativeAsset && exposeSymbolAddress) {
       visitor.visit(selfImport);
     }
   }

pkgs/ffigen/lib/src/code_generator/library.dart

@@ -8,7 +8,6 @@ import 'package:yaml_edit/yaml_edit.dart';
 
 import '../code_generator.dart';
 import '../code_generator/utils.dart';
-import '../config_provider/config.dart' show Config;
 import '../config_provider/config_types.dart';
 import '../context.dart';
 
@@ -24,21 +23,20 @@ class Library {
 
   Library._(this.bindings, this.writer, this.context);
 
-  static Library fromConfig({
-    required Config config,
+  static Library fromContext({
     required List<Binding> bindings,
     required Context context,
   }) => Library(
-    name: config.wrapperName,
-    description: config.wrapperDocComment,
+    name: context.config.wrapperName,
+    description: context.config.wrapperDocComment,
     bindings: bindings,
-    header: config.preamble,
+    header: context.config.preamble,
     generateForPackageObjectiveC:
         // ignore: deprecated_member_use_from_same_package
-        config.objectiveC?.generateForPackageObjectiveC ?? false,
-    libraryImports: config.libraryImports.values.toList(),
-    silenceEnumWarning: config.enums.silenceWarning,
-    nativeEntryPoints: config.entryPoints
+        context.config.objectiveC?.generateForPackageObjectiveC ?? false,
+    libraryImports: context.config.libraryImports.values.toList(),
+    silenceEnumWarning: context.config.enums.silenceWarning,
+    nativeEntryPoints: context.config.entryPoints
         .map((uri) => uri.toFilePath())
         .toList(),
     context: context,
@@ -61,18 +59,16 @@ class Library {
     FfiNativeConfig? nativeConfig;
 
     for (final binding in bindings.whereType<LookUpBinding>()) {
-      final nativeConfigForBinding = switch (binding) {
-        Func() => binding.ffiNativeConfig,
-        Global() => binding.nativeConfig,
-        _ => null,
+      final loadFromNativeAsset = switch (binding) {
+        Func() => binding.loadFromNativeAsset,
+        Global() => binding.loadFromNativeAsset,
+        _ => false,
       };
 
       // At the moment, all bindings share their native config.
-      nativeConfig ??= nativeConfigForBinding;
+      if (loadFromNativeAsset) nativeConfig = context.config.ffiNativeConfig;
 
-      final usesLookup =
-          nativeConfigForBinding == null || !nativeConfigForBinding.enabled;
-      (usesLookup ? lookupBindings : nativeBindings).add(binding);
+      (loadFromNativeAsset ? nativeBindings : lookupBindings).add(binding);
     }
     final noLookUpBindings = bindings.whereType<NoLookUpBinding>().toList();
 

pkgs/ffigen/lib/src/code_generator/objc_built_in_functions.dart

@@ -3,7 +3,6 @@
 // BSD-style license that can be found in the LICENSE file.
 
 import '../code_generator.dart';
-import '../config_provider/config_types.dart';
 import '../context.dart';
 import '../visitor/ast.dart';
 
@@ -180,7 +179,7 @@ class ObjCBuiltInFunctions {
     isLeaf: true,
     isInternal: true,
     useNameForLookup: true,
-    ffiNativeConfig: const FfiNativeConfig(enabled: true),
+    loadFromNativeAsset: true,
   );
 
   final _protocolTrampolines = <String, ObjCProtocolMethodTrampoline>{};
@@ -202,7 +201,7 @@ class ObjCBuiltInFunctions {
         isLeaf: false,
         isInternal: true,
         useNameForLookup: true,
-        ffiNativeConfig: const FfiNativeConfig(enabled: true),
+        loadFromNativeAsset: true,
       ),
     );
   }

pkgs/ffigen/lib/src/code_generator/writer.dart

@@ -317,7 +317,7 @@ class Writer {
     bindings.sort((a, b) => a.usr.compareTo(b.usr));
 
     final usesFfiNative = bindings.whereType<Func>().any(
-      (element) => element.ffiNativeConfig.enabled,
+      (element) => element.loadFromNativeAsset,
     );
 
     return {

pkgs/ffigen/lib/src/header_parser/parser.dart

@@ -31,9 +31,8 @@ import 'translation_unit_parser.dart';
 import 'utils.dart';
 
 /// Main entrypoint for header_parser.
-Library parse(Context context) => Library.fromConfig(
+Library parse(Context context) => Library.fromContext(
   bindings: transformBindings(parseToBindings(context), context),
-  config: context.config,
   context: context,
 );
 

pkgs/ffigen/lib/src/header_parser/sub_parsers/functiondecl_parser.dart

@@ -151,7 +151,7 @@ List<Func> parseFunctionDeclaration(
           exposeFunctionTypedefs: config.functions.includeTypedef(decl),
           isLeaf: config.functions.isLeaf(decl),
           objCReturnsRetained: objCReturnsRetained,
-          ffiNativeConfig: config.ffiNativeConfig,
+          loadFromNativeAsset: config.ffiNativeConfig.enabled,
         ),
       );
     }

pkgs/ffigen/lib/src/header_parser/sub_parsers/var_parser.dart

@@ -47,7 +47,7 @@ Global? parseVarDeclaration(Context context, clang_types.CXCursor cursor) {
     dartDoc: getCursorDocComment(context, cursor),
     exposeSymbolAddress: config.globals.includeSymbolAddress(decl),
     constant: cType.isConstQualified,
-    nativeConfig: config.ffiNativeConfig,
+    loadFromNativeAsset: config.ffiNativeConfig.enabled,
   );
   bindingsIndex.addGlobalVarToSeen(usr, global);