eda: Fixes for cross-bit compiling; ecere: Took out non-sense code no longer needed
authorJerome St-Louis <jerome@ecere.com>
Mon, 22 Apr 2013 04:12:00 +0000 (00:12 -0400)
committerJerome St-Louis <jerome@ecere.com>
Mon, 22 Apr 2013 04:12:00 +0000 (00:12 -0400)
- eda: Moved private members to the end when there area public ata members
- ecere: if(!this || this) -- this non-sense code was needed due to some old bug, but should no longer be required

ecere/locale/ecere.pot
ecere/src/gui/Window.ec
ecere/src/sys/Time.ec
eda/libeda/locale/EDA.pot
eda/libeda/src/ers.ec
eda/libeda/src/gui.ec
eda/libeda/src/idList.ec

index 0357463..9dba9da 100644 (file)
@@ -875,24 +875,24 @@ msgid "\n\nWould you like to view the error log?"
 msgstr "\n\nWould you like to view the error log?"
 
 #: ./src/sys/Date.ec:121
-#: ./src/sys/Time.ec:785
+#: ./src/sys/Time.ec:777
 msgid "now"
 msgstr "now"
 
 #: ./src/sys/Date.ec:120
-#: ./src/sys/Time.ec:784
+#: ./src/sys/Time.ec:776
 msgid "today"
 msgstr "today"
 
 #: ./src/sys/Date.ec:122
 #: ./src/sys/Date.ec:126
-#: ./src/sys/Time.ec:787
+#: ./src/sys/Time.ec:779
 msgid "tomorrow"
 msgstr "tomorrow"
 
 #: ./src/sys/Date.ec:123
 #: ./src/sys/Date.ec:131
-#: ./src/sys/Time.ec:794
+#: ./src/sys/Time.ec:786
 msgid "yesterday"
 msgstr "yesterday"
 
index 26591e7..876d221 100644 (file)
@@ -9367,7 +9367,7 @@ private:
    Size scrollArea;        // Virtual Scroll area size
    Size reqScrollArea;     // Requested virtual area size
    Point scroll;           // Virtual area scrolling position
-   public ScrollBar sbh, sbv;        // Scrollbar window handles
+   ScrollBar sbh, sbv;        // Scrollbar window handles
    Cursor cursor;        // Mouse cursor used for this window
    WindowState state;
    PopupMenu menuBar;
index 26ea723..e5c3ef1 100644 (file)
@@ -293,15 +293,11 @@ public class SecSince1970 : int64
 {
    char * OnGetString(char * tempString, void * fieldData, bool * needClass)
    {
-      if(this || !this)
-      {
-         // TOFIX:  passing argument 2 of '__ecereProp___ecereNameSpace__ecere__sys__DateTime_Set___ecereNameSpace__ecere__sys__SecSince1970' makes integer from pointer without a cast
-         DateTime t = this;
-         return t.OnGetString(tempString, fieldData, needClass);
-         // TOFIX:
-         // return ((DateTime)this).OnGetString(tempString, fieldData, needClass);
-      }
-      return tempString;
+      // TOFIX:  passing argument 2 of '__ecereProp___ecereNameSpace__ecere__sys__DateTime_Set___ecereNameSpace__ecere__sys__SecSince1970' makes integer from pointer without a cast
+      DateTime t = this;
+      return t.OnGetString(tempString, fieldData, needClass);
+      // TOFIX:
+      // return ((DateTime)this).OnGetString(tempString, fieldData, needClass);
    }
 
    // Is this required?
@@ -430,14 +426,10 @@ public class TimeStamp32 : uint32
 public:
    char * OnGetString(char * tempString, void * fieldData, bool * needClass)
    {
-      if(this || !this)
-      {
-         DateTime t = (SecSince1970)(int)this;
-         return t.OnGetString(tempString, fieldData, needClass);
-         // TOFIX:
-         // return ((DateTime)this).OnGetString(tempString, fieldData, needClass);
-      }
-      return tempString;
+      DateTime t = (SecSince1970)(int)this;
+      return t.OnGetString(tempString, fieldData, needClass);
+      // TOFIX:
+      // return ((DateTime)this).OnGetString(tempString, fieldData, needClass);
    }
 
    // Is this required?
index 6e3056d..23976f7 100644 (file)
@@ -10,21 +10,21 @@ msgstr ""
 "Content-Transfer-Encoding: 8bit\n"
 "X-Poedit-Basepath: ../\n"
 
-#: ./src/idList.ec:231
+#: ./src/idList.ec:228
 msgid "(Click to add a new %s...)"
 msgstr "(Click to add a new %s...)"
 
-#: ./src/ers.ec:564
+#: ./src/ers.ec:566
 msgid "All files"
 msgstr "All files"
 
 #: ./src/ers.ec:28
 #: ./src/gui.ec:20
-#: ./src/gui.ec:30
+#: ./src/gui.ec:31
 msgid "Arial"
 msgstr "Arial"
 
-#: ./src/ers.ec:561
+#: ./src/ers.ec:563
 msgid "Comma Separated Values Spreadsheet (*.csv)"
 msgstr "Comma Separated Values Spreadsheet (*.csv)"
 
@@ -36,11 +36,11 @@ msgstr "Database file (%s) could not be created.\n"
 msgid "Database file (%s) could not be opened.\n"
 msgstr "Database file (%s) could not be opened.\n"
 
-#: ./src/gui.ec:809
+#: ./src/gui.ec:812
 msgid "Delete"
 msgstr "Delete"
 
-#: ./src/gui.ec:940
+#: ./src/gui.ec:946
 msgid "Entry"
 msgstr "Entry"
 
@@ -48,7 +48,7 @@ msgstr "Entry"
 msgid "Error reading field"
 msgstr "Error reading field"
 
-#: ./src/ers.ec:617
+#: ./src/ers.ec:619
 msgid "Export as Spreadsheet (CSV)"
 msgstr "Export as Spreadsheet (CSV)"
 
@@ -56,17 +56,17 @@ msgstr "Export as Spreadsheet (CSV)"
 msgid "Invalid, corrupted or in use (%s) database file.\n"
 msgstr "Invalid, corrupted or in use (%s) database file.\n"
 
-#: ./src/gui.ec:643
+#: ./src/gui.ec:649
 msgid "List"
 msgstr "List"
 
-#: ./src/gui.ec:688
-#: ./src/gui.ec:798
+#: ./src/gui.ec:691
+#: ./src/gui.ec:801
 msgid "List Editor"
 msgstr "List Editor"
 
 #: ./src/gui/TableEditor.ec:25
-#: ./src/gui.ec:746
+#: ./src/gui.ec:749
 msgid "New"
 msgstr "New"
 
@@ -78,11 +78,11 @@ msgstr "New|id="
 msgid "Please wait while the report is being generated..."
 msgstr "Please wait while the report is being generated..."
 
-#: ./src/gui.ec:984
+#: ./src/gui.ec:990
 msgid "Revert"
 msgstr "Revert"
 
-#: ./src/gui.ec:973
+#: ./src/gui.ec:979
 msgid "Save"
 msgstr "Save"
 
@@ -108,7 +108,7 @@ msgstr "Unable to detect if table exists!\n"
 msgid "WordList match cannot be found in database."
 msgstr "WordList match cannot be found in database."
 
-#: ./src/gui.ec:801
+#: ./src/gui.ec:804
 msgid "You are about to delete an entry.\nDo you wish to continue?"
 msgstr "You are about to delete an entry.\nDo you wish to continue?"
 
@@ -117,16 +117,16 @@ msgid "You are about to permanently remove an entry.\nDo you wish to continue?"
 msgstr "You are about to permanently remove an entry.\nDo you wish to continue?"
 
 #: ./src/gui/TableEditor.ec:340
-#: ./src/gui.ec:688
+#: ./src/gui.ec:691
 msgid "You have modified this entry. Would you like to save it before proceeding?"
 msgstr "You have modified this entry. Would you like to save it before proceeding?"
 
-#: ./src/gui.ec:777
-#: ./src/gui.ec:783
+#: ./src/gui.ec:780
+#: ./src/gui.ec:786
 msgid "[New]"
 msgstr "[New]"
 
-#: ./src/idList.ec:231
+#: ./src/idList.ec:228
 msgid "item"
 msgstr "item"
 
index ef7f3d8..44b4708 100644 (file)
@@ -70,7 +70,6 @@ public class Page : Window
 {
    background = white;
 
-   Orientation orientation;
 public:
    property Orientation orientation
    {
@@ -94,12 +93,15 @@ public:
 
    int headerHeight;
    
+private:
+   Orientation orientation;
 }
 
 public class ReportRender
 {
-   public virtual void Render(ReportDestination destination, Report report);
-   public virtual int GetPageNumber();
+public:
+   virtual void Render(ReportDestination destination, Report report);
+   virtual int GetPageNumber();
 }
 
 static ReportRenderNormal ersCurrentReport;
@@ -468,10 +470,6 @@ private:
 
 public class ReportDestination : Window
 {
-
-   int pageCount;
-
-   List<PreviewPage> pages { };
 public:
    Report report;
 
@@ -483,6 +481,10 @@ public:
 
    virtual void AddPage(Page page);
    virtual Report GetReport() { return null; }
+private:
+   int pageCount;
+
+   List<PreviewPage> pages { };
 }
 
 public class PrintedReport : ReportDestination
@@ -872,7 +874,6 @@ public:
       }
    }
 
-private:   
    ~Report()
    {
       groupings.Free();
index dc6a7f3..7093621 100644 (file)
@@ -27,7 +27,8 @@ public class Group : Window
    tabCycle = true;
    //inactive = true; // TOFIX causes problems...
 
-   public Label title { this, font = { $"Arial", 10, bold = true }, position = { 16, 2 } };
+public:
+   Label title { this, font = { $"Arial", 10, bold = true }, position = { 16, 2 } };
 
    bool OnCreate()
    {
@@ -56,7 +57,6 @@ public class CheckBool : bool
 {
    Window OnEdit(DataBox dataBox, DataBox obsolete, int x, int y, int w, int h, void * userData)
    {
-      if(this || !this) {     // FIXME
       Button button = dataBox.keepEditor ? (Button)obsolete : null;
       if(!button)
       {
@@ -77,8 +77,6 @@ public class CheckBool : bool
       button.checked = this;
       button.Create();
       return button;
-      }
-      return null;
    }
 }
 
@@ -111,14 +109,15 @@ public class TableDropBox : DropBox
    anchor = { left = 130, top = 180, right = shadowS + sgs * 2 };
    borderStyle = deep;
 
-   public uint filter;
-   public bool filtered;
-   public Field nameField;
-   public uint exclusion;
-   public Table table;
-   public Field filterField;
+public:
+   uint filter;
+   bool filtered;
+   Field nameField;
+   uint exclusion;
+   Table table;
+   Field filterField;
 
-   public property uint filter
+   property uint filter
    {
       set
       {
@@ -127,11 +126,11 @@ public class TableDropBox : DropBox
       }
       get { return filter; }
    }
-   public property Field nameField { set { nameField = value; } }
-   public property uint exclusion { set { exclusion = value; } }
-   public property Table table { set { table = value; if(!nameField && value) nameField = value.FindField(defaultNameField); } }
+   property Field nameField { set { nameField = value; } }
+   property uint exclusion { set { exclusion = value; } }
+   property Table table { set { table = value; if(!nameField && value) nameField = value.FindField(defaultNameField); } }
 
-   public virtual void Refill()
+   virtual void Refill()
    {
       Clear();
       if(table)
@@ -224,7 +223,7 @@ public class TableDropBox : DropBox
       Sort(null, 1);
    }
 
-   public property Field filterField { set { filterField = value; } }
+   property Field filterField { set { filterField = value; } }
 
    bool OnKeyHit(Key key, unichar ch)
    {
@@ -251,13 +250,13 @@ public class TableDropBox : DropBox
       return true;
    }
 
-   public void EditNotifyCharsAdded()
+   void EditNotifyCharsAdded()
    {
       if(!editBox.NotifyUpdate || editBox.NotifyUpdate == EditBox::NotifyUpdate)
          editBox.NotifyUpdate = EditNotifyUpdate;
    }
 
-   public void EditNotifyUpdate(EditBox editBox)
+   void EditNotifyUpdate(EditBox editBox)
    {
       DataRow row;
       char * contents = editBox.contents;
@@ -292,24 +291,19 @@ public class DropDataBox : DataBox
    size.h = 24;
    keepEditor = true;
    borderStyle = deep;
-
-   Field nameField;
-   Field filterField;
-   uint exclusion;
-   uint filter;
-   bool filtered;
-   bool showNone;
    showNone = false; //true;
 
-   public property uint filter { set { filtered = true; filter = value; } get { return filter; } }
-   public property bool filtered { set { filtered = value; } }
-   public property uint exclusion { set { exclusion = value; } }
-   public property Field filterField { set { filterField = value; } }
-   public property Field nameField { set { nameField = value; } }
-   public virtual void TableDropBox::RefillFunction();
-   public property bool showNone { set { showNone = value; } }
+public:
+
+   property uint filter { set { filtered = true; filter = value; } get { return filter; } }
+   property bool filtered { set { filtered = value; } }
+   property uint exclusion { set { exclusion = value; } }
+   property Field filterField { set { filterField = value; } }
+   property Field nameField { set { nameField = value; } }
+   virtual void TableDropBox::RefillFunction();
+   property bool showNone { set { showNone = value; } }
 
-   public void Refill()
+   void Refill()
    {
       if(editor)
       {
@@ -340,6 +334,14 @@ public class DropDataBox : DataBox
       dropBox.exclusion = exclusion;
       dropBox.showNone = showNone;
    }
+
+private:
+   Field nameField;
+   Field filterField;
+   uint exclusion;
+   uint filter;
+   bool filtered;
+   bool showNone;
 }
 
 public class EditDropDataBox : DropDataBox
@@ -358,16 +360,13 @@ public class FieldDataBox : DataBox
    anchor = { left = 110, right = shadowS + sgs * 2 };
    borderStyle = deep;
 
-   Field field;
-   int64 dataHolder; // THERE SEEMS TO BE A BUG WHEN ACCESSING row ACROSS .so
-   Row row;
-
-   public property Row row
+public:
+   property Row row
    {
       set { row = value; }
       get { return row; }
    }
-   public property EditSection editor
+   property EditSection editor
    {
       set
       {
@@ -378,7 +377,7 @@ public class FieldDataBox : DataBox
       }
    };
 
-   public property Field field
+   property Field field
    {
       set
       {
@@ -423,7 +422,7 @@ public class FieldDataBox : DataBox
       }
    }
 
-   public void Clear()
+   void Clear()
    {
       if(visible)
       {
@@ -444,7 +443,7 @@ public class FieldDataBox : DataBox
       }
    }
 
-   public void Load()
+   void Load()
    {
       if(visible && field && row)
       {
@@ -467,7 +466,7 @@ public class FieldDataBox : DataBox
       }
    }
 
-   public virtual void Save()
+   virtual void Save()
    {
       bool result;
       if(visible && field && row)
@@ -481,7 +480,7 @@ public class FieldDataBox : DataBox
       }
    }
 
-   public void Init()
+   void Init()
    {
       if(visible && created)
          Refresh();
@@ -516,13 +515,13 @@ public class FieldDataBox : DataBox
       return DataBox::OnKeyHit(key, ch);
    }
 
-   public bool Window::NotifyChanged(bool closingDropDown)
+   bool Window::NotifyChanged(bool closingDropDown)
    {
       modifiedDocument = true;
       return true;
    }
 
-   public bool Window::NotifyModified()
+   bool Window::NotifyModified()
    {
       modifiedDocument = true;
       return true;
@@ -544,6 +543,11 @@ public class FieldDataBox : DataBox
          dataHolder = 0;
       }
    }
+
+private:
+   Field field;
+   int64 dataHolder; // THERE SEEMS(ED?) TO BE A BUG WHEN ACCESSING row ACROSS .so
+   Row row;
 }
 
 public class FieldCheckButton : FieldDataBox
@@ -558,24 +562,18 @@ public class FieldDropDataBox : FieldDataBox
 {
    size.h = 24;
    keepEditor = true;
-
-   Field nameField;
-   Field filterField;
-   uint exclusion;
-   uint filter;
-   bool filtered;
-   bool showNone;
    showNone = true;
 
-   public property uint filter { set { filtered = true; filter = value; } get { return filter; } }
-   public property bool filtered { set { filtered = value; } }
-   public property uint exclusion { set { exclusion = value; } }
-   public property Field filterField { set { filterField = value; } }
-   public property Field nameField { set { nameField = value; } }
-   public virtual void TableDropBox::RefillFunction();
-   public property bool showNone { set { showNone = value; } }
-
-   public void Refill()
+public:
+   property uint filter { set { filtered = true; filter = value; } get { return filter; } }
+   property bool filtered { set { filtered = value; } }
+   property uint exclusion { set { exclusion = value; } }
+   property Field filterField { set { filterField = value; } }
+   property Field nameField { set { nameField = value; } }
+   virtual void TableDropBox::RefillFunction();
+   property bool showNone { set { showNone = value; } }
+
+   void Refill()
    {
       if(editor)
       {
@@ -599,12 +597,20 @@ public class FieldDropDataBox : FieldDataBox
       dropBox.exclusion = exclusion;
       dropBox.showNone = showNone;
    }
+
+private:
+   Field nameField;
+   Field filterField;
+   uint exclusion;
+   uint filter;
+   bool filtered;
+   bool showNone;
 }
 
 public class EditFieldDropDataBox : FieldDropDataBox
 {
    // showNone = false;
-
+public:
    void OnConfigure(TableDropBox dropBox)
    {
       FieldDropDataBox::OnConfigure(dropBox);
@@ -630,7 +636,7 @@ public class EditFieldDropDataBox : FieldDropDataBox
       FieldDataBox::Save();
    }
 
-   public virtual bool OnAddTextEntry(Row row, TableDropBox dropBox, char * entry)
+   virtual bool OnAddTextEntry(Row row, TableDropBox dropBox, char * entry)
    {
       row.Add();
       row.SetData(dropBox.nameField, entry);
@@ -643,10 +649,8 @@ public class ListSection : Group
    text = $"List";
    size = { 710, 287 };
    anchor = { left = sgs, top = 32 + sgs * 3, bottom = 55 + sgs * 3 };
-
-   EditSection editor;
-
-   public property EditSection editor
+public:
+   property EditSection editor
    {
       set
       {
@@ -655,7 +659,7 @@ public class ListSection : Group
       }
    }
 
-   public property Table table
+   property Table table
    {
       set
       {
@@ -680,10 +684,9 @@ public class ListSection : Group
          }
       }
    }
-   Table table;
-   public Field fldId, fldName, fldActive;
+   Field fldId, fldName, fldActive;
 
-   public virtual DialogResult Window::NotifySaveConfirmation(ListSection listSection)
+   virtual DialogResult Window::NotifySaveConfirmation(ListSection listSection)
    {
       return MessageBox { master = this, type = yesNoCancel, text = $"List Editor", contents = $"You have modified this entry. Would you like to save it before proceeding?" }.Modal();
    }
@@ -706,7 +709,7 @@ public class ListSection : Group
       return true;
    }
 
-   public void RefillList()
+   void RefillList()
    {
       list.Clear();
       //if(fldId && fldName)
@@ -719,7 +722,7 @@ public class ListSection : Group
       editor.modifiedDocument = false;
    }
 
-   public virtual void Window::NotifyRefillList(ListSection listSection, Row r)
+   virtual void Window::NotifyRefillList(ListSection listSection, Row r)
    {
       if(listSection.fldId && listSection.fldName)
       {
@@ -739,9 +742,9 @@ public class ListSection : Group
       }
    }
 
-   public virtual bool Window::NotifyNew(ListSection listSection, Row r);
+   virtual bool Window::NotifyNew(ListSection listSection, Row r);
 
-   public ButtonStyle btnNew
+   ButtonStyle btnNew
    {
       this, anchor = { right = shadowS + sgs * 2, top = 24 }, hotKey = altW, text = $"New";
 
@@ -793,7 +796,7 @@ public class ListSection : Group
       }
    };
 
-   public virtual bool Window::NotifyDeleteConfirmation(ListSection listSection)
+   virtual bool Window::NotifyDeleteConfirmation(ListSection listSection)
    {
       return MessageBox {  master = this, type = yesNo, text = $"List Editor", 
                            contents =  $"You are about to delete an entry.\n"
@@ -801,10 +804,10 @@ public class ListSection : Group
                   }.Modal() == yes;
    }
 
-   public virtual void Window::NotifyDeleting(ListSection listSection);
-   public virtual void Window::NotifyDeleted(ListSection listSection);
+   virtual void Window::NotifyDeleting(ListSection listSection);
+   virtual void Window::NotifyDeleted(ListSection listSection);
 
-   public ButtonStyle btnDelete
+   ButtonStyle btnDelete
    {
       this, anchor = { right = shadowS + sgs * 2, top = 24 }, hotKey = altD, text = $"Delete";
 
@@ -830,7 +833,7 @@ public class ListSection : Group
       }
    };
 
-   public bool FilterNotifyChanged(bool closeDropDown)
+   bool FilterNotifyChanged(bool closeDropDown)
    {
       editor.EditClear();
       RefillList();
@@ -840,8 +843,7 @@ public class ListSection : Group
       return true;
    }
 
-   DataRow lastRow;
-   public ListBox list
+   ListBox list
    {
       this, anchor = { left = sgs * 2, top = 22 + 22 + sgs * 4, right = shadowS + sgs * 2, bottom = shadowS + sgs * 2 };
       alwaysHighLight = true;
@@ -875,9 +877,9 @@ public class ListSection : Group
       }
    };
 
-   public virtual void Window::NotifySelectListRow(ListSection listSection, uint64 id);
+   virtual void Window::NotifySelectListRow(ListSection listSection, uint64 id);
    
-   public void SelectListRow(DataRow row)
+   void SelectListRow(DataRow row)
    {
       // Time startTime = GetTime();
       if(row)
@@ -897,14 +899,14 @@ public class ListSection : Group
       // Logf("SelectListRow took %f seconds\n", GetTime() - startTime);
    }
 
-   public void SelectFirst()
+   void SelectFirst()
    {
       if(list.firstRow)
          SelectListRow(list.firstRow);
       RefreshState();
    }
 
-   public void RefreshState()
+   void RefreshState()
    {
       if(editor)
       {
@@ -932,6 +934,10 @@ public class ListSection : Group
       return Window::OnPostCreate();
    }
 
+private:
+   EditSection editor;
+   Table table;
+   DataRow lastRow;
 }
 
 public class EditSection : Group
@@ -966,9 +972,9 @@ public:
    DataRow listRow;
    OldList editBoxes { };
 
-   public Window editArea { this, borderStyle = deep, tabCycle = true, anchor = { left = 8, top = 54, right = 10, bottom = 10 }, hasVertScroll = true, dontHideScroll = true };
+   Window editArea { this, borderStyle = deep, tabCycle = true, anchor = { left = 8, top = 54, right = 10, bottom = 10 }, hasVertScroll = true, dontHideScroll = true };
    
-   public ButtonStyle btnSave
+   ButtonStyle btnSave
    {
       this, anchor = { right = shadowS + sgs * 2, top = 24 }, hotKey = altV, text = $"Save";
 
@@ -979,7 +985,7 @@ public:
       }
    };
 
-   public ButtonStyle btnReload
+   ButtonStyle btnReload
    {
       this, anchor = { left = 10, top = 24 }, hotKey = altV, text = $"Revert";
 
index 272fa33..80fccb5 100644 (file)
@@ -41,131 +41,128 @@ public class Id : uint
 
    Window OnEdit(DataBox dataBox, void * obsolete, int x, int y, int w, int h, void * userData)
    {
-      if(this || !this) {     // FIXME
-         TableDropBox dropBox = dataBox.keepEditor ? (TableDropBox)dataBox.editor /*obsolete*/ : null;
-         if(!dropBox)
-         {
+      TableDropBox dropBox = dataBox.keepEditor ? (TableDropBox)dataBox.editor /*obsolete*/ : null;
+      if(!dropBox)
+      {
 /*
-            if(eClass_IsDerived(dataBox._class, class(FieldDropBox)))
-            {
-               FieldDropBox fieldDropBox = (FieldDropBox)dataBox;
+         if(eClass_IsDerived(dataBox._class, class(FieldDropBox)))
+         {
+            FieldDropBox fieldDropBox = (FieldDropBox)dataBox;
 
-            }
-            else if(eClass_IsDerived(dataBox._class, class(FieldBox)))
-            {
-               FieldBox fieldBox = (FieldBox)dataBox;
+         }
+         else if(eClass_IsDerived(dataBox._class, class(FieldBox)))
+         {
+            FieldBox fieldBox = (FieldBox)dataBox;
 
-            }
+         }
 */
 
-            dropBox = TableDropBox
-            {
-               dataBox, borderStyle = 0, anchor = { 0, 0, 0, 0 }, 
-               modifyVirtualArea = false, activeStipple = false;
-               showNone = true;
-               nameField = *class_data(nameField);
-               table = *class_data(table);
+         dropBox = TableDropBox
+         {
+            dataBox, borderStyle = 0, anchor = { 0, 0, 0, 0 }, 
+            modifyVirtualArea = false, activeStipple = false;
+            showNone = true;
+            nameField = *class_data(nameField);
+            table = *class_data(table);
 
-               bool DataBox::NotifySelect(DropBox control, DataRow row, Modifiers mods)
-               {
-                  // TOFIX: Id is still 32 bit
-                  uint id = (uint)(row ? row.tag : 0);
-                  SetData(&id, mods.closingDropDown);
-                  return true;
-               }
+            bool DataBox::NotifySelect(DropBox control, DataRow row, Modifiers mods)
+            {
+               // TOFIX: Id is still 32 bit
+               uint id = (uint)(row ? row.tag : 0);
+               SetData(&id, mods.closingDropDown);
+               return true;
+            }
 
-               bool DataBox::NotifyTextEntry(DropBox _dropBox, char * string, bool confirmed)
+            bool DataBox::NotifyTextEntry(DropBox _dropBox, char * string, bool confirmed)
+            {
+               TableDropBox dropBox = (TableDropBox)_dropBox;
+               //Table tbl = dropBox.table.db.OpenTable(dropBox.table.name, { tableRows });
+               //if(tbl)
                {
-                  TableDropBox dropBox = (TableDropBox)_dropBox;
-                  //Table tbl = dropBox.table.db.OpenTable(dropBox.table.name, { tableRows });
-                  //if(tbl)
+                  /*FieldIndex indexedFields[1];
+                  Row r { };*/
+                  char * trimmed = new char[strlen(string) + 1];
+                  /*indexedFields[0] = { dropBox.nameField };
+                  tbl.GenerateIndex(1, indexedFields, false);
+                  r.tbl = tbl;*/
+                  DataRow row = null;
+
+                  TrimLSpaces(string, trimmed);
+                  TrimRSpaces(trimmed, trimmed);
+
+                  /*if(r.Find(dropBox.nameField, middle, nil, trimmed))
                   {
-                     /*FieldIndex indexedFields[1];
-                     Row r { };*/
-                     char * trimmed = new char[strlen(string) + 1];
-                     /*indexedFields[0] = { dropBox.nameField };
-                     tbl.GenerateIndex(1, indexedFields, false);
-                     r.tbl = tbl;*/
-                     DataRow row = null;
-
-                     TrimLSpaces(string, trimmed);
-                     TrimRSpaces(trimmed, trimmed);
-
-                     /*if(r.Find(dropBox.nameField, middle, nil, trimmed))
+                     if(dropBox.filterField)
                      {
-                        if(dropBox.filterField)
-                        {
-                           // TODO: Improve this... Multi field?
-                           while(true)
-                           {
-                              DataRow row;
-                              Id id = 0;
-                              Field fldId = dropBox.table.FindField(defaultIdField);
-                              r.GetData(fldId, id);
-                              row = dropBox.FindSubRow(id);
-                              if(row)
-                              {
-                                 dropBox.SelectRow(row);
-                                 break;
-                              }
-                              //if(!r.Find(dropBox.nameField, next, nil, trimmed))
-                              if(!r.Next())
-                                 break;
-                           }
-                        }
-                        else
+                        // TODO: Improve this... Multi field?
+                        while(true)
                         {
+                           DataRow row;
                            Id id = 0;
                            Field fldId = dropBox.table.FindField(defaultIdField);
                            r.GetData(fldId, id);
-                           dropBox.SelectRow(dropBox.FindSubRow(id));
-                        }
-                     }
-                     */
-
-                     {
-                        for(row = dropBox.firstRow; row; row = row.next)
-                        {
-                           char * string = row.string;
-                           if(string && !strcmp(trimmed, string))
+                           row = dropBox.FindSubRow(id);
+                           if(row)
+                           {
+                              dropBox.SelectRow(row);
+                              break;
+                           }
+                           //if(!r.Find(dropBox.nameField, next, nil, trimmed))
+                           if(!r.Next())
                               break;
                         }
                      }
-                     if(row)
+                     else
                      {
-                        dropBox.SelectRow(row);
+                        Id id = 0;
+                        Field fldId = dropBox.table.FindField(defaultIdField);
+                        r.GetData(fldId, id);
+                        dropBox.SelectRow(dropBox.FindSubRow(id));
                      }
-                     else
+                  }
+                  */
+
+                  {
+                     for(row = dropBox.firstRow; row; row = row.next)
                      {
-                        dropBox.changeContents = false;
-                        dropBox.contents = trimmed;
-                        dropBox.SelectRow(null);
-                        dropBox.changeContents = true;
+                        char * string = row.string;
+                        if(string && !strcmp(trimmed, string))
+                           break;
                      }
-                     //delete r;
                   }
-                  return true;
+                  if(row)
+                  {
+                     dropBox.SelectRow(row);
+                  }
+                  else
+                  {
+                     dropBox.changeContents = false;
+                     dropBox.contents = trimmed;
+                     dropBox.SelectRow(null);
+                     dropBox.changeContents = true;
+                  }
+                  //delete r;
                }
-            };
-            if(class_data(Refill))
-               dropBox.Refill = class_data(Refill);
+               return true;
+            }
+         };
+         if(class_data(Refill))
+            dropBox.Refill = class_data(Refill);
 
-            // dropBox.Refill();
-         }
-         dataBox.OnConfigure(dropBox);
-         dropBox.Create();
-         dropBox.currentRow = dropBox.FindSubRow(this);
-         if(!dropBox.currentRow && this)
-            dataBox.SetData((uint *)&this, false);
-         {
-            DataRow r = dropBox.currentRow;
-            if(r)
-               for(r = r.parent; r; r = r.parent)
-                  r.collapsed = false;
-         }
-         return dropBox;
+         // dropBox.Refill();
+      }
+      dataBox.OnConfigure(dropBox);
+      dropBox.Create();
+      dropBox.currentRow = dropBox.FindSubRow(this);
+      if(!dropBox.currentRow && this)
+         dataBox.SetData((uint *)&this, false);
+      {
+         DataRow r = dropBox.currentRow;
+         if(r)
+            for(r = r.parent; r; r = r.parent)
+               r.collapsed = false;
       }
-      return null;
+      return dropBox;
    }
 
    char * OnGetString(char * tempString, void * fieldData, bool * needClass)
@@ -249,7 +246,7 @@ public:
       get { return class_data(type); }
    };
 
-   public void Clear()
+   void Clear()
    {
       if(this)
       {
@@ -258,7 +255,7 @@ public:
       }
    }
    
-   public bool Includes(Id id)
+   bool Includes(Id id)
    {
       if(this)
       {
@@ -270,7 +267,7 @@ public:
       return false;
    }
 
-   public bool Add(Id id)
+   bool Add(Id id)
    {
       int c;
       for(c = 0; c < count; c++)
@@ -285,7 +282,7 @@ public:
       return false;
    }
 
-   public bool Delete(Id id)
+   bool Delete(Id id)
    {
       int c;
       for(c = 0; c < count; c++)
@@ -435,7 +432,6 @@ public:
       /*if(!this)
          this = eInstance_New(_class);
          */
-      if(this || !this)    // FIXME
       {
          Class type = class_data(type);
          list.AddField({ type, editable = true });
@@ -648,10 +644,7 @@ public class StringList
       }
       */
 
-      if(this || !this)    // FIXME
-      {
       list.AddField({ class(char *), editable = true });
-      }
       for(node = strings.first; node; node = node.next)
       {
          r = list.AddRow();