qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] block-vvfat.c - fix is_free predicate


From: Tristan Gingold
Subject: Re: [Qemu-devel] [PATCH] block-vvfat.c - fix is_free predicate
Date: Thu, 26 Jun 2008 08:46:00 -0400


On Jun 26, 2008, at 8:21 AM, Johannes Schindelin wrote:
This explanation is not good enough.

According to the only documentation I had quickly at hand, Wikipedia, 0x40 is an attribute for a device, which is never found on disk, especially in our implementation. 0x80 is unused. So your "& 0xc0" check seems to be totally bogus, suggesting that we use this kind of attributes anywhere.

However, I agree to the change to 'name[0] == 0x00'.

Ok, let's forget attributes.  Do you agree with this new one?

Tristan.

Attachment: vvfat.diff
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]