search

TacOS-team / tacos

Système d'exploitation / Operating System
GNU General Public License v3.0
9 stars 4 forks source link

Refactoring de create_process #204

Closed NicolasFloquet closed 9 years ago

NicolasFloquet commented 9 years ago

Actuellement on a deux implémentations de create_process, une pour un process à partir d'un elf, l'autre pour un process kernel. C'est crade.

MaximeCheramy commented 9 years ago

Ouais, il y a une grosse redondance de code, c'est assez pénible. Genre lorsqu'on veut rajouter un truc faut pas oublier l'autre. C'est vraiment nécessaire.

NicolasFloquet commented 9 years ago

Je viens de faire la factorisation rapidement en mettant des if(elf) là où il faut, il y en a pas tant que ça, et ils peuvent tous être réglés par une utilisation plus intelligente du process_init_data_t Par ailleurs, j'ai trouvé quelques bugs potentiels probablement issus de la duplication du code, donc ça va effectivement faire du bien je pense.

MaximeCheramy commented 9 years ago

Et tu penses commiter ta factorisation ? :)

NicolasFloquet commented 9 years ago

Evidemment! Mais je suis pas en vacances, ni chercheur :D

MaximeCheramy commented 9 years ago

ref #23

MaximeCheramy commented 9 years ago

Tu penseras à réactiver le hook pour git. Tu as ajouté des warnings.

NicolasFloquet commented 9 years ago

Effectivement, désolé ^^