More helpful comments

src/framebuffer.rs

@@ -85,8 +85,7 @@ pub fn framebuffer_tag(tag: &Tag) -> FramebufferTag {
     let height = reader.read_u32();
     let bpp = reader.read_u8();
     let type_no = reader.read_u8();
-    reader.skip(2); // RAAAAAAAAAAAAARGH THIS WAS _NOT_ IN THE MULTIBOOT SPEC AAAASDAUHSDKJAHSDKJAHSD.
-                    // In the multiboot spec, it has this listed as a u8 _NOT_ a u16.
+    reader.skip(2); // In the multiboot spec, it has this listed as a u8 _NOT_ a u16.
                     // Reading the GRUB2 source code reveals it is in fact a u16.
     let buffer_type = match type_no {
         0 =>  {
@@ -97,10 +96,10 @@ pub fn framebuffer_tag(tag: &Tag) -> FramebufferTag {
             FramebufferType::Indexed { palette }
         },
         1 => {
-            let red_pos = reader.read_u8();     //Also.... WHAT DO THESE MEAN????
-            let red_mask = reader.read_u8();    //i mean, i can guess
-            let green_pos = reader.read_u8();   //but i shouldn't have to.
-            let green_mask = reader.read_u8();  //come on multiboot... why is your spec so shite??
+            let red_pos = reader.read_u8();     //These refer to the bit positions of the MSB of each field
+            let red_mask = reader.read_u8();    //And then the length of the field from MSB to LSB
+            let green_pos = reader.read_u8();   
+            let green_mask = reader.read_u8();  
             let blue_pos = reader.read_u8();
             let blue_mask = reader.read_u8();
             FramebufferType::RGB {