Tweaks

SourceGen/CodeAnalysis.cs

@@ -43,6 +43,17 @@ namespace SourceGen {
         /// The hints are not used directly by the data analyzer, but the effects they
         /// have on the Anattrib array are.
         /// </summary>
+        /// <remarks>
+        /// This invokes methods in extension scripts to handle things like inline data
+        /// following a JSR.  The added cost is generally low, because the AppDomain security
+        /// sandbox doesn't add a lot of overhead.  Unfortunately this approach is deprecated
+        /// by Microsoft and may break or become unavailable.  If that happens, and we have to
+        /// switch to a sandbox approach with significant overhead, we will most likely want
+        /// to move the code analyzer itself into the sandbox.
+        ///
+        /// For this reason it's best to minimize direct interaction between the code here and
+        /// that elsewhere in the program.
+        /// </remarks>
         public enum TypeHint : sbyte {
             // No hint.  Default value populated in new arrays.
             NoHint = 0,

SourceGen/Exporter.cs

@@ -247,6 +247,8 @@ namespace SourceGen {
             sb.Append(IncludeNotes.ToString());
             sb.Append(";gfx=");
             sb.Append(GenerateImageFiles.ToString());
+            sb.Append(";opWrap=");
+            sb.Append(mFormatter.Config.mOperandWrapLen);
 
             // Not included: pseudo-op definitions; delimiter definitions
 

SourceGen/MainController.cs

@@ -21,6 +21,7 @@ using System.Text;
 using System.Web.Script.Serialization;
 using System.Windows;
 using System.Windows.Input;
+using System.Windows.Media;
 using Microsoft.Win32;
 
 using Asm65;
@@ -28,7 +29,6 @@ using CommonUtil;
 using CommonWPF;
 using SourceGen.Sandbox;
 using SourceGen.WpfGui;
-using System.Windows.Media;
 
 namespace SourceGen {
     /// <summary>
@@ -2718,6 +2718,11 @@ namespace SourceGen {
             UndoableChange uc =
                 UndoableChange.CreateDummyChange(UndoableChange.ReanalysisScope.CodeAndData);
             ApplyChanges(new ChangeSet(uc), false);
+
+            // TODO(someday): this would really be better as a special-case dummy change
+            // that caused the "external files have changed" behavior in ApplyChanges() to fire.
+            // Before we can do that, though, we need a way to propagate the load errors and
+            // compiler warnings out for display.
         }
 
         public void Goto() {