I think I probably need to get on the mailing list, but...

On Mon, Apr 22, 2002 at 08:46:03AM +0200, Gerald Pfeifer wrote:
> The following patch
> 
>   revision 1.56
>   date: 2002/04/21 22:06:09;  author: julliard;  state: Exp;  lines: +24 -0
>   <[EMAIL PROTECTED]>
>   Faster serial speed cases for non Linux systems.
> 
> broke dlls/kernel/comm.c on FreeBSD 4.5 (and probably all non-Linux
> systems)

The 57600 and up case (even for Linux) didn't have the CBR_ prefix before
them, which is why I didn't have them in the patch I sent to Bugzilla.
I compiled the changes I submitted successfully on FreeBSD 4.5-RELEASE,
so it should have worked OK.

I'll admit to being a newbie at the entire innards of Wine, so I may have
misunderstood something.  But the code inside the "CBAUD" (I assume this
is what Linux uses) #ifdef has all baud rates >38400 without
the CBR_ prefix in the switch statement.  I simply copied this same
set of case statements to the non-CBAUD version (the #elif !defined(__EMX__)
case) and changed the port.c_cflag to port.c_ospeed.

Hope this helps explain my thought processes.  It looks like that whole
pair of switch/case/#ifdef statements is screaming for a bit of
simplification.

(PS: I'll try to cc the mailing lists, but if it doesn't make it, would
someone please forward it along ... thanks)

---
Gil Kloepfer
[EMAIL PROTECTED]

Reply via email to