tyler/Libation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "DataGridView filtering internal NullReferenceException **HACK**"

Browse Source 
This reverts commit e1f749c3da7c9bc3de8dc0a9eb9d482b92476df9.
This commit is contained in:
Michael Bucari-Tovo 2025-02-10 10:19:24 -07:00
parent e1f749c3da
commit 8ec7e5a9d2
1 changed files with 10 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Source/LibationWinForms/MessageBoxLib.cs
View File

@ -12,9 +12,6 @@ namespace LibationWinForms
{ {
public static class MessageBoxLib public static class MessageBoxLib
{ {
private static int nreCount = 0;
private const int NRE_LIMIT = 5;
/// <summary> /// <summary>
/// Logs error. Displays a message box dialog with specified text and caption. /// Logs error. Displays a message box dialog with specified text and caption.
/// </summary> /// </summary>
@ -24,20 +21,6 @@ namespace LibationWinForms
/// <param name="exception">Exception to log.</param> /// <param name="exception">Exception to log.</param>
public static void ShowAdminAlert(System.ComponentModel.ISynchronizeInvoke owner, string text, string caption, Exception exception) public static void ShowAdminAlert(System.ComponentModel.ISynchronizeInvoke owner, string text, string caption, Exception exception)
{ {
// HACK: limited NRE swallowing -- this. is. AWFUL
// I can't figure out how to circumvent the DataGridView internal NRE when:
// * book has tag: asdf
// * filter is `[asdf]`
// * tag asdf is removed from book
// * DataGridView throws NRE
if (exception is NullReferenceException nre && nreCount < NRE_LIMIT)
{
nreCount++;
Serilog.Log.Logger.Error(nre, "Alert admin error. Swallow NRE: {@DebugText}", new { text, caption, nreCount });
return;
}
// for development and debugging, show me what broke! // for development and debugging, show me what broke!
if (System.Diagnostics.Debugger.IsAttached) if (System.Diagnostics.Debugger.IsAttached)
throw exception; throw exception;