From e3cf8aae28ef3d8672f2153972f449f29bc37922 Mon Sep 17 00:00:00 2001
From: zeromus <zeromus@zeromus.org>
Date: Wed, 2 Mar 2011 03:41:24 +0000
Subject: [PATCH] [NES] some comments

---
 .../Consoles/Nintendo/NES/Boards/SxROM.cs     | 23 +++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/BizHawk.Emulation/Consoles/Nintendo/NES/Boards/SxROM.cs b/BizHawk.Emulation/Consoles/Nintendo/NES/Boards/SxROM.cs
index a0f587bc3d..561763d0ca 100644
--- a/BizHawk.Emulation/Consoles/Nintendo/NES/Boards/SxROM.cs
+++ b/BizHawk.Emulation/Consoles/Nintendo/NES/Boards/SxROM.cs
@@ -13,8 +13,14 @@ namespace BizHawk.Emulation.Consoles.Nintendo.Boards
 	//check UNIF for more information.. it may specify board and MMC1 rev independently because boards may have any MMC1 rev
 	//in that case, we need to capture MMC1 rev in the game database (maybe add a new `chip` parameter)
 
-	//this board is a little more convoluted than it might otherwise be because i switched to a more chip-centered way of modeling it partway through
-	//perhaps i will make other boards work that way, and perhaps not
+	//Final Fantasy
+	//Mega Man 2
+	//Blaster Master
+	//Metroid
+	//Kid Icarus
+	//Zelda
+	//Zelda 2
+	//Castlevania 2
 
 	class MMC1
 	{
@@ -196,19 +202,26 @@ namespace BizHawk.Emulation.Consoles.Nintendo.Boards
 			{
 				case "SGROM":
 					Debug.Assert(RomInfo.CHR_Size == -1, "don't specify in gamedb, it is redundant");
-					romInfo.CHR_Size = 0; RomInfo.CRAM_Size = 8; romInfo.PRAM_Size = 0;
+					romInfo.CHR_Size = 0; 
+					RomInfo.CRAM_Size = 8; 
+					romInfo.PRAM_Size = 0;
 					break;
 				case "SNROM":
 					Debug.Assert(RomInfo.CHR_Size == -1, "don't specify in gamedb, it is redundant");
-					romInfo.CHR_Size = 0; RomInfo.CRAM_Size = 8; RomInfo.PRAM_Size = 8;
+					romInfo.CHR_Size = 0; 
+					RomInfo.CRAM_Size = 8; 
+					RomInfo.PRAM_Size = 8;
 					break;
 				case "SL2ROM":
+					//these boards have CHR roms and nothing else interesting
 					RomInfo.CRAM_Size = 0; 
 					RomInfo.PRAM_Size = 0;
 					break;
+
 				default: throw new InvalidOperationException();
 			}
 
+			//boards that don't contain CHR rom will contain CRAM. only one size is supported; set it up if it is there.
 			Debug.Assert(RomInfo.CRAM_Size == 0 || RomInfo.CRAM_Size == 8);
 			if (RomInfo.CRAM_Size != 0)
 			{
@@ -217,6 +230,7 @@ namespace BizHawk.Emulation.Consoles.Nintendo.Boards
 			}
 			else cram = new byte[0];
 
+			//some boards contain PRAM. we only understand one size right now. set it up if it is there.
 			Debug.Assert(RomInfo.PRAM_Size == 0 || RomInfo.PRAM_Size == 8);
 			if (RomInfo.PRAM_Size != 0)
 			{
@@ -225,6 +239,7 @@ namespace BizHawk.Emulation.Consoles.Nintendo.Boards
 			}
 			else pram = new byte[0];
 
+			//some boards contain CHR roms, so set that up here.
 			if (RomInfo.CHR_Size != 0)
 			{
 				Debug.Assert(RomInfo.CHR_Size == 16);