This should mention what the units of time are for `mtime` in that case, and I'd be more comfortable with it mentioning why it and file_size are both u32.

Also, `&str` can only encode utf-8 strings, but filesystems can have non-utf8 encoded strings, so that's why I recommended using `Path` instead of `&str`