RE: [PATCH 2/5] Cygwin: resolver: Process options forward (not backwards)

2022-01-18 Thread Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
> Thanks for the description. Would you mind to recreate your patch with > a matching commit message text explaining the debug flag setting? Okay, just did. Anton Lavrentiev Contractor NIH/NLM/NCBI

Re: [EXTERNAL] Re: [PATCH 2/5] Cygwin: resolver: Process options forward (not backwards)

2022-01-18 Thread Corinna Vinschen
On Jan 18 13:58, Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches wrote: > > I pushed patches 1 and 3 to 5. I fixed the consitency typo > > throughout. > > Thanks! (and oops :-) > > > Right now, the debug flag gets set in several places throughout the > > code. Given you set the debug f

RE: [EXTERNAL] Re: [PATCH 2/5] Cygwin: resolver: Process options forward (not backwards)

2022-01-18 Thread Lavrentiev, Anton (NIH/NLM/NCBI) [C] via Cygwin-patches
> I pushed patches 1 and 3 to 5. I fixed the consitency typo > throughout. Thanks! (and oops :-) > Right now, the debug flag gets set in several places throughout the > code. Given you set the debug flag above, doesn't that mean several > code snippets setting the debug flag later in the code

Re: [PATCH 2/5] Cygwin: resolver: Process options forward (not backwards)

2022-01-18 Thread Corinna Vinschen
Hi Anton, I pushed patches 1 and 3 to 5. I fixed the consitency typo throughout. As for this path 2: On Jan 17 13:03, Anton Lavrentiev via Cygwin-patches wrote: > --- > winsup/cygwin/libc/minires.c | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/winsup/cygw

[PATCH 2/5] Cygwin: resolver: Process options forward (not backwards)

2022-01-17 Thread Anton Lavrentiev via Cygwin-patches
--- winsup/cygwin/libc/minires.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/winsup/cygwin/libc/minires.c b/winsup/cygwin/libc/minires.c index 0cf9efd9b..fdc6087f5 100644 --- a/winsup/cygwin/libc/minires.c +++ b/winsup/cygwin/libc/minires.c @@ -86,12 +86,12 @@ Re