Hi Gerald,

> This mimicks what we already do a few lines above for buf_type.

It's also unneeded:

    switch(ioctl & 0x18000000)

There are only 4 possible combinations of ioctl & 0x18000000.  Three
are covered in this switch statement, and the fourth is eliminated due
to the outer switch statement.  You'd be adding dead code.
--Juan


Reply via email to