Skip to content

Commit 9f94a91

Browse files
committed
clean debug messages
Signed-off-by: Slice <sergey.slice@gmail.com>
1 parent c1cfdd7 commit 9f94a91

5 files changed

Lines changed: 18 additions & 21 deletions

File tree

rEFIt_UEFI/Settings/ConfigManager.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -599,8 +599,7 @@ EFI_STATUS ConfigManager::LoadSMBIOSPlist(const XStringW& ConfName)
599599
void ConfigManager::ReloadSmbios(XStringW& str) // searches for SMBIOS by system name
600600
{
601601
size_t N = SmbiosList.size();
602-
DBG("search SMBIOS for %ls\n", str.wc_str());
603-
DBG("at Reload OldChosenSmbios =%lld list size=%zu\n", OldChosenSmbios, N);
602+
//DBG("search SMBIOS for %ls\n", str.wc_str());
604603
if (OldChosenSmbios == 0) { // this is auto fill by OSName
605604
for (size_t i=1; i<N; i++) {
606605
if (SmbiosList[i].contains(str)) {

rEFIt_UEFI/entry_scan/loader.cpp

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1290,10 +1290,10 @@ STATIC LOADER_ENTRY *CreateLoaderEntry(
12901290
Entry->LoaderType = OSTYPE_OTHER;
12911291
break;
12921292
}
1293-
DBG("OSIconName=%ls \n", OSIconName.wc_str());
1293+
// DBG("OSIconName=%ls \n", OSIconName.wc_str());
12941294
Entry->OSName = OSIconName.subString(0, OSIconName.indexOf(',')); // TODO
12951295
// SmbiosList.AddReference(OSName.forgetDataWithoutFreeing(), true);
1296-
DBG("OSName=%ls\n", Entry->OSName.wc_str());
1296+
12971297
Entry->Title = FullTitle;
12981298
if (Entry->Title.isEmpty() && Volume->VolLabel.notEmpty()) {
12991299
if (Volume->VolLabel[0] == L'#') {
@@ -1337,7 +1337,7 @@ DBG("OSName=%ls\n", Entry->OSName.wc_str());
13371337
: BasenameXW.wc_str());
13381338
}
13391339
}
1340-
DBG("check Entry->Title \n");
1340+
// DBG("check Entry->Title \n");
13411341
if (Entry->Title.isEmpty()) {
13421342
// DBG("encounter LoaderTitle ==%ls and Entry->VolName ==%ls\n",
13431343
// LoaderTitle.wc_str(), Entry->VolName);
@@ -1503,7 +1503,7 @@ void LOADER_ENTRY::AddDefaultMenu() {
15031503
SubEntry->Title.SWPrintf("Boot %s with selected options", macOS);
15041504
SubScreen->AddMenuEntry(SubEntry, true);
15051505
SubScreen->AddMenuEntry(SubMenuKextInjectMgmt(), true);
1506-
SubScreen->AddMenuInfo_f("=== boot-args ===");DBG("ADM:10\n");
1506+
SubScreen->AddMenuInfo_f("=== boot-args ===");
15071507
if (!KernelIs64BitOnly) {
15081508
if (macOSVersion.notEmpty() && macOSVersion < MacOsVersion("10.8"_XS8)) {
15091509
SubScreen->AddMenuCheck("Mac OS X 32bit", OPT_I386, 68);
@@ -2545,9 +2545,10 @@ void ScanLoader(void)
25452545
DBG("Entries list before ordering\n");
25462546
for (size_t idx = 0; idx < MainMenu.Entries.sizeIncludingHidden(); idx++) {
25472547
if (MainMenu.Entries.ElementAt(idx).getLOADER_ENTRY()) {
2548-
DBG(" Entry %zd : %ls%s\n", idx,
2548+
DBG(" Entry %zd : %ls%s \n", idx,
25492549
MainMenu.Entries.ElementAt(idx).Title.wc_str(),
2550-
MainMenu.Entries.ElementAt(idx).Hidden ? " (hidden)" : "");
2550+
MainMenu.Entries.ElementAt(idx).Hidden ? " (hidden)" : ""
2551+
);
25512552
} else {
25522553
DBG(" Entry %zd : %ls%s\n", idx,
25532554
MainMenu.Entries.ElementAt(idx).Title.wc_str(),

rEFIt_UEFI/gui/menu_items/menu_items.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,11 +92,11 @@ LOADER_ENTRY *LOADER_ENTRY::getPartiallyDuplicatedEntry() const {
9292
DuplicateEntry->BuildVersion = BuildVersion;
9393
DuplicateEntry->Title = Title;
9494
DuplicateEntry->OSName = OSName;
95-
//DBG("gPDE:13\n");
95+
9696
// CopyKernelAndKextPatches(&DuplicateEntry->KernelAndKextPatches,
9797
// &KernelAndKextPatches);
9898
//DuplicateEntry->KernelAndKextPatches = KernelAndKextPatches;
9999
//DuplicateEntry->KernelAndKextPatches.takeValueFrom(KernelAndKextPatches);
100-
//DBG("gPDE:14\n");
100+
101101
return DuplicateEntry;
102102
}

rEFIt_UEFI/gui/menu_items/menu_items.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ class SIDELOAD_KEXT;
104104
class REFIT_ABSTRACT_MENU_ENTRY {
105105
public:
106106
XStringW Title = XStringW();
107+
XStringW OSName = XStringW();
107108
XBool Hidden = false;
108109
UINTN Row = 0;
109110
wchar_t ShortcutDigit = 0;
@@ -312,7 +313,7 @@ class REFIT_MENU_ITEM_ABSTRACT_ENTRY_LOADER : public REFIT_ABSTRACT_MENU_ENTRY {
312313
XStringW DevicePathString;
313314
XString8Array LoadOptions; // moved here for compatibility with legacy
314315
XStringW LoaderPath;
315-
XStringW OSName;
316+
//XStringW OSName; // at super
316317
XIcon DriveImage;
317318
XIcon BadgeImage;
318319

rEFIt_UEFI/refit/main.cpp

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1009,15 +1009,15 @@ void LOADER_ENTRY::DelegateKernelPatches() {
10091009
}
10101010
}
10111011

1012-
void LOADER_ENTRY::StartLoader() {
1012+
void LOADER_ENTRY::StartLoader()
1013+
{
10131014
EFI_STATUS Status;
10141015
EFI_TEXT_STRING ConOutOutputString = 0;
10151016
EFI_HANDLE ImageHandle = NULL;
10161017
EFI_LOADED_IMAGE *LoadedImage = NULL;
10171018
CONST CHAR8 *InstallerVersion;
10181019

10191020
DbgHeader("StartLoader");
1020-
10211021
DBG("Starting %ls\n", FileDevicePathToXStringW(DevicePath).wc_str());
10221022
#ifdef JIEF_DEBUG
10231023
displayFreeMemory("LOADER_ENTRY::StartLoader()"_XS8);
@@ -1077,7 +1077,6 @@ void LOADER_ENTRY::StartLoader() {
10771077
#ifdef JIEF_DEBUG
10781078
displayFreeMemory("LOADER_ENTRY::StartLoader() atfer ThemeX deleted"_XS8);
10791079
#endif
1080-
10811080
if (OSTYPE_IS_OSX(LoaderType) || OSTYPE_IS_OSX_RECOVERY(LoaderType) ||
10821081
OSTYPE_IS_OSX_INSTALLER(LoaderType)) {
10831082

@@ -1364,7 +1363,6 @@ void LOADER_ENTRY::StartLoader() {
13641363
gSettings.Quirks.OcKernelQuirks.XhciPortLimit;
13651364
mOpenCoreConfiguration.Kernel.Quirks.ProvideCurrentCpuInfo =
13661365
gSettings.Quirks.OcKernelQuirks.ProvideCurrentCpuInfo;
1367-
13681366
// Experimental: Auto-detect and apply quirks for modern CPUs
13691367
#ifdef ENABLE_MODERN_CPU_QUIRKS
13701368
if (gSettings.Quirks.OcKernelQuirks.AutoModernCPUQuirks) {
@@ -1981,7 +1979,6 @@ void LOADER_ENTRY::StartLoader() {
19811979
}
19821980
BuildVersion.setEmpty();
19831981
}
1984-
19851982
if (BuildVersion.notEmpty()) {
19861983
DBG(" %s (%s)\n", macOSVersion.asString().c_str(), BuildVersion.c_str());
19871984
} else {
@@ -2002,10 +1999,7 @@ void LOADER_ENTRY::StartLoader() {
20021999
!BooterPatch((UINT8 *)LoadedImage->ImageBase, LoadedImage->ImageSize)) {
20032000
DBG("Will not patch boot.efi\n");
20042001
}
2005-
XStringW OSName_real = this->OSName;
2006-
//DBG("OS Version=%s\n", macOSVersion.asString().c_str());
2007-
DBG("OSName=%ls\n", OSName_real.wc_str());
2008-
gConf.ReloadSmbios(OSName_real);
2002+
gConf.ReloadSmbios(OSName);
20092003
DelegateKernelPatches();
20102004

20112005
// Set boot argument for kernel if no caches, this should force kernel
@@ -2937,7 +2931,7 @@ static void LoadDrivers(void) {
29372931
gDriversFlags.VideoLoaded = true;
29382932
DriversToConnectNum++;
29392933
}
2940-
DBG("UninitRefitLib Num=%llu\n", DriversToConnectNum);
2934+
//DBG("UninitRefitLib Num=%llu\n", DriversToConnectNum);
29412935
UninitRefitLib();
29422936
if (DriversToConnectNum > 0) {
29432937
DBG("%llu drivers needs connecting ...\n", DriversToConnectNum);
@@ -4012,6 +4006,8 @@ DBG("strlen '<' =%ld or %lld\n", L_STR_LEN("<"), AsciiStrLen("<"));
40124006
HelpRefit();
40134007
break;
40144008
*/
4009+
// LOADER_ENTRY* CE = ChosenEntry->getLOADER_ENTRY();
4010+
40154011
if (ChosenEntry->getLOADER_ENTRY()) { // Boot OS via .EFI loader
40164012
SetBootCurrent(ChosenEntry->getLOADER_ENTRY());
40174013
ChosenEntry->StartLoader();

0 commit comments

Comments
 (0)