| Commit message (Collapse) | Author | Age |
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fix integer overflow in mapgen
Some calculations involving the magic seed had overflow because the result of an intermediate arithmetic step could not fit in an s32. By making the magic seed unsigned, the other operand in the equation will be cast to unsigned, and possibly other operands or intermediate operands. This will result in unexpected behavior if an operand is negative, which is technically possible, but logically should not happen.
* comment noise2d bitshift
While working through the code I was momentarily concerned that the right bitshift in noise2d could fill ones in some cases. It turns out that with signed integers, this is indeed true, but this one is shifting an unsigned integer, so the behavior is as expected. I put a comment here to clarify this, in case someone else wonders the same thing down the line.
* noise2d and noise3d unittests
I have added 3 tests each for noise2d and noise3d, testing all zero inputs, a very large seed (case which caused UB in the old implementation) and some fun primes I picked for no particular reason. This should be sufficient to demonstrate that the behavior of the new implementation has not changed. I used uniform initialization because it is a good feature of C++11. Please do not explode.
* uncomment the noise2d bitshift
This reverts commit 583b77ee9f1ad6bb77340ebb5ba51eb9a88ff51c. It's a
well-defined language semantic; it doesn't need to be commented.
* code cleanliness
|
| |
|
|
|
|
|
|
| |
This was added a long time ago in 42bbd5c9ae06a8d8ffb7915599097ead6f848755
and meant to fix prevent the view becoming black when jumping into a
ceiling, this no longer happens today.
|
| |
|
| |
|
| |
|
|
|
|
| |
bug introduced in 8908a9101608d3343023b470743ef63f1c44b0b7
|
|
|
|
| |
fixes #10769
|
| |
|
|
|
|
|
|
| |
issue: #10884
This makes it possible for objects to immediately be activated,
but doesn't guarantee it since blocks may still need be emerged.
|
|
|
|
| |
fixes #12216
|
|
|
|
| |
fixes #11819
|
|
|
|
| |
fixes #11975
|
|
|
|
| |
bug introduced in 2f32044273d107e82fb1c35d4a0f616fa480cdf0
|
|
|
|
| |
closes #12149
|
| |
|
|
|
|
| |
fixes #8943
|
|
|
|
| |
The distance check also never worked as intended, now fixed.
|
|
|
|
| |
closes #12272
|
|
|
|
| |
Of those settings main_menu_script has concrete security impact, the rest are added out of abundance of caution.
|
|
|
|
| |
fixes #9842
|
|
|
|
| |
fixes #11442
|
| |
|
|
|
|
| |
instead of installing outdated packages which lead to failing workflows
|
|
|
| |
Also add formspec_escape unit test
|
| |
|
| |
|
|
|
|
|
|
| |
On my system this is a reduction from 4664 to 3704 bytes.
This is not for the sake of saving RAM but ensuring
commonly used structures fit into caches better.
|
| |
|
| |
|
|
|
|
|
|
|
| |
* Quantize light frustum calculations
Reduces shadow flicker
* Fix function name to match conventions
|
|
|
|
| |
It stopped working with (at least) the last commit.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add relative numbers for commands by prepending ~
* Some builtin code cleanup
* Disallow nan and inf in minetest.string_to_area
* Remove unused local variable teleportee (makes Luacheck happy)
* Clean up core.string_to_pos
* Make area parsing less permissive
* Rewrite tests as busted tests
* /time: Fix negative minutes not working
Co-authored-by: Lars Mueller <appgurulars@gmx.de>
|
| |
|
| |
|
| |
|
|
|
|
|
| |
If the server stops talking to us without saying bye we
should actually end the in-game session with an error message.
|
|
|
|
| |
(#12334)
|
| |
|
| |
|
|
|
|
|
|
| |
stringstreams were shown to be slow when reading/writing single characters
and there is lots of potential by having functions perform on existing
buffers whenever possible.
|