xyris icon indicating copy to clipboard operation
xyris copied to clipboard

Update `liballoc` to use `KernelError` enum

Open Kfeavel opened this issue 2 years ago • 3 comments

Kfeavel avatar Oct 25 '21 23:10 Kfeavel

I think this will require use moving the libs directory from the root to the kernel directory. Not sure how else this will work, especially with SCons.

Kfeavel avatar Nov 26 '21 23:11 Kfeavel

We can just add the kernel's public header directory to the CPPPATH and #include the header with the enum in it.

Alternatively we could wrap liballoc's API so that it returns a KernelError.

micahswitzer avatar Dec 01 '21 20:12 micahswitzer

I was thinking about this a bit more and am still erring on the side of moving liballoc into the Kernel directory since the Libraries directory in the root normally implies it's a library that can be used in either kernel or user space. At least, it seems like that's how other projects like Serenity do it.

Kfeavel avatar Dec 01 '21 21:12 Kfeavel