vlad.os/LuaCsForBarotraumaEP
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Handling and logging exceptions thrown by ItemComponent constructors, added stack traces to a couple of error messages

Browse Source
This commit is contained in:
Joonas Rikkonen
2018-08-06 14:53:07 +03:00
parent 7c93730fc3
commit e60ebc1930
3 changed files with 17 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Barotrauma/BarotraumaClient/Source/Networking/NetEntityEvent/ClientEntityEventManager.cs
+1 -1
View File
@@ -200,7 +200,7 @@ namespace Barotrauma.Networking
catch (Exception e) catch (Exception e)
{ {
string errorMsg = "Failed to read event for entity \"" + entity.ToString() + "\"! (MidRoundSyncing: " + thisClient.MidRoundSyncing + ")\n" + e.StackTrace; string errorMsg = "Failed to read event for entity \"" + entity.ToString() + "\" (" + e.Message + ")! (MidRoundSyncing: " + thisClient.MidRoundSyncing + ")\n" + e.StackTrace;
errorMsg += "\nPrevious entities:"; errorMsg += "\nPrevious entities:";
for (int j = entities.Count - 2; j >= 0; j--) for (int j = entities.Count - 2; j >= 0; j--)
{ {
Barotrauma/BarotraumaShared/Source/Items/Components/ItemComponent.cs
+15 -4
View File
@@ -632,11 +632,22 @@ namespace Barotrauma.Items.Components
return null; return null;
} }
object[] lobject = new object[] { item, element }; ItemComponent ic = null;
object component = constructor.Invoke(lobject); try
{
object[] lobject = new object[] { item, element };
object component = constructor.Invoke(lobject);
ItemComponent ic = (ItemComponent)component; ic = (ItemComponent)component;
ic.name = element.Name.ToString(); ic.name = element.Name.ToString();
}
catch (TargetInvocationException e)
{
DebugConsole.ThrowError("Error while loading entity of the type " + t + ".", e.InnerException);
GameAnalyticsManager.AddErrorEventOnce("ItemComponent.Load:TargetInvocationException" + item.Name + element.Name,
GameAnalyticsSDK.Net.EGAErrorSeverity.Error,
"Error while loading entity of the type " + t + " (" + e.InnerException + ")\n" + Environment.StackTrace);
}
return ic; return ic;
} }
Barotrauma/BarotraumaShared/Source/Map/Entity.cs
+1 -1
View File
@@ -138,7 +138,7 @@ namespace Barotrauma
GameAnalyticsManager.AddErrorEventOnce( GameAnalyticsManager.AddErrorEventOnce(
"Entity.RemoveAll:Exception" + e.ToString(), "Entity.RemoveAll:Exception" + e.ToString(),
GameAnalyticsSDK.Net.EGAErrorSeverity.Error, GameAnalyticsSDK.Net.EGAErrorSeverity.Error,
"Error while removing entity \"" + e.ToString() + "\"" + exception.Message); "Error while removing entity \"" + e.ToString() + " (" + exception.Message + ")\n" + exception.StackTrace);
} }
} }
StringBuilder errorMsg = new StringBuilder(); StringBuilder errorMsg = new StringBuilder();