Tip: don't exec() GuiEditorGui.gui until you need to

In stock TGE 1.3 & 1.4, I've long noticed that during startup, the engine seems to instanciate and then delete one instance of every SimObject class. I'd assumed this was part of IMPLEMENT_CONOBJECT() and just a fact of life, but Tim McClarren commented this morning that it's happening as part of initializing some in-game editor.

Sure enough,
it's due to the instanciation of a GuiControlListPopup in GuiEditorGui.gui,
which initializes its list of all GuiControls by instanciating each SimObject class, then testing the object to see if it's a GuiControl, then deleting it.

i recommend something like the following:

1. delete this line from GuiEditorGui.gui:

GlobalActionMap.bind(keyboard, "f10", GuiEdit);

2. add this to the bottom of default.bind.cs:

$gGuiEditorGuiExeced = false;
function OpenGuiEditor(%val)
{
   if (!$gGuiEditorGuiExeced)
   {
      exec("common/ui/GuiEditorGui.gui");
      $gGuiEditorGuiExeced = true;
   }
   
   GuiEdit(%val);
}

GlobalActionMap.bind   (keyboard, "alt F10", OpenGuiEditor);

this just seems like a cleaner practice, can't hurt startup times, and certainly clarifies the picture if you're trying to do something like count the number of times a class is instanciated.