Please review: Patch for non-Linux serial set_baud_rate()
Ken Thomases
ken at codeweavers.com
Wed Jun 30 11:47:25 CDT 2010
Hi,
I happened to stumble across some code which didn't look quite right in dlls/ntdll/serial.c. However, I have no expertise in serial comms or termios. Also, I have no means to test the change I propose. On the other hand, the change seems clearly right and conforms to what I learn from man pages, etc.
So, I'm hoping somebody who is more familiar with this functionality can review the attached patch before I formally submit it.
Another approach that seems better from my reading of the man pages is to switch to using cfsetspeed() [or cfsetospeed() and cfsetispeed()] instead of having separate branches based on defined(CBAUD). But maybe that has portability problems worse than the current approach.
Thanks,
Ken
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ntdll-Fix-set_baud_rate-for-high-rates-for-non-Linux.patch
Type: application/octet-stream
Size: 1270 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20100630/e1f0044b/attachment.obj>
More information about the wine-devel
mailing list