diff --git a/src/bgen/BindingTouch.cs b/src/bgen/BindingTouch.cs index b93e319dc9dc..3a1d8f404b71 100644 --- a/src/bgen/BindingTouch.cs +++ b/src/bgen/BindingTouch.cs @@ -231,8 +231,9 @@ int Main3 (string [] args) ErrorHelper.ClearWarningLevels (); BindingTouchConfig bindingTouchConfig = new(); // TODO CreateOptionSet creates that value in the datablob. Is that really necessary? - if (!CreateOptionSet (bindingTouchConfig, args) || !InitializeApi (ref bindingTouchConfig) || !InitializeManagers (ref bindingTouchConfig) || !TestLinkWith (bindingTouchConfig)) - return 1; // TODO Notate in the PR every time program returns 1 or 0. + if (!CreateOptionSet (bindingTouchConfig, args) || !InitializeApi (ref bindingTouchConfig) || + !InitializeManagers (ref bindingTouchConfig) || !TestLinkWith (bindingTouchConfig)) + return 1; // TODO Notate in the PR every time program returns 1 or 0. Also, highly disagreeable to have this "Main3" return numbers PopulateTypesAndStrongDictionaries( bindingTouchConfig.api, bindingTouchConfig.process_enums, out List types2, out List strong_dictionaries2); PerformGenerate (ref bindingTouchConfig, types2, strong_dictionaries2); @@ -306,7 +307,6 @@ private List CreateCompilationArguments (BindingTouchConfig bindingTouch private bool CreateOptionSet ( BindingTouchConfig bindingTouchConfig, string[] args) { - //BindingTouchConfig bindingTouchConfig = ogBlob; bindingTouchConfig.os = new OptionSet () { { "h|?|help", "Displays the help", v => bindingTouchConfig.show_help = true }, { "a", "Include alpha bindings (Obsolete).", v => {}, true }, @@ -493,8 +493,6 @@ private bool InitializeManagers (ref BindingTouchConfig bindingTouchConfig) { attributeManager ??= new AttributeManager (this); Frameworks = new Frameworks (CurrentPlatform); - - typeCache ??= new(universe, Frameworks, CurrentPlatform, bindingTouchConfig.api, universe.CoreAssembly, bindingTouchConfig.baselib, BindThirdPartyLibrary);