SwtDebug.dumpEvent is now null-safe.

Author: nedtwigg

CHANGES.md

@@ -10,6 +10,7 @@
 - Added `ControlWrapper.setParent` which is a pure delegation to `Control.setParent`.
 - Improved `StructuredDrop.handler`.
 - Added `SwtMisc.globalBounds` for `ControlWrapper`.
+- `SwtDebug.dumpEvent` is now null-safe.
 
 ### Version 3.2.1 - August 7th 2019 ([javadoc](http://diffplug.github.io/durian-swt/javadoc/3.2.1/), [jcenter](https://bintray.com/diffplug/opensource/durian-swt/3.2.1/view))
 

durian-swt/src/main/java/com/diffplug/common/swt/SwtDebug.java

@@ -43,12 +43,16 @@
  */
 public class SwtDebug {
 	/** Dumps the given SWT event to System.out. */
-	public static void dumpEvent(String name, Event e) {
+	public static void dumpEvent(String name, @Nullable Event e) {
 		dumpEvent(name, e, StringPrinter.systemOut());
 	}
 
 	/** Dumps the given SWT event to the given StringPrinter. */
-	public static void dumpEvent(String name, Event e, StringPrinter to) {
+	public static void dumpEvent(String name, @Nullable Event e, StringPrinter to) {
+		if (e == null) {
+			to.println(name + ": null");
+			return;
+		}
 		// print the name
 		to.println(name + ": " + eventType(e));
 		// print the non-null / non-zero fields