On 2024-10-25 15:24, Shailesh Vashishth wrote:
I would like to take up another Todo item that is listed here --
https://github.com/measurement-factory/squid-notes/blob/start/todos.md
Sounds good, but please finish your work with the previous TODO item
(i.e. PR #1921) first:
> My current PR 1921 has succeeded all Pipeline stages,
Correct.
> only your approval is remaining.
Not exactly. PR 1921 is on your side as detailed at the URL below (look
for words "Please adjust..."):
https://github.com/squid-cache/squid/pull/1921#issuecomment-2437843503
If you need help/clarifications about that specific change request,
please followup on GitHub.
Alex.
On Wed, Oct 23, 2024 at 12:30 AM Shailesh Vashishth wrote:
Hi
I have made a change to a math expression in store_client.cc and
raised the following pull request.
if (curlen > expectlen*(Config.quickAbort.pct/100.0)) {
debugs(90, 3, "quick-abort? NO past point of no return");
return false;
}
I have also ran the test-build.sh (suggested in Submissions
Checklist) and it passed all test cases.
Please review the following pull request -
https://github.com/squid-cache/squid/pull/1921
<https://github.com/squid-cache/squid/pull/1921>
Regards
Shailesh
On Sat, Oct 5, 2024 at 1:27 PM Shailesh Vashishth
<shavashis...@gmail.com <mailto:shavashis...@gmail.com>> wrote:
Hi Alex,
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).
I will do the steps given in the Submission Checklist and submit
a PR in a day or two.
Thank you for helping me with getting started and joining the
project.
Regards
Shailesh
On Fri, Oct 4, 2024 at 6:53 PM Alex Rousskov
<rouss...@measurement-factory.com
<mailto:rouss...@measurement-factory.com>> wrote:
On 2024-10-03 15:49, Shailesh Vashishth wrote:
> I would like to contribute to the Squid Project and I
have picked one
> small task out of the ToDos for myself.
Hello Shailesh,
Welcome to the Project and good luck with your first
task! If you
need any help, please do not hesitate to ask on this mailing
list or to
post a draft pull request for discussion on GitHub:
https://wiki.squid-cache.org/MergeProcedure
<https://wiki.squid-cache.org/MergeProcedure>
BTW, I have also emailed you a few hints regarding the code
below when
you asked about it privately -- check your spam folder if
you have not
seen my response dated September 30, 2024...
Cheers,
Alex.
> *// XXX: This is absurd! TODO: For positives,
"a/(b/c) > d" is "a*c
> > b*d".
> **if (expectlen < 100) {
> ** debugs(90, 3, "quick-abort? NO avoid FPE");
> ** return false;
> **}*
>
> */_if ((curlen / (expectlen / 100)) >
(Config.quickAbort.pct)) {
> _/** debugs(90, 3, "quick-abort? NO past point of
no return");
> ** return false;
> **}*
_______________________________________________
squid-dev mailing list
squid-dev@lists.squid-cache.org
https://lists.squid-cache.org/listinfo/squid-dev
_______________________________________________
squid-dev mailing list
squid-dev@lists.squid-cache.org
https://lists.squid-cache.org/listinfo/squid-dev
