#define IDB_CLOSE_12_12                 197
#define IDB_CLOSE_8_8                   197
...
#define IDB_PNG3                        201
#define IDB_CLOSE_16_16                 201

void CAdvGeneral::OnBnClickedBtCompactAndRepair()
{
	auto msg = theApp.m_Language.GetString("CompactRepairWarning", "Warning this can take quite a long time and require up to double the hard drive space as your current database size, Continue?");
	int ret = MessageBox(msg, _T("Ditto"), MB_YESNO);

	if (ret == IDYES)
	{
		CWaitCursor wait;

		try
		{
			// The RemoveOldEntries loop seems excessive and might not be what's intended for a simple compact.
			// Usually, VACUUM is enough after deletions. If RemoveOldEntries is part of a larger cleanup,
			// it might be okay, but ensure it's efficient.
			// For now, let's assume it's intentional.
			try
			{
				for (int i = 0; i < 100; i++)
				{
					// Consider if this query and loop are truly necessary before VACUUM.
					// VACUUM itself reclaims space from deleted rows.
					int toDeleteCount = theApp.m_db.execScalar(_T("SELECT COUNT(clipID) FROM MainDeletes"));
					if (toDeleteCount <= 0)
						break;

					RemoveOldEntries(false);
				}
			}
			CATCH_SQLITE_EXCEPTION

			theApp.m_db.execDML(_T("PRAGMA auto_vacuum = FULL")); // FULL is more thorough than INCREMENTAL (1)
			theApp.m_db.execDML(_T("VACUUM")); // execDML for commands that don't return data
			MessageBox(_T("Compact and Repair completed."), _T("Ditto"), MB_OK | MB_ICONINFORMATION);
		}
		CATCH_SQLITE_EXCEPTION
	}
}