[SabreTools, DatFiles/] Add flag for keeping empty games instead of making it default

RombaSharp/RombaSharp.Inits.cs

@@ -36,7 +36,7 @@ namespace RombaSharp
 		/// <param name="useGolangZip">True to use go zip implementation instead of zlib, false otherwise</param>
 		/// <param name="noDb">True to archive into depot but do not touch DB index and ignore only-needed flag, false otherwise</param>
 		/// TODO: Add ability to update .romba files with proper size AND use the correct depot if it fills up
-		/// TODO: Add ability to mark which depot the files are being rebuilt to in the DB
+		/// TODO: Add ability correctly to mark which depot the files are being rebuilt to in the DB
 		private static void InitArchive(
 			List<string> inputs,
 			bool onlyNeeded,

SabreTools.Library/DatFiles/ClrMamePro.cs

@@ -652,16 +652,19 @@ namespace SabreTools.Library.DatFiles
 			// If no items were found for this machine, add a Blank placeholder
 			if (!containsItems)
 			{
-				Blank blank = new Blank()
+				if (this.KeepEmptyGames)
 				{
-					SystemID = sysid,
-					System = filename,
-					SourceID = srcid,
-				};
-				blank.CopyMachineInformation(machine);
+					Blank blank = new Blank()
+					{
+						SystemID = sysid,
+						System = filename,
+						SourceID = srcid,
+					};
+					blank.CopyMachineInformation(machine);
 
-				// Now process and add the rom
-				ParseAddHelper(blank, clean, remUnicode);
+					// Now process and add the rom
+					ParseAddHelper(blank, clean, remUnicode);
+				}
 			}
 		}
 

SabreTools.Library/DatFiles/DatFile.cs

@@ -451,6 +451,27 @@ namespace SabreTools.Library.DatFiles
 				_datHeader.ExcludeOf = value;
 			}
 		}
+		public bool KeepEmptyGames
+		{
+			get
+			{
+				if (_datHeader == null)
+				{
+					_datHeader = new DatHeader();
+				}
+
+				return _datHeader.KeepEmptyGames;
+			}
+			set
+			{
+				if (_datHeader == null)
+				{
+					_datHeader = new DatHeader();
+				}
+
+				_datHeader.KeepEmptyGames = value;
+			}
+		}
 		public bool SceneDateStrip
 		{
 			get

SabreTools.Library/DatFiles/DatHeader.cs

@@ -32,6 +32,7 @@ namespace SabreTools.Library.DatFiles
 		private ForcePacking _forcePacking;
 		private DatFormat _datFormat;
 		private bool _excludeOf;
+		private bool _keepEmptyGames;
 		private bool _sceneDateStrip;
 		private DedupeType _dedupeRoms;
 		private Hash _stripHash;
@@ -147,6 +148,11 @@ namespace SabreTools.Library.DatFiles
 			get { return _excludeOf; }
 			set { _excludeOf = value; }
 		}
+		public bool KeepEmptyGames
+		{
+			get { return _keepEmptyGames; }
+			set { _keepEmptyGames = value; }
+		}
 		public bool SceneDateStrip
 		{
 			get { return _sceneDateStrip; }
@@ -243,6 +249,7 @@ namespace SabreTools.Library.DatFiles
 				_forcePacking = this._forcePacking,
 				_datFormat = this._datFormat,
 				_excludeOf = this._excludeOf,
+				_keepEmptyGames = this._keepEmptyGames,
 				_sceneDateStrip = this._sceneDateStrip,
 				_dedupeRoms = this._dedupeRoms,
 				_stripHash = this._stripHash,

SabreTools.Library/DatFiles/Listxml.cs

@@ -494,17 +494,19 @@ namespace SabreTools.Library.DatFiles
 			// If no items were found for this machine, add a Blank placeholder
 			if (!containsItems)
 			{
-				Blank blank = new Blank()
+				if (this.KeepEmptyGames)
 				{
-					SystemID = sysid,
-					System = filename,
-					SourceID = srcid,
-				};
+					Blank blank = new Blank()
+					{
+						SystemID = sysid,
+						System = filename,
+						SourceID = srcid,
+					};
+					blank.CopyMachineInformation(machine);
 
-				blank.CopyMachineInformation(machine);
-
-				// Now process and add the rom
-				key = ParseAddHelper(blank, clean, remUnicode);
+					// Now process and add the rom
+					ParseAddHelper(blank, clean, remUnicode);
+				}
 			}
 		}
 

SabreTools.Library/DatFiles/Logiqx.cs

@@ -534,17 +534,19 @@ namespace SabreTools.Library.DatFiles
 			// If no items were found for this machine, add a Blank placeholder
 			if (!containsItems)
 			{
-				Blank blank = new Blank()
+				if (this.KeepEmptyGames)
 				{
-					SystemID = sysid,
-					System = filename,
-					SourceID = srcid,
-				};
+					Blank blank = new Blank()
+					{
+						SystemID = sysid,
+						System = filename,
+						SourceID = srcid,
+					};
+					blank.CopyMachineInformation(machine);
 
-				blank.CopyMachineInformation(machine);
-
-				// Now process and add the rom
-				key = ParseAddHelper(blank, clean, remUnicode);
+					// Now process and add the rom
+					ParseAddHelper(blank, clean, remUnicode);
+				}
 			}
 		}
 

SabreTools.Library/DatFiles/SabreDat.cs

@@ -462,11 +462,18 @@ namespace SabreTools.Library.DatFiles
 								break;
 							default:
 								// By default, create a new Blank, just in case
-								datItem = new Blank();
+								if (this.KeepEmptyGames)
+								{
+									datItem = new Blank();
+								}
+								else
+								{
+									datItem = null;
+								}
 								break;
 						}
 
-						datItem.CopyMachineInformation(dir);
+						datItem?.CopyMachineInformation(dir);
 
 						// Now process and add the rom
 						key = ParseAddHelper(datItem, clean, remUnicode);

SabreTools.Library/DatFiles/SoftwareList.cs

@@ -231,17 +231,19 @@ namespace SabreTools.Library.DatFiles
 			// If no items were found for this machine, add a Blank placeholder
 			if (!containsItems)
 			{
-				Blank blank = new Blank()
+				if (this.KeepEmptyGames)
 				{
-					SystemID = sysid,
-					System = filename,
-					SourceID = srcid,
-				};
+					Blank blank = new Blank()
+					{
+						SystemID = sysid,
+						System = filename,
+						SourceID = srcid,
+					};
+					blank.CopyMachineInformation(machine);
 
-				blank.CopyMachineInformation(machine);
-
-				// Now process and add the rom
-				key = ParseAddHelper(blank, clean, remUnicode);
+					// Now process and add the rom
+					ParseAddHelper(blank, clean, remUnicode);
+				}
 			}
 		}
 

SabreTools.Library/README.1ST

@@ -287,6 +287,11 @@ Options:
 		  If this flag is enabled, then the romof, cloneof, and sampleof tags
 		  will be omitted from the outputted DAT.
 		  
+	  -keg, --keep-empty-games	  Keep originally empty sets from the input(s)
+		  Normally, any sets that are considered empty will not be included in
+		  the output, this flag allows these empty sets to be added to the
+		  output.
+
 	  -sds, --scene-date-strip	Remove date from scene-named sets
 		  If this flag is enabled, sets with "scene" names will have the date
 		  removed from the beginning. For example "01.01.01-Game_Name-GROUP"

SabreTools/SabreTools.Help.cs

@@ -465,6 +465,18 @@ namespace SabreTools
 					longDescription: "Instead of the normal behavior of rebuilding using a DAT, this flag allows the user to use the DAT as a filter instead. All files that are found in the DAT will be skipped and everything else will be output in the selected format.");
 			}
 		}
+		private static Feature keepEmptyGamesFlag
+		{
+			get
+			{
+				return new Feature(
+					"keep-empty-games",
+					new List<string>() { "-keg", "--keep-empty-games" },
+					"Keep originally empty sets from the input(s)",
+					FeatureType.Flag,
+					longDescription: "Normally, any sets that are considered empty will not be included in the output, this flag allows these empty sets to be added to the output.");
+			}
+		}
 		private static Feature levelFlag
 		{
 			get
@@ -2550,6 +2562,7 @@ The stats that are outputted are as follows:
 				update[outputTsvFlag].AddFeature(quotesFlag);
 			update.AddFeature(outputXmlFlag);
 				update[outputXmlFlag].AddFeature(depreciatedFlag);
+			// End OLD
 			update.AddFeature(filenameStringInput);
 			update.AddFeature(nameStringInput);
 			update.AddFeature(descriptionStringInput);
@@ -2568,6 +2581,7 @@ The stats that are outputted are as follows:
 			update.AddFeature(forcenodumpStringInput);
 			update.AddFeature(forcepackingStringInput);
 			update.AddFeature(excludeOfFlag);
+			update.AddFeature(keepEmptyGamesFlag);
 			update.AddFeature(sceneDateStripFlag);
 			update.AddFeature(cleanFlag);
 			update.AddFeature(removeUnicodeFlag);

SabreTools/SabreTools.cs

@@ -317,6 +317,9 @@ namespace SabreTools
 					case "inverse":
 						inverse = true;
 						break;
+					case "keep-empty-games":
+						datHeader.KeepEmptyGames = true;
+						break;
 					case "level":
 						splittingMode |= SplittingMode.Level;
 						break;