From c3856eddb5a8146751dac82ad1ba4433f5ca6d6d Mon Sep 17 00:00:00 2001 From: Amlal El Mahrouss Date: Fri, 11 Apr 2025 17:51:19 +0200 Subject: tweak kernel, boot: use -Wall, -Werror, -Wpedantic now. Signed-off-by: Amlal El Mahrouss --- dev/boot/src/BootSupport.cc | 3 --- dev/boot/src/BootTextWriter.cc | 2 +- dev/boot/src/BootThread.cc | 5 +++-- dev/boot/src/HEL/AMD64/BootATA.cc | 9 +++++---- dev/boot/src/HEL/AMD64/BootEFI.cc | 4 ---- dev/boot/src/New+Delete.cc | 15 +++++++++++++++ 6 files changed, 24 insertions(+), 14 deletions(-) (limited to 'dev/boot/src') diff --git a/dev/boot/src/BootSupport.cc b/dev/boot/src/BootSupport.cc index f8a527d9..ade8ed01 100644 --- a/dev/boot/src/BootSupport.cc +++ b/dev/boot/src/BootSupport.cc @@ -58,9 +58,6 @@ EXTERN_C size_t strlen(const char* whatToCheck) /// @brief strcmp definition in C++. EXTERN_C int strcmp(const char* whatToCheck, const char* whatToCheckRight) { - if (!whatToCheck || *whatToCheck == 0) - return 0; - SizeT len = 0; while (whatToCheck[len] == whatToCheckRight[len]) diff --git a/dev/boot/src/BootTextWriter.cc b/dev/boot/src/BootTextWriter.cc index 0b132f83..95e248a2 100644 --- a/dev/boot/src/BootTextWriter.cc +++ b/dev/boot/src/BootTextWriter.cc @@ -157,7 +157,7 @@ Boot::BootTextWriter& Boot::BootTextWriter::_Write(const UInt64& x) return *this; } - if (y < 0) + if (y == ~0UL) y = -y; const char cNumbers[] = "0123456789ABCDEF"; diff --git a/dev/boot/src/BootThread.cc b/dev/boot/src/BootThread.cc index ee70812f..ce691f96 100644 --- a/dev/boot/src/BootThread.cc +++ b/dev/boot/src/BootThread.cc @@ -25,7 +25,7 @@ namespace Boot EXTERN_C Int32 rt_jump_to_address(VoidPtr code, HEL::BootInfoHeader* handover, UInt8* stack); BootThread::BootThread(VoidPtr blob) - : fBlob(blob), fStartAddress(nullptr) + : fStartAddress(nullptr), fBlob(blob) { // detect the format. const Char* blob_bytes = reinterpret_cast(fBlob); @@ -84,7 +84,6 @@ namespace Boot constexpr auto sectionForCode = ".text"; constexpr auto sectionForBootZ = ".ldr"; - constexpr auto sectionForBSS = ".bss"; for (SizeT sectIndex = 0; sectIndex < numSecs; ++sectIndex) { @@ -168,6 +167,8 @@ namespace Boot Int32 BootThread::Start(HEL::BootInfoHeader* handover, Bool own_stack) { HEL::HandoverProc err_fn = [](HEL::BootInfoHeader* rcx) -> Int32 { + NE_UNUSED(rcx); + fb_render_string("BootZ: Invalid Boot Image...", 50, 10, RGB(0xFF, 0xFF, 0xFF)); ::Boot::Stop(); diff --git a/dev/boot/src/HEL/AMD64/BootATA.cc b/dev/boot/src/HEL/AMD64/BootATA.cc index a172e4cb..8b1b5a0a 100644 --- a/dev/boot/src/HEL/AMD64/BootATA.cc +++ b/dev/boot/src/HEL/AMD64/BootATA.cc @@ -23,10 +23,9 @@ using namespace Boot; -#define kATADataLen 256 +#define kATADataLen (256) static Boolean kATADetected = false; -static Int32 kATADeviceType = kATADeviceCount; static UInt16 kATAData[kATADataLen] = {0}; Boolean boot_ata_detected(Void); @@ -64,6 +63,8 @@ Void boot_ata_select(UInt16 Bus) Boolean boot_ata_init(UInt16 Bus, UInt8 Drive, UInt16& OutBus, UInt8& OutMaster) { + NE_UNUSED(Drive); + if (boot_ata_detected()) return true; @@ -214,7 +215,7 @@ BootDeviceATA::operator bool() @param Sz Sector size @param Buf buffer */ -BootDeviceATA& BootDeviceATA::Read(CharacterTypeUTF8* Buf, const SizeT& SectorSz) +BootDeviceATA& BootDeviceATA::Read(CharacterTypeUTF8* Buf, SizeT SectorSz) { if (!boot_ata_detected()) { @@ -238,7 +239,7 @@ BootDeviceATA& BootDeviceATA::Read(CharacterTypeUTF8* Buf, const SizeT& SectorSz @param Sz Sector size @param Buf buffer */ -BootDeviceATA& BootDeviceATA::Write(CharacterTypeUTF8* Buf, const SizeT& SectorSz) +BootDeviceATA& BootDeviceATA::Write(CharacterTypeUTF8* Buf, SizeT SectorSz) { if (!boot_ata_detected()) { diff --git a/dev/boot/src/HEL/AMD64/BootEFI.cc b/dev/boot/src/HEL/AMD64/BootEFI.cc index 4d0e9a59..4220c6cb 100644 --- a/dev/boot/src/HEL/AMD64/BootEFI.cc +++ b/dev/boot/src/HEL/AMD64/BootEFI.cc @@ -86,10 +86,6 @@ EFI_EXTERN_C EFI_API Int32 ModuleMain(EfiHandlePtr image_handle, new HEL::BootInfoHeader(); UInt32 map_key = 0; - UInt32 size_struct_ptr = 0; - EfiMemoryDescriptor* struct_ptr = nullptr; - UInt32 sz_desc = 0; - UInt32 rev_desc = 0; #ifdef ZBA_USE_FB if (!boot_init_fb()) diff --git a/dev/boot/src/New+Delete.cc b/dev/boot/src/New+Delete.cc index 3e3cf48f..61698510 100644 --- a/dev/boot/src/New+Delete.cc +++ b/dev/boot/src/New+Delete.cc @@ -63,6 +63,21 @@ void operator delete(void* buf, size_t size) if (!buf) return; + NE_UNUSED(size); + + BS->FreePool(buf); +} + +/// @brief Deletes the object (array specific). +/// @param buf the object. +/// @param size it's size. +void operator delete[](void* buf, size_t size) +{ + if (!buf) + return; + + NE_UNUSED(size); + BS->FreePool(buf); } -- cgit v1.2.3