[PATCH] Cygwin: Provide more COM devices
Corinna Vinschen
corinna-cygwin@cygwin.com
Mon Nov 4 09:28:00 GMT 2019
On Nov 3 20:13, Achim Gratz wrote:
> Corinna Vinschen writes:
> […]
> > ttyS%(0-255) takes another 23K btw. Even that should be ok, if
> > the need arises. Alternatively we could shortcut shilka as for
> > /dev/sd*, but that involved much bigger numbers.
>
> I've searched for some documentation (anywhere the glob syntax %(1-128)
> would turn up, btw?) and I think Cygwin is misusing shilka a bit here.
> It's a keyword scanner, so the arithmetically coded parts of the device
> shouldn't be targeted at all. Instead, only the device path prefix
> should be searched via the shilka lexer and the rest of the conversion
> done in code.
Sure, fine with me.
> For the disks we might keep the globbing that gets us the
> device major part. That of course means we construct more devices
> on-the-fly (or even all of them) and have a much smaller table of static
> device entries
I don't think so. We already construct the numbered devices on the
fly, see fhandler_dev::readdir() and the `exists' test in there,
which translates into on of the exists_* tests in devices.in.
> (which get searched linearly, so in the end that should
> be a net speed improvement).
That in turn would be great.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20191104/e7f17679/attachment.sig>
More information about the Cygwin-patches
mailing list