Remove old panel if ModuleWidget::setPanel is called multiple times,

move in2px and mm2px to widgets.hpp

include/app.hpp

@@ -14,6 +14,7 @@ struct Wire;
 struct RackWidget;
 struct ParamWidget;
 struct Port;
+struct SVGPanel;
 
 ////////////////////
 // module
@@ -31,6 +32,7 @@ struct ModuleWidget : OpaqueWidget {
 	/** Owns the module pointer */
 	Module *module = NULL;
 
+	SVGPanel *panel = NULL;
 	std::vector<Port*> inputs;
 	std::vector<Port*> outputs;
 	std::vector<ParamWidget*> params;

include/rack.hpp

@@ -18,15 +18,6 @@ namespace rack {
 ////////////////////
 
 
-inline Vec in2px(Vec inches) {
-	return inches.mult(SVG_DPI);
-}
-
-inline Vec mm2px(Vec millimeters) {
-	return millimeters.mult(SVG_DPI / 25.4);
-}
-
-
 template <class TModuleWidget>
 Model *createModel(std::string manufacturerSlug, std::string manufacturerName, std::string slug, std::string name) {
 	struct TModel : Model {

include/widgets.hpp

@@ -16,6 +16,15 @@
 namespace rack {
 
 
+inline Vec in2px(Vec inches) {
+	return inches.mult(SVG_DPI);
+}
+
+inline Vec mm2px(Vec millimeters) {
+	return millimeters.mult(SVG_DPI / 25.4);
+}
+
+
 ////////////////////
 // resources
 ////////////////////

src/app/ModuleWidget.cpp

@@ -43,7 +43,13 @@ void ModuleWidget::addParam(ParamWidget *param) {
 }
 
 void ModuleWidget::setPanel(std::shared_ptr<SVG> svg) {
-	SVGPanel *panel = new SVGPanel();
+	// Remove old panel
+	if (panel) {
+		removeChild(panel);
+		panel = NULL;
+	}
+
+	panel = new SVGPanel();
 	panel->setBackground(svg);
 	addChild(panel);