Fix typo in tensoralloc for Bumper allocator

[leburgel]

Fixes what I think is a typo in the Bumper allocator. I assume this was supposed to be an ndims check to flag zero-dimensional arrays, but I think there were some parentheses missing which made it so that temporary arrays of rank<4 were never actually allocated in the buffer.

Or I'm missing something, but the patch seems to fix leaking allocations on my end and the tests still pass. @lkdvos

[lkdvos]

That does indeed look like a typo... If I'm not mistaken I intended it to be &&, which seemingly has different precedence rules than & for some reason. Could you check if that also fixes it?

Edit: only now saw @Jutho s reply

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.77%. Comparing base (455a69f) to head (efe4659).
Report is 27 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   83.06%   82.77%   -0.30%     
==========================================
  Files          25       28       +3     
  Lines        2150     2456     +306     
==========================================
+ Hits         1786     2033     +247     
- Misses        364      423      +59     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[leburgel]

That does indeed look like a typo... If I'm not mistaken I intended it to be &&, which seemingly has different precedence rules than & for some reason. Could you check if that also fixes it?

Yes, this also fixes it, so changed the fix to && here.

[Jutho]

I'll merge. I thought this was a good time to check whether the second part of the if clause was still necessary, but it seems the issue in StrideArraysCore.jl is still open:
JuliaSIMD/StrideArraysCore.jl#28