This is the mail archive of the cygwin-apps 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]
Other format: [Raw text]

Re: Remove legacy networking code


On 28/01/2018 21:47, Jon Turney wrote:
On 26/01/2018 16:56, SZAVAI Gyula wrote:
[PATCH setup 1/5] Remove direct(legacy) connection type
[PATCH setup 2/5] Rename IDC_NET_IE5 to IDC_NET_PRECONFIG
[PATCH setup 3/5] Remove NetIO_File
[PATCH setup 4/5] Add Proxy class
[PATCH setup 5/5] Remove NetIO_HTTP

Thanks, this is great.

I applied this patch series to setup, with a few tweaks to address my comments.

Sorry about the delay and thanks again!


A few minor comments:

1/5:

It would be nice if you'd said in the patch commentary what will happen to existing installations which are configured to use direct(legacy) - they get converted to preconfig.

3/5:

This changes to wininet handling all file access (converting raw paths into file:// format URLs)

I had some concerns [1] that this maybe changes the set of malformed file:// URLs we handle (which is ok, although it would be nice to know the details), and still doesn't handle correctly formed ones.

[1] https://cygwin.com/ml/cygwin-apps/2017-05/msg00117.html

4/5:

operator!= could probably use a comment along the lines of "it's only meaningful to compare host:port if IDC_NET_PROXY"

It's unclear to me what Proxy::bypass() is doing.  MSDN actually says "Do not use an empty string", for unclear reasons.

We can also remove simpsock.{cc,h} and linking with winsock, after this series.


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