Hi Anton,

On Jan 29 14:29, Anton Lavrentiev via Cygwin-patches wrote:
> - Use %S (instead of %s) when a wide-character output is due;

Please use %ls, %S is non-standard.

> - Use native byte order for host and add port when doing I/O with DNS server;

This puzzeled me a bit so I took another look into the original code.
Do I see this right that we have only limited IPv6 support in the
resolver code?  For instance, write_record appears to handle DNS_TYPE_A,
but not DNS_TYPE_AAAA.

CCing Pierre, hopefully he's still around.  Pierre, can you please
have a look and suggest how to go forward here?

> - Use forward way for resolv.conf's "options" processing, so listing "debug" 
> as a
>   first option, will show all following option(s) as they are read;
> - Re-evaluate debug output flag after each "options" processing as it may 
> chance.

Would you mind to split this into a patchset with patches for different
tasks?  ATM I'm a bit concerned about the ntoh{sl} calls, given the
noticable absence of IPv6 support...


Thanks,
Corinna
  • [PATCH] CYGWIN: F... Anton Lavrentiev via Cygwin-patches
    • Re: [PATCH] C... Corinna Vinschen via Cygwin-patches
      • RE: [PATC... Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
        • Re: [... Corinna Vinschen via Cygwin-patches
          • R... Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
            • ... Corinna Vinschen via Cygwin-patches
              • ... Brian Inglis
                • ... Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches

Reply via email to