select or move over the transform tool #57737

New Issue

Florent · 2018-11-09T02:13:17+01:00

Florent commented

2018-11-09 02:13:17 +01:00

System Information
windows 10 64 bits and GeForce GTX 1080 Ti

Blender Version
Broken: 2.80 a421cfa8d3
Worked: (optional)

Short description of error
blender crash

Exact steps for others to reproduce the error
Based on a (as simple as possible) attached .blend file with minimum amount of steps

There's 2 ways to crash blender:

the faster way
select the Move tool with the mouse
Hold Shift and select the transform tool
if blender not crash immediately select the Move tool again
repeat the previous steps until blender crash
the slower

select the Move tool with the mouse
Hold Shift and move the mouse up and down over the other tools (transform, rotation, scale)

with these 2 methods blender crash immediately or you have to wait 10, 15 seconds

**System Information** windows 10 64 bits and GeForce GTX 1080 Ti **Blender Version** Broken: 2.80 a421cfa8d31 Worked: (optional) **Short description of error** blender crash **Exact steps for others to reproduce the error** Based on a (as simple as possible) attached .blend file with minimum amount of steps There's 2 ways to crash blender: 1. the faster way select the Move tool with the mouse Hold Shift and select the transform tool if blender not crash immediately select the Move tool again repeat the previous steps until blender crash 2. the slower select the Move tool with the mouse Hold Shift and move the mouse up and down over the other tools (transform, rotation, scale) with these 2 methods blender crash immediately or you have to wait 10, 15 seconds

Florent commented

2018-11-09 02:13:17 +01:00

Added subscriber: @laaameche

Philipp Oeser commented

2018-11-09 12:52:36 +01:00

Added subscribers: @ideasman42, @lichtwerk

Campbell Barton was assigned by Philipp Oeser

2018-11-09 12:52:36 +01:00

Philipp Oeser commented

2018-11-09 12:52:36 +01:00

Can confirm the crash.

Operator TRANSFORM_OT_from_gizmo description ( ot-description ) is NULL here:

1 ui_tooltip_data_append_from_keymap interface_region_tooltip.c 333  0x1f9cad4 
2 ui_tooltip_data_from_tool          interface_region_tooltip.c 577  0x1f9d5ae 
3 UI_tooltip_create_from_button      interface_region_tooltip.c 1226 0x1f9f5f6 
4 ui_but_tooltip_init                interface_handlers.c       7213 0x1f6c02f 
5 WM_tooltip_init                    wm_tooltip.c               123  0x1c72e0b 
6 wm_event_do_handlers               wm_event_system.c          2998 0x1c4dc97

This used to work in ba36e9, bisecting now ...
Since this is tool related, this is one for @ideasman42, I guess?

Can confirm the crash. Operator `TRANSFORM_OT_from_gizmo` description ( `ot-description` ) is NULL here: ``` 1 ui_tooltip_data_append_from_keymap interface_region_tooltip.c 333 0x1f9cad4 2 ui_tooltip_data_from_tool interface_region_tooltip.c 577 0x1f9d5ae 3 UI_tooltip_create_from_button interface_region_tooltip.c 1226 0x1f9f5f6 4 ui_but_tooltip_init interface_handlers.c 7213 0x1f6c02f 5 WM_tooltip_init wm_tooltip.c 123 0x1c72e0b 6 wm_event_do_handlers wm_event_system.c 2998 0x1c4dc97 ``` This used to work in ba36e9, bisecting now ... Since this is tool related, this is one for @ideasman42, I guess?

blender-admin commented

2018-11-09 13:14:10 +01:00

This issue was referenced by 3d0383d259

This issue was referenced by 3d0383d25932aab3091bf321b3a43e51329882fb

Philipp Oeser commented

2018-11-09 13:16:31 +01:00

caused by 60d7740fe6.

Simple fix in D3923 (not sure if this should be solved more gracefully though -- maybe make ot->description not mandatory?)

caused by 60d7740fe6. Simple fix in [D3923](https://archive.blender.org/developer/D3923) (not sure if this should be solved more gracefully though -- maybe make ot->description not mandatory?)

Martin Capitanio commented

2018-11-09 15:25:32 +01:00

Added subscriber: @capnm

Martin Capitanio commented

2018-11-09 15:25:32 +01:00

Oh, this null pointer is actually the cause of #57639.

Philipp Oeser commented

2018-11-09 15:32:35 +01:00

@capnm : so D3923 fixes #57639 as well?

@capnm : so [D3923](https://archive.blender.org/developer/D3923) fixes #57639 as well?

Brecht Van Lommel commented

2018-11-09 15:33:16 +01:00

Added subscriber: @brecht

Brecht Van Lommel commented

2018-11-09 15:33:16 +01:00

The description is allowed to be NULL to indicate there being no description. The check should be ot->description ? rather than ot->description[0] ?.

Adding a tooltip to the operator is good too, since basically all operators should have descriptions, but not the entire fix.

The description is allowed to be `NULL` to indicate there being no description. The check should be `ot->description ?` rather than `ot->description[0] ?`. Adding a tooltip to the operator is good too, since basically all operators should have descriptions, but not the entire fix.

Philipp Oeser commented

2018-11-09 15:56:48 +01:00

thanx @brecht, D3923 updated

thanx @brecht, [D3923](https://archive.blender.org/developer/D3923) updated

Martin Capitanio commented

2018-11-09 15:58:41 +01:00

so D3923 fixes #57639 as well?

Yes, the null-check fixes that. I think I have sometimes accidentally pressed the shift-key while using the transform tool.

By the way, I wonder if it makes sense that the tooltips change:

> so [D3923](https://archive.blender.org/developer/D3923) fixes #57639 as well? Yes, the null-check fixes that. I think I have sometimes accidentally pressed the shift-key while using the transform tool. By the way, I wonder if it makes sense that the tooltips change: ![image.png](https://archive.blender.org/developer/F5460749/image.png)

Philipp Oeser commented

2018-11-09 16:37:55 +01:00

Changed status from 'Open' to: 'Resolved'

Philipp Oeser closed this issue

2018-11-09 16:37:55 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

select or move over the transform tool #57737