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.

Sign up for GitHub

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

Jump to bottom

Lua Editor: Crash on launch in AzToolsFramework::AssetBrowser::SourceAssetBrowserEntry::GetSourceControlThumbnailKey #15635

Closed
jckand opened this issue Apr 7, 2023 · 1 comment
Closed

Lua Editor: Crash on launch in AzToolsFramework::AssetBrowser::SourceAssetBrowserEntry::GetSourceControlThumbnailKey #15635

jckand opened this issue Apr 7, 2023 · 1 comment
Assignees
@dtamkin1
Labels
feature/asset-browser This item is related to the asset-browser subsystem. feature/scripting This item is related to the scripting tools. kind/bug Categorizes issue or PR as related to a bug. priority/critical Critical priority. Must be actively worked on as someone's top priority right now. sig/content Categorizes an issue or PR as relevant to SIG Content. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@jckand
Copy link
Contributor

jckand commented Apr 7, 2023

Describe the bug
Crash on launch in AzToolsFramework::AssetBrowser::SourceAssetBrowserEntry::GetSourceControlThumbnailKey.

Steps to reproduce

  1. Launch the Lua Editor.

Expected behavior
Editor opens without issue.

Actual behavior
Lua Editor crashes shortly after launch:

>	LuaIDE.exe!AzToolsFramework::AssetBrowser::SourceAssetBrowserEntry::GetSourceControlThumbnailKey() Line 161	C++
 	LuaIDE.exe!AzToolsFramework::AssetBrowser::EntryDelegate::DrawThumbnail(QPainter * painter, const QPoint & point, const QSize & size, const AzToolsFramework::AssetBrowser::AssetBrowserEntry * entry) Line 165	C++
 	LuaIDE.exe!AzToolsFramework::AssetBrowser::EntryDelegate::paint(QPainter * painter, const QStyleOptionViewItem & option, const QModelIndex & index) Line 107	C++
 	[External Code]	
 	LuaIDE.exe!AzToolsFramework::Framework::Run() Line 283	C++
 	[Inline Frame] LuaIDE.exe!AZStd::Internal::INVOKE(void(LegacyFramework::CoreMessages::*)() &) Line 177	C++
 	[Inline Frame] LuaIDE.exe!AZStd::invoke(void(LegacyFramework::CoreMessages::*)() &) Line 54	C++
 	[Inline Frame] LuaIDE.exe!AZ::EBusEventProcessingPolicy::Call(void(LegacyFramework::CoreMessages::*)() &) Line 437	C++
 	LuaIDE.exe!AZ::Internal::EBusContainer<LegacyFramework::CoreMessages,LegacyFramework::CoreMessages,0,2>::Dispatcher<AZ::EBus<LegacyFramework::CoreMessages,LegacyFramework::CoreMessages>>::Broadcast<void (__cdecl LegacyFramework::CoreMessages::*)(void)>(void(LegacyFramework::CoreMessages::*)() && func) Line 1360	C++
 	LuaIDE.exe!LegacyFramework::Application::Run(const LegacyFramework::ApplicationDesc & desc) Line 223	C++
 	LuaIDE.exe!main(int argc, char * * argv) Line 72	C++
 	LuaIDE.exe!WinMain(HINSTANCE__ * __formal, HINSTANCE__ * __formal, char * __formal, int __formal) Line 97	C++
 	[External Code]

Found in Branch
development

Commit ID from o3de/o3de Repository
b25c961
@jckand jckand added feature/scripting This item is related to the scripting tools. kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/content Categorizes an issue or PR as relevant to SIG Content. feature/asset-browser This item is related to the asset-browser subsystem. labels Apr 7, 2023
@dtamkin1 dtamkin1 added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/critical Critical priority. Must be actively worked on as someone's top priority right now. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Apr 10, 2023
@dtamkin1 dtamkin1 self-assigned this Apr 10, 2023
@dtamkin1 dtamkin1 moved this to 👀 In review in Asset Browser Apr 10, 2023
@dtamkin1 dtamkin1 mentioned this issue Apr 10, 2023
Merged
@AMZN-Dk AMZN-Dk mentioned this issue Apr 10, 2023
Open
30 tasks
@jckand
Copy link
Contributor Author

jckand commented Apr 11, 2023

This issue has been fix verified in stabilization/2305@ a23b797.

@dtamkin1 dtamkin1 moved this from 👀 In review to ✅ Done in Asset Browser Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dtamkin1 dtamkin1

Labels
feature/asset-browser This item is related to the asset-browser subsystem. feature/scripting This item is related to the scripting tools. kind/bug Categorizes issue or PR as related to a bug. priority/critical Critical priority. Must be actively worked on as someone's top priority right now. sig/content Categorizes an issue or PR as relevant to SIG Content. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Asset Browser
Status: ✅ Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@guillaume-haerinck @jckand @dtamkin1