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

Bug#220814: xfree86_4.2.1-14(unstable/sparc): FTBFS



On Tue, Jan 13, 2004 at 02:41:51PM -0500, Daniel Jacobowitz wrote:
> On Mon, Jan 12, 2004 at 12:12:51PM -0500, Branden Robinson wrote:
> > On Thu, Jan 08, 2004 at 03:24:10PM -0500, Daniel Jacobowitz wrote:
> > > OK, here's what I think is happening.
> > > 
> > > X is working around a changed type in kd.h by #define rate period.
> > 
> > A changed type?  To date I have only heard this characterized as a
> > change to the name of a structure member.
> 
> That's a change in the type of the structure, my friend :)

Okay.  I will shamefully wear my scarlet letter of inability to
communicate with Real C Programmers.  :-/

> > What do you mean by the "latter rate"?  I thought the same thing was
> > being referred to in both cases: the kbd_rate struct.
> 
> That is the center of the confusion.
> 
> asm-sparc/kbio.h:
> 
> /* Used by KIOC[GS]RATE */
> struct kbd_rate {
>         unsigned char delay;    /* Delay in Hz before first repeat.  */
>         unsigned char rate;     /* In characters per second (0..50).  */
> };
> 
> linux/kd.h:
> struct kbd_repeat {
>         int delay;      /* in msec; <= 0: don't change */
>         int period;     /* in msec; <= 0: don't change */
>                         /* earlier this field was misnamed "rate" */
> };
> 
> One of those is kbd_rate, the other is kbd_repeat.  They aren't the
> same type.  I imagine from the errors that the X source file in
> question uses both types.

Aha.  Thanks for the clarification; I think I understand now.

I'm leaving the bug tagged "help", though, because I could use it.

Thanks again.

-- 
G. Branden Robinson                |
Debian GNU/Linux                   |           //     // //  /     /
branden@debian.org                 |           EI 'AANIIGOO 'AHOOT'E
http://people.debian.org/~branden/ |

Attachment: signature.asc
Description: Digital signature


Reply to: