Differential D5839
Fix T69910: Crash when using a hotkey for a New file template or Open Recent file Authored by Pedro Reis (veryprofessionaldodo) on Sep 18 2019, 2:03 PM.
Details
Hi there, this is my first contribution to Blender, so I apologize before hand if something isn't correct! This was caused by the tooltip clear being called when a there was no window in the wm property of the Context. Tested it by replicating the same behaviour as described in the task, and using Blender normally after that. The output of all actions done were expected behaviour, no crashes. Hope this fixes the problem! :)
Diff Detail
Event TimelineComment Actions Hi @Pedro Reis (veryprofessionaldodo) and thx for the contribution! Also style-wise, please use brackets [this should be caught by running make format in the blender directory, I guess] Comment Actions Oh sorry, thought the formatter of VS Code did that. I'll add the brackets in a bit, going to lunch now Comment Actions @Philipp Oeser (lichtwerk) Quick question, how do I assign a Maniphest Task to a revision? I added a new contribution to a different revision, but I don't know how to associate an existing task. Sorry for bothering with such a trivial question :) EDIT: Nevermind, found it! Comment Actions @Philipp Oeser (lichtwerk) Are there more reviewers who can review this? It looks like a simple fix, and I feel like there's not much that could go wrong with this tweak :) Comment Actions @Pedro Reis (veryprofessionaldodo): not sure, I'd just poke the reviewers already here again :) Comment Actions @Philipp Oeser (lichtwerk) Humm, hadn't considered that, but it does make a lot of sense. At least I hope it might point someone in the right direction! Comment Actions Thank YOU! It's always awesome to help, hope I can add more to Blender in the future! <3 |