Fixes memory leaks (see brunschgi#72)

spec/utilsSpec.js

@@ -46,6 +46,27 @@ describe('Utils', function () {
 			expect(Module.foo()).toEqual('foo');
 			expect(Module.bar).toEqual('bar');
 		});
+
+		it('should deconstruct the module when calling stop', function () {
+			var Module = T.createModule({
+				name: 'Foo'
+			});
+			var instance = new Module({}, {});
+			spyOn(T.Module.prototype, 'stop');
+			instance.stop();
+			expect(T.Module.prototype.stop).toHaveBeenCalled();
+		});
+
+		it('should always deconstruct the module also if a custom stop function exists', function () {
+			var Module = T.createModule({
+				name: 'Foo',
+				stop: function() {}
+			});
+			var instance = new Module({}, {});
+			spyOn(T.Module.prototype, 'stop');
+			instance.stop();
+			expect(T.Module.prototype.stop).toHaveBeenCalled();
+		});
 	});
 });
 

src/Utils.js

@@ -200,8 +200,19 @@ var Utils = {
 			Utils.extend(Constructor, spec.statics);
 		}
 
+		// Ensure that stop is called even if the spec contains a custom stop function
+		if (spec.hasOwnProperty('stop')) {
+			proto.stop = function() {
+				// Call the spec stop function
+				spec.stop.apply(this, arguments);
+				// Call the original stop function
+				Module.prototype.stop.apply(this, arguments);
+			};
+		}
+
 		var reservedKeys = [
-			'statics'
+			'statics',
+			'stop'
 		];
 
 		// mixin spec properties to module prototype