<div dir="ltr">Hi Alex,<br>I read your hint and made changes accordingly, it didn't go to spam this time. I was going through the Squid Docs, on how to submit PR and do those pre-merge self tests (Submission Checklist).<br><br>I will do the steps given in the Submission Checklist and submit a PR in a day or two.<br><br>Thank you for helping me with getting started and joining the project.<br><br>Regards<br>Shailesh<br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 4, 2024 at 6:53 PM Alex Rousskov <<a href="mailto:rousskov@measurement-factory.com">rousskov@measurement-factory.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2024-10-03 15:49, Shailesh Vashishth wrote:<br>
<br>
> I would like to contribute to the Squid Project and I have picked one <br>
> small task out of the ToDos  for myself.<br>
<br>
<br>
Hello Shailesh,<br>
<br>
     Welcome to the Project and good luck with your first task! If you <br>
need any help, please do not hesitate to ask on this mailing list or to <br>
post a draft pull request for discussion on GitHub:<br>
<a href="https://wiki.squid-cache.org/MergeProcedure" rel="noreferrer" target="_blank">https://wiki.squid-cache.org/MergeProcedure</a><br>
<br>
BTW, I have also emailed you a few hints regarding the code below when <br>
you asked about it privately -- check your spam folder if you have not <br>
seen my response dated September 30, 2024...<br>
<br>
<br>
Cheers,<br>
<br>
Alex.<br>
<br>
<br>
>     *// XXX: This is absurd! TODO: For positives, "a/(b/c) > d" is "a*c<br>
>      > b*d".<br>
>     **if (expectlen < 100) {<br>
>     **   debugs(90, 3, "quick-abort? NO avoid FPE");<br>
>     **   return false;<br>
>     **}* <br>
> <br>
>     */_if ((curlen / (expectlen / 100)) > (Config.quickAbort.pct)) {<br>
>     _/**   debugs(90, 3, "quick-abort? NO past point of no return");<br>
>     **   return false;<br>
>     **}*<br>
<br>
<br>
<br>
<br>
<br>
</blockquote></div>