search

FrOSt-Foundation / cFrOSt

FrOSt, but in C
4 stars 0 forks source link

First part of refactoring #23

Closed L3nn0x closed 7 years ago

L3nn0x commented 8 years ago

/!\ Do not merge until @Yamakaky solves the issue with the .zero opcode in its assembler.

Yamakaky commented 8 years ago

OK for the renaming of HardwareInfos (no idea why I put it in snake case), but why force the use of struct?

L3nn0x commented 8 years ago

It makes it clearer in the code that we are using a struct. It also reduces the pollution of the global namespace.

Yamakaky commented 8 years ago

I don't find it's a problem with camelcase but why not

Yamakaky commented 8 years ago

I think we should do a separate flibc, maybe in an other repo. It would be clearer which part is userspace

L3nn0x commented 8 years ago

well, without going as far as a different repo, changing the current tree directory could be an improvement.

Yamakaky commented 8 years ago

yes