Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry-picks for the 3.2 branch (future 3.2.4) - 6th batch #43162

Merged
merged 27 commits into from
Oct 28, 2020

Conversation

akien-mga
Copy link
Member

No description provided.

YeldhamDev and others added 23 commits October 28, 2020 14:05
Continuing the work from f43a0ef,
It seems the maps visibility was not actually set and a flase value was propogated to the meshes
Trying to set the maps visibility directly (Using `set_visibility` causes the map to no longer to recieve visibility notifications, instead this approach was chosen

(cherry picked from commit fe52c6b)
(cherry picked from commit 395cb57)
(cherry picked from commit 42959d5)
They use the same underlying `CanvasItem.draw_rect()` API.

(cherry picked from commit 803b046)
Exposes a hidden parameter behind `CanvasItem.draw_rect()`.

(cherry picked from commit aef0002)
minizip documentation describes tm_mon as expecting the number of months
since January - [0, 11], but the month returned by OS.get_date() is in
the range of [1, 12].

(cherry picked from commit 5fe9022)
(cherry picked from commit 0e392bd)
(cherry picked from commit 0d3a168)
It's not an engine module. This is handled by GDNative's
`SCsub` instead, as done for other subdirectories already.

(cherry picked from commit 650ae41)
(cherry picked from commit 6acd450)
Undo/redo log messages will now specify the modified node's
name (or number of modified nodes if several were modified).
On top of that, the new position/rotation/scale/pivot offset
will also be mentioned in the message.

(cherry picked from commit 996740d)
(cherry picked from commit 9e57a39)
Allows to extend `RandomNumberGenerator` via C++ modules.

(cherry picked from commit 1923f0d)
@akien-mga akien-mga requested review from hpvb and a team as code owners October 28, 2020 13:36
@akien-mga
Copy link
Member Author

The regression test project found a crash from 325b41248efe6890e4495436315825e159789ffb, nice CI!

ERROR: _get_tab_width: Index p_index = 0 is out of bounds (get_tab_count() = 0).
   At: scene/gui/tab_container.cpp:491.
ERROR: get: FATAL: Index p_index = 0 is out of bounds (size() = 0).
   At: ./core/cowdata.h:152.
handle_crash: Program crashed with signal 4
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] bin/godot.x11.tools.64s() [0x1456f46] (/home/runner/work/godot/godot/platform/x11/crash_handler_x11.cpp:54)
[2] /lib/x86_64-linux-gnu/libc.so.6(+0x46210) [0x7f80b32a0210] (??:0)
[3] CowData<Control*>::get(int) const (/home/runner/work/godot/godot/./core/cowdata.h:152 (discriminator 7))
[4] Vector<Control*>::operator[](int) const (/home/runner/work/godot/godot/./core/vector.h:85)
[5] TabContainer::_draw_tab(Ref<StyleBox>&, Color&, int, float) (/home/runner/work/godot/godot/scene/gui/tab_container.cpp:428)
[6] TabContainer::_notification(int) (/home/runner/work/godot/godot/scene/gui/tab_container.cpp:380)
[7] TabContainer::_notificationv(int, bool) (/home/runner/work/godot/godot/scene/gui/tab_container.h:38 (discriminator 14))
[8] Object::notification(int, bool) (/home/runner/work/godot/godot/core/object.cpp:934)
[9] CanvasItem::_update_callback() (/home/runner/work/godot/godot/scene/2d/canvas_item.cpp:460)
[10] MethodBind0::call(Object*, Variant const**, int, Variant::CallError&) (/home/runner/work/godot/godot/./core/method_bind.gen.inc:61 (discriminator 4))
[11] Object::call(StringName const&, Variant const**, int, Variant::CallError&) (/home/runner/work/godot/godot/core/object.cpp:922 (discriminator 1))
[12] MessageQueue::_call_function(Object*, StringName const&, Variant const*, int, bool) (/home/runner/work/godot/godot/core/message_queue.cpp:250)
[13] MessageQueue::flush() (/home/runner/work/godot/godot/core/message_queue.cpp:299)
[14] SceneTree::iteration(float) (/home/runner/work/godot/godot/scene/main/scene_tree.cpp:484 (discriminator 2))
[15] Main::iteration() (/home/runner/work/godot/godot/main/main.cpp:2086)
[16] OS_X11::run() (/home/runner/work/godot/godot/platform/x11/os_x11.cpp:3396)
[17] bin/godot.x11.tools.64s(main+0x334) [0x144dd3a] (/home/runner/work/godot/godot/platform/x11/godot_x11.cpp:57)
[18] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf3) [0x7f80b32810b3] (??:0)
[19] bin/godot.x11.tools.64s(_start+0x2e) [0x144d94e] (??:?)

Poke @volzhs (might simply be that I messed up in my cherry-pick, I'll investigate too).

pouleyKetchoupp and others added 4 commits October 28, 2020 15:09
New corporate sponsors:
  Spiffcode (Platinum)
  ASIFA-Hollywood (Silver)
  Zenva Academy (Silver)

Thanks to all contributors and donors for making Godot possible!

(cherry picked from commit 4960894)
@akien-mga
Copy link
Member Author

The regression test project found a crash from 325b412, nice CI!

Reverted in that PR.

@akien-mga akien-mga merged commit 0b6d75c into godotengine:3.2 Oct 28, 2020
@akien-mga akien-mga modified the milestones: 3.2, 3.3 Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet