Bug in fat16_get_first_cluster

[gierens]

Correct me if I'm wrong but we are just ORing the high and low bits here: https://github.com/nibblebits/PeachOS/blob/9518f7a15134c31d66a4efe5a67a8ffba67edf50/src/fs/fat/fat16.c#L355-L358

Shouldn't we shift the high bits like so:

return (item->high_16_bits_first_cluster << 16) | item->low_16_bits_first_cluster;

[nibblebits]

Yes you are correct, this function by the end of the course does not get used anymore, it will be removed in a future lecture entirely

[nibblebits]

Thanks for mentioning the issue