[PHP-DEV] Re:

[email protected] (سپهر محمودی)
Newsgroups php.internals
Message-ID <CA+Jcf5t-DQfhKWZtH8dTsvNE4A5prNqSzw_K9SGmv_ggTJtAmQ@mail.gmail.com>
Hi Yuya, the "genius",

Look—I don't just talk.

First, when I introduced the RFC, before the discussion started, many
people asked: "How is this different from other functions like
array_search?" I provided the analysis, and there were many questions like
that.

You told me to run a real-world test, and you saw that my function
performed better than those functions.

All the code was designed based on proper principles, and I am sure you saw
that yourself. You didn't even leave a single comment during the discussion
period because you knew you would face a completely technical response.

Just for your information: someone praised my work who holds a high
position in php-src and whom I truly respect—meaning your opinion doesn't
really matter to me.

Respectfully,
Sepehr


در تاریخ چهارشنبه ۹ سپتامبر ۲۰۲۶، ۱۹:۴۷ youkidearitai <
[email protected]> نوشت:

> No, If you are a skilled engineer, you should accept criticism.
> Your skill is not enough php-src development.
>
> To begin with, To unable resolving issue, You must compile and test on
> your PC.
> Otherwise, Your problem is CI is red, but your test code is broken.
> Not CI's problem.
> Fix below..
>
>   ========DIFF========
>   001- === BENCHMARK RESULTS (%d iterations) ===
>   002- \[Match at start\]
>   003-   C (array_str_contains) : %f ms
>   004-   Userland (foreach)     : %f ms
>   005-   Speedup                : %f x faster
>   001+ === BENCHMARK RESULTS (1,000 iterations)label
>
>   007- \[Match in middle\]
>   008-   C (array_str_contains) : %f ms
>   009-   Userland (foreach)     : %f ms
>   010-   Speedup                : %f x faster
>   003+ [Match at start]
>   004+   C (array_str_contains) :   11.40 ms
>   005+   Userland (foreach)     :    2.74 ms
>   006+   Speedup                :    0.24x faster
>
>   012- \[Match at end\]
>   013-   C (array_str_contains) : %f ms
>   014-   Userland (foreach)     : %f ms
>   015-   Speedup                : %f x faster
>   008+ [Match in middle]
>   009+   C (array_str_contains) :   12.85 ms
>   010+   Userland (foreach)     :    5.94 ms
>   011+   Speedup                :    0.46x faster
>
>   017- \[No match (all)\]
>   018-   C (array_str_contains) : %f ms
>   019-   Userland (foreach)     : %f ms
>   020-   Speedup                : %f x faster
>   013+ [Match at end]
>   014+   C (array_str_contains) :   11.58 ms
>   015+   Userland (foreach)     :    7.16 ms
>   016+   Speedup                :    0.62x faster
>   017+
>   018+ [No match (all)]
>   019+   C (array_str_contains) :    6.47 ms
>   020+   Userland (foreach)     :    8.22 ms
>   021+   Speedup                :    1.27x faster
>   ========DONE========
>   FAIL Benchmark array_str_contains vs userland implementations
> [ext/standard/tests/array_str_contains_benchmark.phpt]
>   =====================================================================
>
> Furthermore, seems don't increase speed. This approach is maybe wrong.
>
> Anyway, You seems don't hear from people.
> Please hear from people.
>
> Yuya
>
> ‪2026年9月9日(水) 20:56 ‫سپهر محمودی‬‎ <[email protected]>:‬
> >
> >
> >
> > در تاریخ چهارشنبه ۹ سپتامبر ۲۰۲۶، ۱۳:۵۹ youkidearitai <
> [email protected]> نوشت:
> >>
> >> No,
> >>
> >> ‪2026年9月9日(水) 0:40 ‫سپهر محمودی‬‎ <[email protected]>:‬
> >> >
> >> > Hi Yua,
> >> >
> >> > Thanks for your follow-up and for the time you've dedicated to this.
> >> >
> >> > As you know, we've been encountering `Segmentation Fault` or
> `Timeout` errors in the CI tests. I've reduced the benchmark iterations
> from 100,000 to 1,000 to see if the issue was due to excessive system load,
> but unfortunately, the CI error persists, and it seems the problem lies
> elsewhere.
> >> >
> >> > The exact cause of this error is still unclear at this moment.
> >> >
> >> > Following this function, I plan to work on graphics-related functions
> such as `mask`, `str_mask`, and others.
> >> >
> >> >
> >> > در تاریخ سه‌شنبه ۸ سپتامبر ۲۰۲۶، ۰۸:۰۹ youkidearitai <
> [email protected]> نوشت:
> >> >>
> >> >> Hi Sepehr
> >> >>
> >> >> That PR fails your test.
> >> >> Please compile and test from your PC before push the CI.
> >> >> You can check php-src's README
> >> >> https://github.com/php/php-src/blob/master/README.md, If you don't
> >> >> understand how to build and test.
> >> >>
> >> >> ```
> >> >>
> >> >>   ========DIFF========
> >> >>   001- === BENCHMARK RESULTS (%d iterations) ===
> >> >>   002- \[Match at start\]
> >> >>   003-   C (array_str_contains) : %f ms
> >> >>   004-   Userland (foreach)     : %f ms
> >> >>   005-   Speedup                : %f x faster
> >> >>   001+ === BENCHMARK RESULTS (100,000 iterations) ===
> >> >>
> >> >>   007- \[Match in middle\]
> >> >>   008-   C (array_str_contains) : %f ms
> >> >>   009-   Userland (foreach)     : %f ms
> >> >>   010-   Speedup                : %f x faster
> >> >>   003+ [Match at start]
> >> >>   004+   C (array_str_contains) :  577.66 ms
> >> >>   005+   Userland (foreach)     :   13.74 ms
> >> >>   006+   Speedup                :    0.02x faster
> >> >>
> >> >>   012- \[Match at end\]
> >> >>   013-   C (array_str_contains) : %f ms
> >> >>   014-   Userland (foreach)     : %f ms
> >> >>   015-   Speedup                : %f x faster
> >> >>   008+ [Match in middle]
> >> >>   009+   C (array_str_contains) :  632.95 ms
> >> >>   010+   Userland (foreach)     :  251.77 ms
> >> >>   011+   Speedup                :    0.40x faster
> >> >>
> >> >>   017- \[No match (all)\]
> >> >>   018-   C (array_str_contains) : %f ms
> >> >>   019-   Userland (foreach)     : %f ms
> >> >>   020-   Speedup                : %f x faster
> >> >>   013+ [Match at end]
> >> >>   014+   C (array_str_contains) :  625.46 ms
> >> >>   015+   Userland (foreach)     :  482.63 ms
> >> >>   016+   Speedup                :    0.77x faster
> >> >>   017+
> >> >>   018+ [No match (all)]
> >> >>   019+   C (array_str_contains) :  465.12 ms
> >> >>   020+   Userland (foreach)     :  474.41 ms
> >> >>   021+   Speedup                :    1.02x faster
> >> >> ```
> >> >>
> >> >> Regards
> >> >> Yuya
> >> >>
> >> >> ‪2026年9月8日(火) 2:21 ‫سپهر محمودی‬‎ <[email protected]>:‬
> >> >> >
> >> >> > Hi Yuya,
> >> >> >
> >> >> > Could you please take a look at the failed CI checks in my PR when
> >> >> > you have time?
> >> >> >
> >> >> > https://github.com/php/php-src/pull/23512/checks
> >> >> >
> >> >> > I’m getting 503 Service Unavailable errors when trying to access
> >> >> > the logs, so I haven’t been able to determine whether the failures
> >> >> > are related to my changes.
> >> >> >
> >> >> > I’m particularly concerned about the Tracing JIT and FreeBSD
> failures.
> >> >> > I’d appreciate your help understanding them, or rerunning the
> failed
> >> >> > jobs if appropriate.
> >> >> >
> >> >> > Thank you for your time.
> >> >> >
> >> >> > Best regards,
> >> >> > Sepehr
> >> >> >
> >> >>
> >> >>
> >> >> --
> >> >> ---------------------------
> >> >> Yuya Hamada (tekimen)
> >> >> - https://tekitoh-memdhoi.info
> >> >> - https://github.com/youkidearitai
> >> >> -----------------------------
> >>
> >> As I mentioned on GitHub,
> >> https://github.com/php/php-src/pull/23512#issuecomment-5594220142
> >> CI red is your test code is wrong, It's not CI timeout.
> >> You can confirm when build & test on your local PC.
> >>
> >> I'm telling you straight, If you cannot hear from PHP internals
> >> people, You are not enough ability on php-src yet.
> >> You can choose other way to contribute in PHP.
> >>
> >> Regards
> >> Yuya
> >>
> >> --
> >> ---------------------------
> >> Yuya Hamada (tekimen)
> >> - https://tekitoh-memdhoi.info
> >> - https://github.com/youkidearitai
> >> -----------------------------
> >
> >
> > Hello Yuya,
> > Mr. Hamada—several prominent figures in the PHP community have praised
> my work.
> > Please bear in mind that I am just preparing my very first function—and
> yes, it had a minor issue that I shared with you.
> > There was a time when you, too, were creating your first function and
> surely encountered a small glitch—unless, of course, you happen to be Larry
> Page. So, instead of being so critical, please offer your support and
> encouragement to someone who—arguably—is among the first one or two
> Iranians to reach this level through exceptional programming skills.
> > Best regards,
> > Sepehr
>
>
>
> --
> ---------------------------
> Yuya Hamada (tekimen)
> - https://tekitoh-memdhoi.info
> - https://github.com/youkidearitai
> -----------------------------
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.