Minor change to set_drawFramerate

[SimonArs]

Maybe its better to change drawFramerate setter a bit?

From:

static function set_drawFramerate(value:Int):Int
{
	if (value > updateFramerate)
		log.warn("FlxG.drawFramerate: the update framerate shouldn't be smaller than the draw framerate," + " since it can stop your game from updating.");

	drawFramerate = Std.int(Math.abs(value));

	if (game.stage != null)
		game.stage.frameRate = drawFramerate;

	game._maxAccumulation = 2000 / drawFramerate - 1;

	if (game._maxAccumulation < game._stepMS)
		game._maxAccumulation = game._stepMS;

	return value;
}

to:

static function set_drawFramerate(value:Int):Int
{
	if (value > updateFramerate)
		log.warn("FlxG.drawFramerate: the update framerate shouldn't be smaller than the draw framerate," + " since it can stop your game from updating.");

	value = Std.int(Math.abs(value));

	if (game.stage != null)
		game.stage.frameRate = value;

	game._maxAccumulation = 2000 / value - 1;

	if (game._maxAccumulation < game._stepMS)
		game._maxAccumulation = game._stepMS;

	return drawFramerate = value;
}

[Geokureli]

Can you be specific about what issue this would fix, please provide examples

[SimonArs]

This doesn't fix user-visible bug by itself. My intention was to avoid mutating static field until all dependent state like stage.frameRate and _maxAccumulation has been updated, so setter commits new value only after it has been fully applied. It's mostly small consistency/readability improvement rather than functional fix

[Geokureli]

sure, yeah

[Geokureli]

If you wanna make this exact PR I would accept, otherwise I'll make it when I get some time