Skip to content

Peer review - Linnea Villskog

Din kod ser bra ut och gör det den ska vid körning med rimlig tid. Den har heller inga problem med CI och det finns kommentarer som beskriver det som händer. Jag hittar inga större problem men har tre små punkter om jag ska vara lite petig, se nedan.

Utöver det riktigt bra jobbat! 😁

  • Saknar tester för kernel, din kod fungerar dock som den ska med tester jag skrev själv.

    lab3a_tests.png

  • Lite mycket onödiga nop-instruktioner, exempelvis på rad 61

  • Hade önskat lite mer kommentarer som beskriver din kernel-kod, det är något svårt att förstå ibland utan att kolla på assembly koden