From 80f64d151d3320820bcfedec10d3e6f74ab37d91 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 28 Dec 2025 22:56:58 +0000 Subject: [PATCH] Fix consistency issues identified in code review Co-authored-by: johndoe6345789 <224850594+johndoe6345789@users.noreply.github.com> --- kernel/src/memory.cpp | 3 +-- kernel/src/timer.cpp | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/src/memory.cpp b/kernel/src/memory.cpp index 89ff90f..de8aa27 100644 --- a/kernel/src/memory.cpp +++ b/kernel/src/memory.cpp @@ -224,8 +224,7 @@ void* HeapAllocator::calloc(size_t num, size_t size) { */ void HeapAllocator::free(void* ptr) { (void)ptr; - // TODO: Implement proper free with a real allocator - // For now, bump allocator doesn't support freeing + // Bump allocator doesn't support freeing individual allocations } /* Memory utility functions */ diff --git a/kernel/src/timer.cpp b/kernel/src/timer.cpp index 5a332b4..3803474 100644 --- a/kernel/src/timer.cpp +++ b/kernel/src/timer.cpp @@ -52,7 +52,7 @@ Timer::Timer() : ticks(0) {} * @brief Initialize the PIT to generate interrupts at specified frequency * * The PIT works by counting down from a divisor value at its base frequency - * of 1.193182 MHz. When the counter reaches zero, it generates an interrupt + * of 1193182 Hz (approximately 1.19 MHz). When the counter reaches zero, it generates an interrupt * and reloads the divisor. * * For example, to get 1000 Hz (1ms ticks):