<span class="Apple-style-span" style="font-family: Arial; font-size: medium; ">Thanks for the comments.<br style="clear: both; ">I have several questions:<br style="clear: both; "><br style="clear: both; ">1. You mentioned &quot;style problems&quot;. What exactly are they? The web page of<br style="clear: both; ">

mail list will trim the spaces/tabs so it&#39;s hard to judge the problem. Last<br style="clear: both; ">time, you commented my code that there should be no space between the<br style="clear: both; ">function name and &quot;(&quot;, that&#39;s clear. I want the suggestions like that.<br style="clear: both; ">

<br style="clear: both; "><br style="clear: both; ">2. You mentioned &quot;It&#39;s not clear why you are trying to do two different<br style="clear: both; ">&quot;resolve&quot; in a raw, each of them calling gethostbyname/getaddrinfo. Looks<br style="clear: both; ">

silly.&quot; Do you mean I use &quot;getaddrinfo/gethostname&quot; in<br style="clear: both; ">&quot;ngx_inet_parse_hostname&quot; and &quot;ngx_inet_resolve_host&quot; respectively? The old<br style="clear: both; ">code use twice &quot;gethostbyname&quot; too. So I follow the way. What&#39;s your<br style="clear: both; ">

expectation? Please make that clear?<br style="clear: both; "><br style="clear: both; "><br style="clear: both; ">3. You mentioned &quot;This change is completely unrelated and not really needed.<br style="clear: both; ">

You may want to avoid cluttering patch with such changes.&quot; Do u mean I<br style="clear: both; ">shouldn&#39;t convert the code to the new function &quot;ngx_inet_resolve_host_name&quot;,<br style="clear: both; ">or shouldn&#39;t I move up the &quot;sin-&gt;sin_port&quot; set code.<br style="clear: both; ">

<br style="clear: both; "><br style="clear: both; ">4. I admit this diff is hard to review, especially in the function<br style="clear: both; ">&quot;ngx_inet_resolve_host&quot;. The old code structure is completely changed and<br style="clear: both; ">

rewritten because a) the interface of getaddrinfo and gethostbyname is<br style="clear: both; ">different; 2) I try to avoid too much duplicated code brought by directly<br style="clear: both; ">expending the old code. So I strongly suggest you should review the merge<br style="clear: both; ">

result of &quot;ngx_inet_resolve_host&quot;. It&#39;s not feasible that you can always<br style="clear: both; ">tell the mechanism of the fix from diff. Of course, I&#39;ll try my best to<br style="clear: both; ">shrink and split the diff into little ones.<br style="clear: both; ">

<br style="clear: both; ">I can imagine how busy you are. But I&#39;m a new guys to do the nginx dev so<br style="clear: both; ">please give more advices like &quot;what is good&quot; and &quot;what should be done&quot;. This<br style="clear: both; ">

would save time to both of us.<br style="clear: both; "><br style="clear: both; ">Thanks.</span><br><br>