Added some TODOs

IPA.Loader/Config/Stores/GeneratedStore.cs

@@ -288,7 +288,7 @@ namespace IPA.Config.Stores
                 var nonNullables = attrs.Select(o => o as NonNullableAttribute).NonNull();
 
                 name = member.Name;
-                isNullable = memberType.IsConstructedGenericType
+                isNullable = memberType.IsConstructedGenericType // TODO: figure out something to support this in .NET 3
                           && memberType.GetGenericTypeDefinition() == typeof(Nullable<>);
                 allowNull = !nonNullables.Any() && (!memberType.IsValueType || isNullable);
 

IPA.Loader/Loader/PluginManager.cs

@@ -153,7 +153,7 @@ namespace IPA.Loader
         /// <param name="plugin">the plugin to enable</param>
         /// <returns>whether a restart is needed to activate</returns>
         public static bool EnablePlugin(PluginMetadata plugin)
-        {
+        { // TODO: fix some of this behaviour, by adding better support for runtime enable/disable of mods
             if (plugin == null) return false;
 
             if (plugin.IsBare)

IPA.Loader/PluginInterfaces/BeatSaber/IDisablablePlugin.cs

@@ -5,16 +5,6 @@
     /// </summary>
     public interface IDisablablePlugin
     {
-        /*/// <summary>
-        /// Called when a plugin is enabled. This is where you should set up Harmony patches and the like.
-        /// </summary>
-        /// <remarks>
-        /// This will be called after Init, and will be called when the plugin loads normally too.
-        /// When a plugin is disabled at startup, neither this nor Init will be called until it is enabled.
-        /// 
-        /// Init will only ever be called once.
-        /// </remarks>
-        void OnEnable();*/
 
         /// <summary>
         /// Called when a plugin is disabled at runtime. This should disable things like Harmony patches and unsubscribe

IPA.Loader/PluginInterfaces/BeatSaber/IPlugin.cs

@@ -16,7 +16,7 @@ namespace IPA
         /// Called when a plugin is enabled. This is where you should set up Harmony patches and the like.
         /// </summary>
         /// <remarks>
-        /// This will be called after Init, and will be called when the plugin loads normally too.
+        /// This will be called after <c>Init</c>, and will be called when the plugin loads normally too.
         /// When a plugin is disabled at startup, neither this nor Init will be called until it is enabled.
         /// 
         /// Init will only ever be called once.
@@ -28,6 +28,8 @@ namespace IPA
         /// </summary>
         void OnApplicationQuit();
 
+        // TODO: move more of these out of IPlugin
+
         /// <summary>
         /// Gets invoked on every graphic update.
         /// </summary>