On Tue, Jan 03, 2006 at 12:30:45PM +0100, Jan Blunck wrote:
> typedef struct T30_s {
> /* session parameters */
> - __u8 resolution __attribute__ ((packed));
> - __u8 rate __attribute__ ((packed));
> - __u8 width __attribute__ ((packed));
> - __u8 length __attribute__ ((packed));
> - __u8 compression __attribute__ ((packed));
> - __u8 ecm __attribute__ ((packed));
> - __u8 binary __attribute__ ((packed));
> - __u8 scantime __attribute__ ((packed));
> - __u8 id[FAXIDLEN] __attribute__ ((packed));
> + __u8 resolution;
> + __u8 rate ;
> + __u8 width ;
> + __u8 length ;
> + __u8 compression ;
> + __u8 ecm ;
> + __u8 binary ;
> + __u8 scantime ;
> + __u8 id[FAXIDLEN] ;
What's with the funky placement of ; ?
The rest of the struct looks sensible.
>
> /* various cruft */
> - u32 dataA[6] __attribute((packed));
> - u16 dataB[5] __attribute((packed));
> - u32 dataC[14] __attribute((packed));
> -};
> + u32 dataA[6];
> + u16 dataB[5];
> + u32 dataC[14];
> +} __attribute((packed));
You could fix the horked indentation at the same time.
(This crops up in a few places in this diff)
Dave
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
