DEADSOFTWARE

cosmetix in dynrecs; fixed mapcvt
[d2df-sdl.git] / src / shared / xdynrec.pas
index 6c4e376194b85e63beb7f3572b4cd389a5446e3e..cdabf24010918c630a1ac82e8e2f6a4c1daeb6ec 100644 (file)
@@ -132,6 +132,7 @@ type
   protected
     // returns `true` for duplicate record id
     function addListItem (rec: TDynRecord): Boolean; inline;
+    function removeListItem (const aid: AnsiString): TDynRecord; // returns nil or removed record
 
   public
     // get string name for the given type
@@ -166,7 +167,7 @@ type
 
     // binary parser and writer (DO NOT USE!)
     procedure parseBinValue (st: TStream);
-    procedure writeBinTo (st: TStream);
+    procedure writeBinTo (var hasLostData: Boolean; st: TStream);
 
   public
     // the following functions are here only for 'mapgen'! DO NOT USE!
@@ -207,6 +208,7 @@ type
     property hasTPrefix: Boolean read mAsT;
     property separatePasFields: Boolean read mSepPosSize;
     property binOfs: Integer read mBinOfs;
+    property equToDefault: Boolean read isDefaultValue;
   end;
 
 
@@ -215,7 +217,7 @@ type
   private
     mOwner: TDynMapDef;
     mId: AnsiString;
-    mName: AnsiString;
+    mTypeName: AnsiString;
     mSize: Integer;
     mFields: TDynFieldList;
     {$IF DEFINED(XDYNREC_USE_FIELDHASH)}
@@ -253,6 +255,8 @@ type
     function getUserVar (const aname: AnsiString): Variant;
     procedure setUserVar (const aname: AnsiString; val: Variant);
 
+    procedure clearRefRecs (rec: TDynRecord);
+
   protected
     function findRecordByTypeId (const atypename, aid: AnsiString): TDynRecord;
     function findRecordNumByType (const atypename: AnsiString; rc: TDynRecord): Integer;
@@ -283,8 +287,22 @@ type
 
     // only for headers: create new record with the given type
     // will return cloned record ready for use, or `nil` on unknown type name
+    // `aid` must not be empty, and must be unique
     function newTypedRecord (const atypename, aid: AnsiString): TDynRecord;
 
+    // remove record with the given type and id
+    // return `true` if record was successfully found and removed
+    // this will do all necessary recref cleanup too
+    // WARNING: not tested yet
+    function removeTypedRecord (const atypename, aid: AnsiString): Boolean;
+
+    //TODO:
+    //  [.] API to create triggers
+    //  [.] API to properly remove triggers (remove trigdata)
+    //  [.] check if `removeTypedRecord()` does the right thing with inline records
+    //  [.] for fields: assigning `recref` should remove previously assigned inline record (record without id)
+    //  [.] other API i forgot
+
   public
     // text parser
     // `beginEaten`: `true` if "{" was eaten
@@ -296,12 +314,12 @@ type
 
     // binary parser and writer (DO NOT USE!)
     procedure parseBinValue (st: TStream; forceData: Boolean=false);
-    procedure writeBinTo (st: TStream; trigbufsz: Integer=-1; onlyFields: Boolean=false);
+    procedure writeBinTo (var hasLostData: Boolean; st: TStream; trigbufsz: Integer=-1; onlyFields: Boolean=false);
 
   public
     property mapdef: TDynMapDef read mOwner;
     property id: AnsiString read mId; // record id in text map
-    property typeName: AnsiString read mName; // record type name (like "panel", or "trigger")
+    property typeName: AnsiString read mTypeName; // record type name (like "panel", or "trigger")
     property has[const aname: AnsiString]: Boolean read hasByName; // do we have field with the given name?
     property count: Integer read getCount; // number of fields in this record
     property field[const aname: AnsiString]: TDynField read getFieldByName; default; // get field by name
@@ -331,7 +349,7 @@ type
   private
     mOwner: TDynMapDef;
     mIsEnum: Boolean;
-    mName: AnsiString;
+    mTypeName: AnsiString;
     mIds: array of AnsiString;
     mVals: array of Integer;
     mMaxName: AnsiString; // MAX field
@@ -359,7 +377,7 @@ type
 
   public
     property mapdef: TDynMapDef read mOwner;
-    property typeName: AnsiString read mName; // enum/bitset type name
+    property typeName: AnsiString read mTypeName; // enum/bitset type name
     property isEnum: Boolean read mIsEnum; // is this enum? `false` means "bitset"
     property has[const aname: AnsiString]: Boolean read hasByName;
     property field[const aname: AnsiString]: Integer read getFieldByName;
@@ -404,7 +422,7 @@ type
   public
     // parse text or binary map, return new header record
     // WARNING! stream must be seekable
-    function parseMap (st: TStream): TDynRecord;
+    function parseMap (st: TStream; wasBinary: PBoolean=nil): TDynRecord;
 
     // returns `true` if the given stream can be a map file
     // stream position is 0 on return
@@ -912,7 +930,7 @@ begin
   if not mHasDefault then
   begin
     if mInternal then exit;
-    raise TDynRecException.CreateFmt('field ''%s'' in record ''%s'' of record type ''%s'' is not set', [mName, mOwner.mId, mOwner.mName]);
+    raise TDynRecException.CreateFmt('field ''%s'' in record ''%s'' of record type ''%s'' is not set', [mName, mOwner.mId, mOwner.mTypeName]);
   end;
   if (mEBS = TEBS.TRec) then mRecRef := mDefRecRef;
   mSVal := mDefSVal;
@@ -967,6 +985,26 @@ begin
 end;
 
 
+function TDynField.removeListItem (const aid: AnsiString): TDynRecord;
+var
+  f, idx: Integer;
+begin
+  result := nil;
+  if mRHash.get(aid, idx) then
+  begin
+    assert((idx >= 0) and (idx < mRVal.count));
+    result := mRVal[idx];
+    // fix hash and list
+    for f := idx+1 to mRVal.count-1 do
+    begin
+      if (Length(mRVal[f].mId) > 0) then mRHash.put(mRVal[f].mId, f-1);
+    end;
+    mRHash.del(aid);
+    mRVal.delete(idx);
+  end;
+end;
+
+
 class function TDynField.getTypeName (t: TType): AnsiString;
 begin
   case t of
@@ -1255,7 +1293,7 @@ begin
 end;
 
 
-procedure TDynField.writeBinTo (st: TStream);
+procedure TDynField.writeBinTo (var hasLostData: Boolean; st: TStream);
 var
   s: AnsiString;
   f: Integer;
@@ -1282,7 +1320,7 @@ begin
             if (mRecRef <> nil) then
             begin
               ws := TSFSMemoryChunkStream.Create(buf, mMaxDim);
-              mRecRef.writeBinTo(ws, mMaxDim); // as trigdata
+              mRecRef.writeBinTo(hasLostData, ws, mMaxDim); // as trigdata
             end;
             st.WriteBuffer(buf^, mMaxDim);
           finally
@@ -1397,15 +1435,9 @@ begin
         exit;
       end;
     TType.TList:
-      begin
-        assert(false);
-        exit;
-      end;
+      raise TDynRecException.Create('cannot write lists to binary format');
     TType.TTrigData:
-      begin
-        assert(false);
-        exit;
-      end;
+      raise TDynRecException.Create('cannot write triggers to binary format (internal error)');
     else raise TDynRecException.Create('ketmar forgot to handle some field type');
   end;
 end;
@@ -1489,7 +1521,7 @@ begin
             begin
               if (es.mVals[f] = mask) then
               begin
-                if not first then wr.put('+') else first := false;
+                if not first then wr.put(' | ') else first := false;
                 wr.put(es.mIds[f]);
                 found := true;
                 break;
@@ -1576,9 +1608,9 @@ begin
           rec := mOwner;
           // find trigger definition
           tfld := rec.trigTypeField();
-          if (tfld = nil) then raise TDynRecException.CreateFmt('triggerdata value for field ''%s'' in record ''%s'' without TriggerType field', [mName, rec.mName]);
+          if (tfld = nil) then raise TDynRecException.CreateFmt('triggerdata value for field ''%s'' in record ''%s'' without TriggerType field', [mName, rec.mTypeName]);
           rc := mOwner.mOwner.trigTypeFor[tfld.mSVal]; // find in mapdef
-          if (rc = nil) then raise TDynRecException.CreateFmt('triggerdata definition for field ''%s'' in record ''%s'' with type ''%s'' not found', [mName, rec.mName, tfld.mSVal]);
+          if (rc = nil) then raise TDynRecException.CreateFmt('triggerdata definition for field ''%s'' in record ''%s'' with type ''%s'' not found', [mName, rec.mTypeName, tfld.mSVal]);
           rc := rc.clone(mOwner.mHeaderRec);
           rc.mHeaderRec := mOwner.mHeaderRec;
           // on error, it will be freed by memowner
@@ -1776,9 +1808,9 @@ begin
             rec := mOwner;
             // find trigger definition
             tfld := rec.trigTypeField();
-            if (tfld = nil) then raise TDynParseException.CreateFmt(pr, 'triggerdata value for field ''%s'' in record ''%s'' without ''type'' field', [mName, rec.mName]);
+            if (tfld = nil) then raise TDynParseException.CreateFmt(pr, 'triggerdata value for field ''%s'' in record ''%s'' without ''type'' field', [mName, rec.mTypeName]);
             rc := mOwner.mOwner.trigTypeFor[tfld.mSVal]; // find in mapdef
-            if (rc = nil) then raise TDynParseException.CreateFmt(pr, 'triggerdata definition for field ''%s'' in record ''%s'' with type ''%s'' not found', [mName, rec.mName, tfld.mSVal]);
+            if (rc = nil) then raise TDynParseException.CreateFmt(pr, 'triggerdata definition for field ''%s'' in record ''%s'' with type ''%s'' not found', [mName, rec.mTypeName, tfld.mSVal]);
             rc := rc.clone(mOwner.mHeaderRec);
             rc.mHeaderRec := mOwner.mHeaderRec;
             //writeln(rc.definition);
@@ -1828,7 +1860,7 @@ begin
           mDefined := true;
           if mOwner.addRecordByType(mEBSTypeName, rc) then
           begin
-            raise TDynParseException.CreateFmt(pr, 'duplicate record with id ''%s'' for field ''%s'' in record ''%s''', [rc.mId, mName, mOwner.mName]);
+            raise TDynParseException.CreateFmt(pr, 'duplicate record with id ''%s'' for field ''%s'' in record ''%s''', [rc.mId, mName, mOwner.mTypeName]);
           end;
           pr.eatTT(pr.TTSemi); // hack: allow (but don't require) semicolon after inline records
           exit;
@@ -1987,7 +2019,7 @@ constructor TDynRecord.Create (pr: TTextParser);
 begin
   if (pr = nil) then raise TDynParseException.Create(pr, 'cannot create record type without type definition');
   mId := '';
-  mName := '';
+  mTypeName := '';
   mSize := 0;
   mFields := TDynFieldList.Create();
   {$IF DEFINED(XDYNREC_USE_FIELDHASH)}
@@ -2005,7 +2037,7 @@ end;
 
 constructor TDynRecord.Create ();
 begin
-  mName := '';
+  mTypeName := '';
   mSize := 0;
   mFields := TDynFieldList.Create();
   {$IF DEFINED(XDYNREC_USE_FIELDHASH)}
@@ -2038,7 +2070,7 @@ begin
     mRec2Free.Free();
     mRec2Free := nil;
   end;
-  mName := '';
+  mTypeName := '';
   for fld in mFields do fld.Free();
   mFields.Free();
   mFields := nil;
@@ -2167,7 +2199,7 @@ begin
   result := TDynRecord.Create();
   result.mOwner := mOwner;
   result.mId := mId;
-  result.mName := mName;
+  result.mTypeName := mTypeName;
   result.mSize := mSize;
   result.mHeader := mHeader;
   result.mBinBlock := mBinBlock;
@@ -2278,7 +2310,7 @@ begin
     if not (fld.mEBSType is TDynEBS) then continue;
     es := (fld.mEBSType as TDynEBS);
     assert(es <> nil);
-    if StrEqu(es.mName, 'TriggerType') then begin result := fld; exit; end;
+    if StrEqu(es.mTypeName, 'TriggerType') then begin result := fld; exit; end;
   end;
   result := nil;
 end;
@@ -2298,14 +2330,67 @@ end;
 function TDynRecord.newTypedRecord (const atypename, aid: AnsiString): TDynRecord;
 var
   trc: TDynRecord;
+  fld: TDynField;
 begin
   if not mHeader then raise TDynRecException.Create('cannot create new records with non-header');
+  if (Length(aid) = 0) then raise TDynRecException.CreateFmt('cannot create new record of type ''%s'' without id', [atypename]);
   trc := mapdef.recType[atypename];
   if (trc = nil) then begin result := nil; exit; end;
+  // check if aid is unique
+  fld := field[atypename];
+  if (fld <> nil) and (fld.getListItem(aid) <> nil) then raise TDynRecException.CreateFmt('cannot create record of type ''%s'' with duplicate id ''%s''', [atypename, aid]);
   result := trc.clone(self);
-  result.mId := ''; // for now
-  addRecordByType(atypename, result);
   result.mId := aid;
+  addRecordByType(atypename, result);
+end;
+
+
+procedure TDynRecord.clearRefRecs (rec: TDynRecord);
+  procedure clearRefs (fld: TDynField);
+  var
+    rc: TDynRecord;
+  begin
+    if (fld = nil) then exit;
+    if (fld.mRecRef = rec) then fld.mRecRef := nil;
+    if (fld.mType = fld.TType.TList) then for rc in fld.mRVal do rc.clearRefRecs(rec);
+  end;
+var
+  fld: TDynField;
+begin
+  if (rec = nil) or (mFields = nil) then exit;
+  for fld in mFields do clearRefs(fld);
+end;
+
+
+// remove record with the given type and id
+// return `true` if record was successfully found and removed
+// this will do all necessary recref cleanup too
+function TDynRecord.removeTypedRecord (const atypename, aid: AnsiString): Boolean;
+var
+  trc, rec: TDynRecord;
+  fld: TDynField;
+  f: Integer;
+  doFree: Boolean = false;
+begin
+  result := false;
+  if not mHeader then raise TDynRecException.Create('cannot remove records with non-header');
+  if (Length(aid) = 0) then exit;
+  trc := mapdef.recType[atypename];
+  if (trc = nil) then exit;
+  fld := field[atypename];
+  if (fld = nil) then exit;
+  rec := fld.removeListItem(aid);
+  if (rec = nil) then exit;
+  clearRefRecs(rec);
+  for f := 0 to mRec2Free.count-1 do
+  begin
+    if (mRec2Free[f] = rec) then
+    begin
+      mRec2Free[f] := nil;
+      doFree := true;
+    end;
+  end;
+  if doFree then rec.Free();
 end;
 
 
@@ -2353,7 +2438,7 @@ begin
         while pr.eatTT(pr.TTComma) do begin end;
         if pr.eatDelim(')') then break;
         tdn := pr.expectId();
-        if isForTrig[tdn] then raise TDynParseException.CreateFmt(pr, 'duplicate trigdata ''%s'' trigtype ''%s''', [mName, tdn]);
+        if isForTrig[tdn] then raise TDynParseException.CreateFmt(pr, 'duplicate trigdata ''%s'' trigtype ''%s''', [mTypeName, tdn]);
         SetLength(mTrigTypes, Length(mTrigTypes)+1);
         mTrigTypes[High(mTrigTypes)] := tdn;
       end;
@@ -2364,27 +2449,27 @@ begin
       SetLength(mTrigTypes, 1);
       mTrigTypes[0] := tdn;
     end;
-    mName := 'TriggerData';
+    mTypeName := 'TriggerData';
   end
   else
   begin
-    mName := pr.expectStrOrId();
+    mTypeName := pr.expectStrOrId();
     while (pr.tokType <> pr.TTBegin) do
     begin
       if pr.eatId('header') then begin mHeader := true; continue; end;
       if pr.eatId('size') then
       begin
-        if (mSize > 0) then raise TDynParseException.CreateFmt(pr, 'duplicate `size` in record ''%s''', [mName]);
+        if (mSize > 0) then raise TDynParseException.CreateFmt(pr, 'duplicate `size` in record ''%s''', [mTypeName]);
         mSize := pr.expectInt();
-        if (mSize < 1) then raise TDynParseException.CreateFmt(pr, 'invalid record ''%s'' size: %d', [mName, mSize]);
+        if (mSize < 1) then raise TDynParseException.CreateFmt(pr, 'invalid record ''%s'' size: %d', [mTypeName, mSize]);
         pr.expectId('bytes');
         continue;
       end;
       if pr.eatId('binblock') then
       begin
-        if (mBinBlock >= 0) then raise TDynParseException.CreateFmt(pr, 'duplicate `binblock` in record ''%s''', [mName]);
+        if (mBinBlock >= 0) then raise TDynParseException.CreateFmt(pr, 'duplicate `binblock` in record ''%s''', [mTypeName]);
         mBinBlock := pr.expectInt();
-        if (mBinBlock < 1) then raise TDynParseException.CreateFmt(pr, 'invalid record ''%s'' binblock: %d', [mName, mBinBlock]);
+        if (mBinBlock < 1) then raise TDynParseException.CreateFmt(pr, 'invalid record ''%s'' binblock: %d', [mTypeName, mBinBlock]);
         continue;
       end;
     end;
@@ -2434,7 +2519,7 @@ begin
   else
   begin
     // record
-    result := quoteStr(mName);
+    result := quoteStr(mTypeName);
     if (mSize >= 0) then result += Format(' size %d bytes', [mSize]);
     if mHeader then result += ' header';
   end;
@@ -2479,7 +2564,7 @@ var
       rt := findRecordByTypeId(fld.mEBSTypeName, fld.mRecRefId);
       if (rt = nil) then
       begin
-        e_LogWritefln('record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mName, rec.mId, fld.mEBSTypeName, fld.mRecRefId], MSG_WARNING);
+        e_LogWritefln('record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mTypeName, rec.mId, fld.mEBSTypeName, fld.mRecRefId], MSG_WARNING);
         //raise TDynRecException.CreateFmt('record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mName, rec.mId, fld.mEBSTypeName, fld.mRecRefId]);
       end;
       //writeln(' ', rec.mName, '.', rec.mId, ':', fld.mName, ' -> ', rt.mName, '.', rt.mId, ' (', fld.mEBSTypeName, '.', fld.mRecRefId, ')');
@@ -2533,7 +2618,7 @@ begin
         else
         begin
           // create list for this type
-          fld := TDynField.Create(rec.mName, TDynField.TType.TList);
+          fld := TDynField.Create(rec.mTypeName, TDynField.TType.TList);
           fld.mOwner := self;
           addField(fld);
           if (bsize > 0) then
@@ -2546,7 +2631,7 @@ begin
               rec := rect.clone(self);
               rec.mHeaderRec := self;
               rec.parseBinValue(mst);
-              rec.mId := Format('%s%d', [rec.mName, f]);
+              rec.mId := Format('%s%d', [rec.mTypeName, f]);
               fld.addListItem(rec);
               //writeln('parsed ''', rec.mId, '''...');
             end;
@@ -2566,8 +2651,8 @@ begin
     end;
 
     // read fields
-    if StrEqu(mName, 'TriggerData') then mSize := Integer(st.size-st.position);
-    if (mSize < 1) then raise TDynRecException.CreateFmt('cannot read record of type ''%s'' with unknown size', [mName]);
+    if StrEqu(mTypeName, 'TriggerData') then mSize := Integer(st.size-st.position);
+    if (mSize < 1) then raise TDynRecException.CreateFmt('cannot read record of type ''%s'' with unknown size', [mTypeName]);
     GetMem(buf, mSize);
     st.ReadBuffer(buf^, mSize);
     for fld in mFields do
@@ -2586,14 +2671,13 @@ begin
 end;
 
 
-procedure TDynRecord.writeBinTo (st: TStream; trigbufsz: Integer=-1; onlyFields: Boolean=false);
+procedure TDynRecord.writeBinTo (var hasLostData: Boolean; st: TStream; trigbufsz: Integer=-1; onlyFields: Boolean=false);
 var
   fld: TDynField;
   rec, rv: TDynRecord;
   buf: PByte = nil;
   ws: TStream = nil;
   blk, blkmax: Integer;
-  //f, c: Integer;
   bufsz: Integer = 0;
   blksz: Integer;
 begin
@@ -2618,11 +2702,15 @@ begin
       // record list?
       if (fld.mType = fld.TType.TList) then continue; // later
       if fld.mInternal then continue;
-      if (fld.mBinOfs < 0) then continue;
+      if (fld.mBinOfs < 0) then
+      begin
+        if not fld.equToDefault then hasLostData := true;
+        continue;
+      end;
       if (fld.mBinOfs >= bufsz) then raise TDynRecException.Create('binary value offset is outside of the buffer');
       TSFSMemoryChunkStream(ws).setup(buf+fld.mBinOfs, bufsz-fld.mBinOfs);
       //writeln('writing field <', fld.mName, '>');
-      fld.writeBinTo(ws);
+      fld.writeBinTo(hasLostData, ws);
     end;
 
     // write block with normal fields
@@ -2672,7 +2760,7 @@ begin
             if (rec = nil) then continue;
             if (rec.mBinBlock <> blk) then continue;
             if (ws = nil) then ws := TMemoryStream.Create();
-            for rv in fld.mRVal do rv.writeBinTo(ws);
+            for rv in fld.mRVal do rv.writeBinTo(hasLostData, ws);
           end;
         end;
         // flush block
@@ -2704,10 +2792,12 @@ procedure TDynRecord.writeTo (wr: TTextWriter; putHeader: Boolean=true);
 var
   fld: TDynField;
   rec: TDynRecord;
+  putTypeComment: Boolean;
+  f: Integer;
 begin
   if putHeader then
   begin
-    wr.put(mName);
+    wr.put(mTypeName);
     if (Length(mId) > 0) then begin wr.put(' '); wr.put(mId); end;
     wr.put(' ');
   end;
@@ -2720,11 +2810,31 @@ begin
       if (fld.mType = fld.TType.TList) then
       begin
         if not mHeader then raise TDynRecException.Create('record list in non-header record');
-        if (fld.mRVal <> nil) then
+        if (fld.mRVal <> nil) and (fld.mRVal.count > 0) then
         begin
+          putTypeComment := true;
           for rec in fld.mRVal do
           begin
-            if (Length(rec.mId) = 0) then continue;
+            if (rec = nil) or (Length(rec.mId) = 0) then continue;
+            if putTypeComment then
+            begin
+              wr.put(#10);
+              if (80-wr.curIndent*2 >= 2) then
+              begin
+                wr.putIndent();
+                for f := wr.curIndent to 80-wr.curIndent do wr.put('/');
+                wr.put(#10);
+              end;
+              putTypeComment := false;
+              wr.putIndent();
+              wr.put('// ');
+              wr.put(fld.name);
+              wr.put(#10);
+            end
+            else
+            begin
+              wr.put(#10);
+            end;
             wr.putIndent();
             rec.writeTo(wr, true);
           end;
@@ -2798,7 +2908,7 @@ var
       if (rt = nil) then
       begin
         //e_LogWritefln('record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mName, rec.mId, fld.mEBSTypeName, fld.mRecRefId], MSG_WARNING);
-        raise TDynParseException.CreateFmt(pr, 'record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mName, rec.mId, fld.mEBSTypeName, fld.mRecRefId]);
+        raise TDynParseException.CreateFmt(pr, 'record of type ''%s'' with id ''%s'' links to inexistant record of type ''%s'' with id ''%s''', [rec.mTypeName, rec.mId, fld.mEBSTypeName, fld.mRecRefId]);
       end;
       //writeln(' ', rec.mName, '.', rec.mId, ':', fld.mName, ' -> ', rt.mName, '.', rt.mId, ' (', fld.mEBSTypeName, '.', fld.mRecRefId, ')');
       fld.mRecRefId := '';
@@ -2813,7 +2923,7 @@ var
   end;
 
 begin
-  if (mOwner = nil) then raise TDynParseException.CreateFmt(pr, 'can''t parse record ''%s'' value without owner', [mName]);
+  if (mOwner = nil) then raise TDynParseException.CreateFmt(pr, 'can''t parse record ''%s'' value without owner', [mTypeName]);
 
   {$IF DEFINED(D2D_DYNREC_PROFILER)}stall := curTimeMicro();{$ENDIF}
 
@@ -2852,7 +2962,7 @@ begin
         pr.skipToken();
         rec.parseValue(pr);
         {$IF DEFINED(D2D_DYNREC_PROFILER)}stt := curTimeMicro();{$ENDIF}
-        addRecordByType(rec.mName, rec);
+        addRecordByType(rec.mTypeName, rec);
         {$IF DEFINED(D2D_DYNREC_PROFILER)}profAddRecByType := curTimeMicro()-stt;{$ENDIF}
         continue;
       end;
@@ -2867,8 +2977,8 @@ begin
     if (fld <> nil) then
     begin
       //writeln('2: <', mName, '.', pr.tokStr, '>');
-      if fld.defined then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in record ''%s''', [fld.mName, mName]);
-      if fld.internal then raise TDynParseException.CreateFmt(pr, 'internal field ''%s'' in record ''%s''', [fld.mName, mName]);
+      if fld.defined then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in record ''%s''', [fld.mName, mTypeName]);
+      if fld.internal then raise TDynParseException.CreateFmt(pr, 'internal field ''%s'' in record ''%s''', [fld.mName, mTypeName]);
       pr.skipToken(); // skip field name
       //writeln('3: <', mName, '.', pr.tokStr, '>:', pr.tokType);
       {$IF DEFINED(D2D_DYNREC_PROFILER)}stt := curTimeMicro();{$ENDIF}
@@ -2878,7 +2988,7 @@ begin
     end;
 
     // something is wrong
-    raise TDynParseException.CreateFmt(pr, 'unknown field ''%s'' in record ''%s''', [pr.tokStr, mName]);
+    raise TDynParseException.CreateFmt(pr, 'unknown field ''%s'' in record ''%s''', [pr.tokStr, mTypeName]);
   end;
   pr.expectTT(pr.TTEnd);
 
@@ -2920,7 +3030,7 @@ end;
 procedure TDynEBS.cleanup ();
 begin
   mIsEnum := false;
-  mName := '';
+  mTypeName := '';
   mIds := nil;
   mVals := nil;
   mMaxName := '';
@@ -2960,7 +3070,7 @@ var
   f, cv: Integer;
 begin
   if mIsEnum then result :='enum ' else result := 'bitset ';
-  result += mName;
+  result += mTypeName;
   result += ' {'#10;
   // fields
   if mIsEnum then cv := 0 else cv := 1;
@@ -2990,7 +3100,7 @@ function TDynEBS.pasdef (): AnsiString;
 var
   f: Integer;
 begin
-  result := '// '+mName+#10'const'#10;
+  result := '// '+mTypeName+#10'const'#10;
   // fields
   for f := 0 to High(mIds) do
   begin
@@ -3022,7 +3132,7 @@ begin
        if pr.eatId('enum') then mIsEnum := true
   else if pr.eatId('bitset') then mIsEnum := false
   else pr.expectId('enum');
-  mName := pr.expectId();
+  mTypeName := pr.expectId();
   mMaxVal := Integer($80000000);
   if mIsEnum then cv := 0 else cv := 1;
   pr.expectTT(pr.TTBegin);
@@ -3031,9 +3141,9 @@ begin
     idname := pr.expectId();
     for f := 0 to High(mIds) do
     begin
-      if StrEqu(mIds[f], idname) then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in enum/bitset ''%s''', [idname, mName]);
+      if StrEqu(mIds[f], idname) then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in enum/bitset ''%s''', [idname, mTypeName]);
     end;
-    if StrEqu(mMaxName, idname) then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in enum/bitset ''%s''', [idname, mName]);
+    if StrEqu(mMaxName, idname) then raise TDynParseException.CreateFmt(pr, 'duplicate field ''%s'' in enum/bitset ''%s''', [idname, mTypeName]);
     skipAdd := false;
     hasV := false;
     v := cv;
@@ -3042,7 +3152,7 @@ begin
     begin
       if pr.eatId('MAX') then
       begin
-        if (Length(mMaxName) > 0) then raise TDynParseException.CreateFmt(pr, 'duplicate max field ''%s'' in enum/bitset ''%s''', [idname, mName]);
+        if (Length(mMaxName) > 0) then raise TDynParseException.CreateFmt(pr, 'duplicate max field ''%s'' in enum/bitset ''%s''', [idname, mTypeName]);
         mMaxName := idname;
         skipAdd := true;
       end
@@ -3311,11 +3421,12 @@ end;
 
 
 // WARNING! stream must be seekable
-function TDynMapDef.parseMap (st: TStream): TDynRecord;
+function TDynMapDef.parseMap (st: TStream; wasBinary: PBoolean=nil): TDynRecord;
 var
   sign: packed array[0..3] of AnsiChar;
   pr: TTextParser;
 begin
+  if (wasBinary <> nil) then wasBinary^ := false;
   st.position := 0;
   st.ReadBuffer(sign[0], 4);
   st.position := 0;
@@ -3323,6 +3434,7 @@ begin
   begin
     if (sign[3] = #1) then
     begin
+      if (wasBinary <> nil) then wasBinary^ := true;
       result := parseBinMap(st);
       exit;
     end;