History Bug

[Illnab1024]

I think this is in plugins without icons that use token dialogs, although I know that Conditional Hue/Saturation does it:

This text file was created because Paint.NET crashed.
Please e-mail this file to paint.net@hotmail.com so we can diagnose and fix the problem.

Application version: Paint.NET v3.0 (Milestone 2 Release build 3.0.2472.34062)
Time of crash: 10/11/2006 2:22:04 AM
Application uptime: 01:36:13.3281250
OS Version: 5.1.2600.131072 Service Pack 2 Workstation x86
.NET Framework version: 2.0.50727.42 x86
Processor: 1x Intel(R) Pentium(R) 4 CPU 2.53GHz
Physical memory: 639 MB
Locale: Setting: en-USCI.CC: en-US, CC.CUIC: en-US

Exception details:
System.NullReferenceException: Object reference not set to an instance of an object.
  at PaintDotNet.HistoryControl.OnPaint(PaintEventArgs e)
  at System.Windows.Forms.Control.PaintWithErrorHandling(PaintEventArgs e, Int16 layer, Boolean disposeEventArgs)
  at System.Windows.Forms.Control.WmPaint(Message& m)
  at System.Windows.Forms.Control.WndProc(Message& m)
  at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
  at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
  at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
------------------------------------------------------------------------------

After you run the effect, the history dialog gets a big red X and PdN crashes.

[Rick Brewster]

Oh, duh. Probably because I assumed all history items would have an icon. Woops!

[BoltBait]

If only we could specify an icon with codelab...

Hmmm... I'm going to have to look at the source code for that thing...

[Illnab1024]

You know, that actually wouldn't be too hard; you woul just have to write the filename into the base class override, no? And perhaps verify the file...