On 03/03/25 3:17 pm, Brendan Jackman wrote:
On Fri, Feb 28, 2025 at 11:06:35PM +0530, Dev Jain wrote:
Taking a cursory look at the test, it creates three threads for each cpu.
The bounding of the variable is fine but that being the reason to rename the
variable is not making sense to me.
Hmm yeah the name needs to be more abstract. Do you think nr_workers
would be confusing? Or even just "parallelism" or nr_parallel? Or any
other ideas?
FWIW I briefly looked at just cleaning this up to remove the global
variable but that's a bigger time investment than I can afford here I
think. (The local variable in stress() would still need a better name
anyway).
Thanks for the review BTW!
Your welcome.
I personally prefer leaving it as is; unless someone comes up and
completely cleans up the structure, let us save our collective brain
cycles for more meaningful battles than renaming variables :)
