Description Wim Jongman 2021-01-17 06:02:58 EST

As a follow up to Bug 570094 I found myself creating yet another resource factory to manage "one-off" resource leaks.

JFace provides ResourceManager and *Registry but that API is 

a) intended for long term resource caching.
b) complicated


Instead of 

setTitleImage(new Image(display, inputStream))); // resource leak

Use

setTitleImage(Factory.create(someControl, inputStream)));

// Factory class replacing this specific Image constructor
public static Image create(Control control, InputStream inputStream) {
	Image image = new Image(control.getDisplay(), inputStream);
	control.addDisposeListener(e -> image.dispose());
	return image;
}