This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Serial programming patch


On Wed, May 23, 2001 at 09:46:59AM +0200, Jacek Trzcinski wrote:
> Hi Corinna,
> I see it will be cost a little pain.
> What concerns Your 3-rd remark (code not conforming GNU standard) is it
> enough to
> transform my sources utilizing indent.exe ?. If so, what switches should
> I use ?

GNU indent does what it should do without switches. The GNU coding
standard is the default. However, I suggest to change your patched
sources by hand since there are two problems with indent. On one
hand it's sometimes not that clever when indenting C++ sources and
- most important - if you indent all concerned files they will be
reformated everywhere and your patch will include parts which you
never touched.

Corinna

> I have two remarks I forgot mention about in my previous e-mail.
> 
> To read input queue there should be use constant FIONREAD (information
> taken from different sources) but cygwin occupy it for socket software
> !(it is macro). Constant TIOCINQ I have
> taken from linux where it is equal FIONREAD(FIONREAD is not occupied in
> linux like in cygwin).

I see. You should be able to use it anyway by including asm/socket.h.
The macro expands to a constant expression. I understand that including
asm/socket.h seems to make not much sense. Probably we should create
a asm/ioctls.h file like in Linux.

Anyway, may I suggest to discuss that in cygwin-patches@cygwin.com?

> Next matter concerns device name convention. In cygwin one may use
> /dev/com1 or /dev/ttyS1
> and so on. (what for is utilizing /dev/comx I do not know - it is not
> portable to linux).

Good ol' COM...

> I think also that number of devices should start from 0 not from 1
> (/dev/ttyS0 and so on) like in linux because again it is not portable.
> What is Your opinion ?

As above: Let's discuss that in cygwin-patches@cygwin.com.

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.

--
Want to unsubscribe from this list?
Check out: http://cygwin.com/ml/#unsubscribe-simple


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]