[ros-diffs] [hbelusca] 66174: [FREELDR]: Remove some extra newlines at the end of error messages that are displayed via message boxes, and also for few of them, use the fact that our message boxes implement pr...

Author: hbelusca
Date: Thu Feb  5 21:24:26 2015
New Revision: 66174

URL: http://svn.reactos.org/svn/reactos?rev=66174&view=rev
Log:
[FREELDR]: Remove some extra newlines at the end of error messages that are displayed via
message boxes, and also for few of them, use the fact that our message boxes implement
printf-like features.

Modified:
    trunk/reactos/boot/freeldr/freeldr/arch/amd64/winldr.c
    trunk/reactos/boot/freeldr/freeldr/arch/i386/drivemap.c
    trunk/reactos/boot/freeldr/freeldr/arch/i386/winldr.c
    trunk/reactos/boot/freeldr/freeldr/disk/ramdisk.c
    trunk/reactos/boot/freeldr/freeldr/freeldr.c
    trunk/reactos/boot/freeldr/freeldr/linuxboot.c
    trunk/reactos/boot/freeldr/freeldr/miscboot.c
    trunk/reactos/boot/freeldr/freeldr/windows/peloader.c
    trunk/reactos/boot/freeldr/freeldr/windows/winldr.c
    trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c
    trunk/reactos/boot/freeldr/freeldr/windows/wlregistry.c

Modified: trunk/reactos/boot/freeldr/freeldr/arch/amd64/winldr.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/arch/amd64/winldr.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/arch/amd64/winldr.c	[iso-8859-1] Thu Feb  5
21:24:26 2015
@@ -384,7 +384,7 @@
     PcrBasePage = Pcr >> MM_PAGE_SHIFT;
     if (Pcr == 0)
     {
-        UiMessageBox("Can't allocate PCR\n");
+        UiMessageBox("Can't allocate PCR.");
         return;
     }
     RtlZeroMemory((PVOID)Pcr, 2 * MM_PAGE_SIZE);
@@ -400,7 +400,7 @@
     GdtIdt = (PKGDTENTRY)MmAllocateMemoryWithType(NumPages * MM_PAGE_SIZE,
LoaderMemoryData);
     if (GdtIdt == NULL)
     {
-        UiMessageBox("Can't allocate pages for GDT+IDT!\n");
+        UiMessageBox("Can't allocate pages for GDT+IDT!");
         return;
     }
 

Modified: trunk/reactos/boot/freeldr/freeldr/arch/i386/drivemap.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/arch/i386/drivemap.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/arch/i386/drivemap.c	[iso-8859-1] Thu Feb  5
21:24:26 2015
@@ -30,16 +30,15 @@
 #ifndef _MSC_VER
 VOID DriveMapMapDrivesInSection(PCSTR SectionName)
 {
-    CHAR            SettingName[80];
-    CHAR            SettingValue[80];
-    CHAR            ErrorText[260];
-    CHAR            Drive1[80];
-    CHAR            Drive2[80];
-    ULONG                SectionId;
-    ULONG                SectionItemCount;
-    ULONG                Index;
-    ULONG                Index2;
-    DRIVE_MAP_LIST    DriveMapList;
+    CHAR           SettingName[80];
+    CHAR           SettingValue[80];
+    CHAR           Drive1[80];
+    CHAR           Drive2[80];
+    ULONG          SectionId;
+    ULONG          SectionItemCount;
+    ULONG          Index;
+    ULONG          Index2;
+    DRIVE_MAP_LIST DriveMapList;
 
     RtlZeroMemory(&DriveMapList, sizeof(DRIVE_MAP_LIST));
 
@@ -62,8 +61,7 @@
                 // Make sure we haven't exceeded the drive map max count
                 if (DriveMapList.DriveMapCount >= 4)
                 {
-                    sprintf(ErrorText, "Max DriveMap count exceeded in section
[%s]:\n\n%s=%s", SectionName, SettingName, SettingValue);
-                    UiMessageBox(ErrorText);
+                    UiMessageBox("Max DriveMap count exceeded in section
[%s]:\n\n%s=%s", SectionName, SettingName, SettingValue);
                     continue;
                 }
 
@@ -88,8 +86,7 @@
                 // Make sure we got good values before we add them to the map
                 if (!DriveMapIsValidDriveString(Drive1) ||
!DriveMapIsValidDriveString(Drive2))
                 {
-                    sprintf(ErrorText, "Error in DriveMap setting in section
[%s]:\n\n%s=%s", SectionName, SettingName, SettingValue);
-                    UiMessageBox(ErrorText);
+                    UiMessageBox("Error in DriveMap setting in section
[%s]:\n\n%s=%s", SectionName, SettingName, SettingValue);
                     continue;
                 }
 

Modified: trunk/reactos/boot/freeldr/freeldr/arch/i386/winldr.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/arch/i386/winldr.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/arch/i386/winldr.c	[iso-8859-1] Thu Feb  5 21:24:26
2015
@@ -365,7 +365,7 @@
 
     if (Pcr == 0)
     {
-        UiMessageBox("Can't allocate PCR\n");
+        UiMessageBox("Can't allocate PCR.");
         return;
     }
 
@@ -384,7 +384,7 @@
 
     if (GdtIdt == NULL)
     {
-        UiMessageBox("Can't allocate pages for GDT+IDT!\n");
+        UiMessageBox("Can't allocate pages for GDT+IDT!");
         return;
     }
 

Modified: trunk/reactos/boot/freeldr/freeldr/disk/ramdisk.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/disk/…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/disk/ramdisk.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/disk/ramdisk.c	[iso-8859-1] Thu Feb  5 21:24:26
2015
@@ -155,7 +155,7 @@
     //
     if (Information.EndingAddress.HighPart != 0)
     {
-        UiMessageBox("RAM disk too big\n");
+        UiMessageBox("RAM disk too big.");
         FsCloseFile(RamFile);
         return FALSE;
     }
@@ -172,7 +172,7 @@
     gRamDiskBase = MmAllocateMemoryWithType(gRamDiskSize, LoaderXIPRom);
     if (!gRamDiskBase)
     {
-        UiMessageBox("Failed to allocate memory for RAM disk\n");
+        UiMessageBox("Failed to allocate memory for RAM disk.");
         FsCloseFile(RamFile);
         return FALSE;
     }
@@ -222,7 +222,7 @@
             gRamDiskBase = NULL;
             gRamDiskSize = 0;
             FsCloseFile(RamFile);
-            UiMessageBox("Failed to read RAM disk\n");
+            UiMessageBox("Failed to read RAM disk.");
             return FALSE;
         }
     }

Modified: trunk/reactos/boot/freeldr/freeldr/freeldr.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/freel…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/freeldr.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/freeldr.c	[iso-8859-1] Thu Feb  5 21:24:26 2015
@@ -41,13 +41,13 @@
 
     if (!UiInitialize(FALSE))
     {
-        UiMessageBoxCritical("Unable to initialize UI.\n");
+        UiMessageBoxCritical("Unable to initialize UI.");
         goto Quit;
     }
 
     if (!MmInitializeMemoryManager())
     {
-        UiMessageBoxCritical("Unable to initialize memory manager");
+        UiMessageBoxCritical("Unable to initialize memory manager.");
         goto Quit;
     }
 

Modified: trunk/reactos/boot/freeldr/freeldr/linuxboot.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/linux…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/linuxboot.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/linuxboot.c	[iso-8859-1] Thu Feb  5 21:24:26 2015
@@ -82,7 +82,6 @@
     PCSTR Description = OperatingSystem->LoadIdentifier;
     PFILE LinuxKernel = 0;
     PFILE LinuxInitrdFile = 0;
-    CHAR  TempString[260];
 
     UiDrawBackdrop();
 
@@ -102,8 +101,7 @@
     LinuxKernel = FsOpenFile(LinuxKernelName);
     if (!LinuxKernel)
     {
-        sprintf(TempString, "Linux kernel \'%s\' not found.",
LinuxKernelName);
-        UiMessageBox(TempString);
+        UiMessageBox("Linux kernel \'%s\' not found.",
LinuxKernelName);
         goto LinuxBootFailed;
     }
 
@@ -113,8 +111,7 @@
         LinuxInitrdFile = FsOpenFile(LinuxInitrdName);
         if (!LinuxInitrdFile)
         {
-            sprintf(TempString, "Linux initrd image \'%s\' not found.",
LinuxInitrdName);
-            UiMessageBox(TempString);
+            UiMessageBox("Linux initrd image \'%s\' not found.",
LinuxInitrdName);
             goto LinuxBootFailed;
         }
     }
@@ -212,7 +209,6 @@
 BOOLEAN LinuxParseIniSection(PCSTR SectionName)
 {
     ULONG_PTR SectionId;
-    CHAR SettingName[260];
 
     /* Find all the message box settings and run them */
     UiShowMessageBoxesInSection(SectionName);
@@ -220,8 +216,7 @@
     /* Try to open the operating system section in the .ini file */
     if (!IniOpenSection(SectionName, &SectionId))
     {
-        sprintf(SettingName, "Section [%s] not found in freeldr.ini.\n",
SectionName);
-        UiMessageBox(SettingName);
+        UiMessageBox("Section [%s] not found in freeldr.ini.", SectionName);
         return FALSE;
     }
 

Modified: trunk/reactos/boot/freeldr/freeldr/miscboot.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/miscb…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/miscboot.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/miscboot.c	[iso-8859-1] Thu Feb  5 21:24:26 2015
@@ -34,7 +34,6 @@
     CHAR  FileName[260];
     PFILE FilePointer;
     ULONG BytesRead;
-    CHAR  SettingName[80];
 
     /* Find all the message box settings and run them */
     UiShowMessageBoxesInSection(SectionName);
@@ -42,8 +41,7 @@
     /* Try to open the operating system section in the .ini file */
     if (!IniOpenSection(SectionName, &SectionId))
     {
-        sprintf(SettingName, "Section [%s] not found in freeldr.ini.\n",
SectionName);
-        UiMessageBox(SettingName);
+        UiMessageBox("Section [%s] not found in freeldr.ini.", SectionName);
         return;
     }
 
@@ -56,8 +54,7 @@
     FilePointer = FsOpenFile(FileName);
     if (!FilePointer)
     {
-        strcat(FileName, " not found.");
-        UiMessageBox(FileName);
+        UiMessageBox("%s not found.", FileName);
         return;
     }
 
@@ -97,7 +94,6 @@
 {
     ULONG_PTR SectionId;
     PCSTR SectionName = OperatingSystem->SystemPartition;
-    CHAR  SettingName[80];
     CHAR  SettingValue[80];
     PARTITION_TABLE_ENTRY PartitionTableEntry;
     UCHAR DriveNumber;
@@ -109,8 +105,7 @@
     /* Try to open the operating system section in the .ini file */
     if (!IniOpenSection(SectionName, &SectionId))
     {
-        sprintf(SettingName, "Section [%s] not found in freeldr.ini.\n",
SectionName);
-        UiMessageBox(SettingName);
+        UiMessageBox("Section [%s] not found in freeldr.ini.", SectionName);
         return;
     }
 
@@ -175,7 +170,6 @@
 {
     ULONG_PTR SectionId;
     PCSTR SectionName = OperatingSystem->SystemPartition;
-    CHAR  SettingName[80];
     CHAR  SettingValue[80];
     UCHAR DriveNumber;
 
@@ -185,8 +179,7 @@
     /* Try to open the operating system section in the .ini file */
     if (!IniOpenSection(SectionName, &SectionId))
     {
-        sprintf(SettingName, "Section [%s] not found in freeldr.ini.\n",
SectionName);
-        UiMessageBox(SettingName);
+        UiMessageBox("Section [%s] not found in freeldr.ini.", SectionName);
         return;
     }
 

Modified: trunk/reactos/boot/freeldr/freeldr/windows/peloader.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/windows/peloader.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/windows/peloader.c	[iso-8859-1] Thu Feb  5 21:24:26
2015
@@ -282,7 +282,7 @@
     Status = ArcOpen(FileName, OpenReadOnly, &FileId);
     if (Status != ESUCCESS)
     {
-        //UiMessageBox("Can not open the file");
+        // UiMessageBox("Can not open the file.");
         return FALSE;
     }
 
@@ -290,7 +290,7 @@
     Status = ArcRead(FileId, HeadersBuffer, SECTOR_SIZE * 2, &BytesRead);
     if (Status != ESUCCESS)
     {
-        UiMessageBox("Error reading from file");
+        UiMessageBox("Error reading from file.");
         ArcClose(FileId);
         return FALSE;
     }
@@ -299,8 +299,8 @@
     NtHeaders = RtlImageNtHeader(HeadersBuffer);
     if (!NtHeaders)
     {
-        //Print(L"Error - no NT header found in %s\n", FileName);
-        UiMessageBox("Error - no NT header found");
+        // Print(L"Error - no NT header found in %s\n", FileName);
+        UiMessageBox("Error - no NT header found.");
         ArcClose(FileId);
         return FALSE;
     }
@@ -308,8 +308,8 @@
     /* Ensure this is executable image */
     if (((NtHeaders->FileHeader.Characteristics & IMAGE_FILE_EXECUTABLE_IMAGE) ==
0))
     {
-        //Print(L"Not an executable image %s\n", FileName);
-        UiMessageBox("Not an executable image");
+        // Print(L"Not an executable image %s\n", FileName);
+        UiMessageBox("Not an executable image.");
         ArcClose(FileId);
         return FALSE;
     }
@@ -330,8 +330,8 @@
 
         if (PhysicalBase == NULL)
         {
-            //Print(L"Failed to alloc pages for image %s\n", FileName);
-            UiMessageBox("Failed to alloc pages for image");
+            // Print(L"Failed to alloc pages for image %s\n", FileName);
+            UiMessageBox("Failed to alloc pages for image.");
             ArcClose(FileId);
             return FALSE;
         }
@@ -347,7 +347,7 @@
     Status = ArcSeek(FileId, &Position, SeekAbsolute);
     if (Status != ESUCCESS)
     {
-        UiMessageBox("Error seeking to start of file");
+        UiMessageBox("Error seeking to start of file.");
         ArcClose(FileId);
         return FALSE;
     }
@@ -355,8 +355,8 @@
     Status = ArcRead(FileId, PhysicalBase, NtHeaders->OptionalHeader.SizeOfHeaders,
&BytesRead);
     if (Status != ESUCCESS)
     {
-        //Print(L"Error reading headers %s\n", FileName);
-        UiMessageBox("Error reading headers");
+        // Print(L"Error reading headers %s\n", FileName);
+        UiMessageBox("Error reading headers.");
         ArcClose(FileId);
         return FALSE;
     }

Modified: trunk/reactos/boot/freeldr/freeldr/windows/winldr.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/windows/winldr.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/windows/winldr.c	[iso-8859-1] Thu Feb  5 21:24:26
2015
@@ -682,7 +682,7 @@
         /* Load the ramdisk */
         if (!RamDiskLoadVirtualFile(FileName))
         {
-            UiMessageBox("Failed to load RAM disk file %s\n", FileName);
+            UiMessageBox("Failed to load RAM disk file %s", FileName);
             return;
         }
     }

Modified: trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c	[iso-8859-1] Thu Feb  5 21:24:26
2015
@@ -220,7 +220,7 @@
     MemoryMap = MmGetMemoryMap(&NoEntries);
     if (MemoryMap == NULL)
     {
-        UiMessageBox("Can not retrieve the current memory map");
+        UiMessageBox("Can not retrieve the current memory map.");
         return FALSE;
     }
 
@@ -334,7 +334,7 @@
     /*Status = MempSetupPaging(OsLoaderBase >> MM_PAGE_SHIFT, OsLoaderSize >>
MM_PAGE_SHIFT);
     if (!Status)
     {
-        UiMessageBox("Error during MempSetupPaging");
+        UiMessageBox("Error during MempSetupPaging.");
         return;
     }*/
 

Modified: trunk/reactos/boot/freeldr/freeldr/windows/wlregistry.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/windows/wlregistry.c	[iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/windows/wlregistry.c	[iso-8859-1] Thu Feb  5
21:24:26 2015
@@ -437,8 +437,7 @@
     return TRUE;
 
 Failure:
-    //UiMessageBox("Error reading NLS file %s\n", Filename);
-    UiMessageBox("Error reading NLS file!");
+    UiMessageBox("Error reading NLS file %s", FileName);
     return FALSE;
 }