<div dir="ltr">Hi <br>There is another very small ToDo that I would like take up -- <br><br>This is also a very simple and small change -- replacing <b><span style="background-color:rgb(255,255,0)">Io with Store::IoStatus</span> </b>and removing the <b><span style="background-color:rgb(255,255,0)">using</span> </b>statement<br><div><br></div><div><div style="color:rgb(204,204,204);background-color:rgb(31,31,31);font-family:Menlo,Monaco,"Courier New",monospace;font-size:12px;line-height:18px;white-space:pre"><div>Address the following TODO:</div><div><span style="color:rgb(106,153,85)">/* TODO: Remove this change-minimizing hack */</span></div><div><span style="color:rgb(197,134,192)">using</span> <span style="color:rgb(78,201,176)">Io</span> <span style="color:rgb(212,212,212)">=</span> <span style="color:rgb(78,201,176)">Store</span>::<span style="color:rgb(78,201,176)">IoStatus</span>;</div><div><span style="color:rgb(86,156,214)">static</span> <span style="color:rgb(86,156,214)">constexpr</span> Io ioUndecided <span style="color:rgb(212,212,212)">=</span> <span style="color:rgb(78,201,176)">Store</span>::ioUndecided;</div><div><span style="color:rgb(86,156,214)">static</span> <span style="color:rgb(86,156,214)">constexpr</span> Io ioReading <span style="color:rgb(212,212,212)">=</span> <span style="color:rgb(78,201,176)">Store</span>::ioReading;</div><div><span style="color:rgb(86,156,214)">static</span> <span style="color:rgb(86,156,214)">constexpr</span> Io ioWriting <span style="color:rgb(212,212,212)">=</span> <span style="color:rgb(78,201,176)">Store</span>::ioWriting;</div><div><span style="color:rgb(86,156,214)">static</span> <span style="color:rgb(86,156,214)">constexpr</span> Io ioDone <span style="color:rgb(212,212,212)">=</span> <span style="color:rgb(78,201,176)">Store</span>::ioDone;</div></div></div><div><br></div><div><br></div><div>Regards<br>Shailesh</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Oct 26, 2024 at 12:54 AM Shailesh Vashishth <<a href="mailto:shavashishth@gmail.com">shavashishth@gmail.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"><div dir="ltr">Hi<br>I would like to take up another Todo item that is listed here -- <a href="https://github.com/measurement-factory/squid-notes/blob/start/todos.md" target="_blank">https://github.com/measurement-factory/squid-notes/blob/start/todos.md</a><div><br></div><div>I would like to pick this item : I think this requires very small change, where we just print <b>name_len </b>on the <b>stdout </b>using the custom function for printing debug logs.<br><br><ul dir="auto" style="box-sizing:border-box;padding-left:2em;margin-top:0px;color:rgb(240,246,252);font-family:-apple-system,"system-ui","Segoe UI","Noto Sans",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji";background-color:rgb(13,17,23)"><li style="box-sizing:border-box">Address the following TODO by adding any using Raw::upto(n) which will limit the size value of passed to PrintHex() and write() inside Raw::print(). That value is currently size_.</li></ul><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="color:rgb(204,204,204);background-color:rgb(31,31,31);font-family:Menlo,Monaco,"Courier New",monospace;font-size:12px;line-height:18px;white-space:pre-wrap"><div><span style="color:rgb(197,134,192)">if</span> (name_len <span style="color:rgb(212,212,212)">></span> <span style="color:rgb(181,206,168)">65534</span>) {</div><div><span style="color:rgb(106,153,85)">    /* String must be LESS THAN 64K and it adds a terminating NULL */</span></div><div><span style="color:rgb(106,153,85)">    // TODO: update this to show proper name_len in Raw markup, but not print all that</span></div><div>    <span style="color:rgb(220,220,170)">debugs</span>(<span style="color:rgb(181,206,168)">55</span>, <span style="color:rgb(181,206,168)">2</span>, <span style="color:rgb(206,145,120)">"ignoring huge header field ("</span> <span style="color:rgb(212,212,212)"><<</span> <span style="color:rgb(220,220,170)">Raw</span>(<span style="color:rgb(206,145,120)">"field_start"</span>, field_start, <span style="color:rgb(181,206,168)">100</span>) <span style="color:rgb(212,212,212)"><<</span> <span style="color:rgb(206,145,120)">"...)"</span>);</div></div></blockquote><br><br>My current PR (<u><a href="https://github.com/squid-cache/squid/pull/1921" target="_blank">https://github.com/squid-cache/squid/pull/1921</a></u>) has succeeded all Pipeline stages, only your approval is remaining.<br><br>Regards<br>Shailesh<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 23, 2024 at 12:30 AM Shailesh Vashishth <<a href="mailto:shavashishth@gmail.com" target="_blank">shavashishth@gmail.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"><div dir="ltr"><div dir="ltr">Hi<br>I have made a change to a math expression in store_client.cc and raised the following pull request.<br><br><div style="color:rgb(204,204,204);background-color:rgb(31,31,31);font-family:Menlo,Monaco,"Courier New",monospace;font-size:12px;line-height:18px;white-space:pre-wrap"><div><span style="color:rgb(197,134,192)">if</span> (<span style="color:rgb(156,220,254)">curlen</span> <span style="color:rgb(212,212,212)">></span> <span style="color:rgb(156,220,254)">expectlen</span><span style="color:rgb(212,212,212)">*</span>(<span style="color:rgb(156,220,254)">Config</span>.<span style="color:rgb(156,220,254)">quickAbort</span>.<span style="color:rgb(156,220,254)">pct</span><span style="color:rgb(212,212,212)">/</span><span style="color:rgb(181,206,168)">100.0</span>)) {</div><div>        <span style="color:rgb(86,156,214)">debugs</span>(<span style="color:rgb(181,206,168)">90</span>, <span style="color:rgb(181,206,168)">3</span>, <span style="color:rgb(206,145,120)">"quick-abort? NO past point of no return"</span>);</div><div>        <span style="color:rgb(197,134,192)">return</span> <span style="color:rgb(86,156,214)">false</span>;</div><div>    }</div></div><br>I have also ran the test-build.sh (suggested in Submissions Checklist) and it passed all test cases. <br><br>Please review the following pull request - <a href="https://github.com/squid-cache/squid/pull/1921" target="_blank">https://github.com/squid-cache/squid/pull/1921</a><br><br>Regards<br>Shailesh<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Oct 5, 2024 at 1:27 PM Shailesh Vashishth <<a href="mailto:shavashishth@gmail.com" target="_blank">shavashishth@gmail.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"><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" target="_blank">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>
</blockquote></div></div>
</blockquote></div>
</blockquote></div>