update() broken

[schulzch]

The update method has weird logic (see below). Moreover, it does not return this, as suggested in the documentation. If you want to keep setter and getter merged, you could test for arguments.length === 0 to return the property and this otherwise.

		update: function(redraw){
			if (redraw === true)
				this._redrawEnabled = false;

			if (redraw === false)
				this._redrawEnabled = true;

			return !!this._redrawEnabled;
		},

This simplifies to this._redrawEnabled = !redraw, which seems wrong.

[otm]

I must admit that I will not have time to fix this any time soon due to other projects, but a PR is welcome. I also must say that this function is not well designed.

I'm not a fan of breaking changes, I would probably just add a deprecation to the function and make a new function for example suspendRedraw() which could return this, and then end it by calling redraw().