charlotte-os / charlotte-core

The Kernel of CharlotteOS
GNU General Public License v3.0
115 stars 9 forks source link

The Big Refactor #135

Closed mdpatelcsecon closed 3 months ago

mdpatelcsecon commented 3 months ago

Refactor the entire kernel codebase to accomplish the following:

    pub mod my_mod;
    use my_mod::*;
Wrench56 commented 3 months ago

One more thing: I would also like to eliminate all Clippy warnings. Yep, I did do this on #101 but it became problematic with the pmm + acpi merges. So I think the easiest way to fix it is to simply redo it.

mdpatelcsecon commented 3 months ago

One more thing: I would also like to eliminate all Clippy warnings. Yep, I did do this on #101 but it became problematic with the pmm + acpi merges. So I think the easiest way to fix it is to simply redo it.

I think this warrants its own separate issue after this one is complete. Feel free to write it up and list it as blocked on this issue.

mdpatelcsecon commented 3 months ago

This is way too much work and would almost require a rewrite of most of the kernel. We can't really afford to make changes this large. Thus this is being cancelled and replaced with a task to refactor arch into one single trait and a set of C wrappers in the bridge library.