DadOfMrLog wrote:

drmcw wrote:

So who will be first to up the clone limit or remove it?

Oh, that one looks easy enough…

In src/primitives/Primitives.as:

public class Primitives {

private const MaxCloneCount:int = 300;
…



One of the first things I'd be happy to do would be:

public function keepOnStage():void {
// removed everything from here…
return;
}

speakvisually wrote:

We've open sourced the Scratch 2.0 editor code! You can find the code in the following repository on Github, https://github.com/LLK/scratch-flash

Scratch On!
Eric

DadOfMrLog wrote:

Woah, excellent! Never done anything with Flash/ActionScript before, but maybe I should have a go. (Oh for more hours in the day…)


After a quick scan through, I can already see a number of simple things that might be worth doing.
e.g. some basic checks to prevent unnecessary work if the new direction/size are same as previous:


public function setSize(percent:Number):void {
…
var newscale = Math.max(minScale, Math.min(percent / 100.0, maxScale));
if (scaleX == newscale) return;
scaleX = scaleY = newscale;
…


public function setDirection(d:Number):void {
…
d = (d > 180) ? d - 360 : d;
if (direction == d) return;
direction = d;
…
clearCachedBitmap(); <— will adding this deal with the rotated sprite touching bug…?
updateBubble(); <— also note that speech bubble should change when rotated
}


Would above really ‘just work’ in the way I think, or am I missing something…?

DadOfMrLog wrote:

Whoa! Custom reporters!
-Certainly looking forward to those. Was wondering how the actual “report” might work…

Nice to see the speech bubbles for reporter values also making a comeback.

(Oh, yes, and a bugtracker, too…)

griffpatch wrote:

local variables within custom blocks…

griffpatch wrote:

Also, I often want to return more than one value from a custom block, so it might not make a huge difference to me after all.

griffpatch wrote:

The bug fix around set var and change var always missing their caches interested me a lot… I wonder if that will speed things up some. I do wish that list manipulation could be sped up though.

griffpatch wrote:

I second that! I also saw the potential for not re-rasterizing costumes when they are not rotated or scaled… Would be excellent, and would save a few lines of script every time I change costume to manually do the check

public function showCostume(costumeIndex:Number):void {
	...
	var c:ScratchCostume = currentCostume();
	if (c == lastCostume) return; // optimization: already showing that costume
	...
}

public function showCostume(costumeIndex:Number):void {
	if (isNaNOrInfinity(costumeIndex)) costumeIndex = 0;
	var newIndex = costumeIndex % costumes.length;
	if (newIndex < 0) newIndex += costumes.length;
	if (newIndex == currentCostumeIndex) return;  // <--- optimise: already this costume
	currentCostumeIndex = newIndex;
	...

griffpatch wrote:

So… who can find and fix the Chrome pepflashplayer keyboard lag issues now!

public function stepThreads():void {
	startTime = getTimer();
	var workTime:int = (0.75 * 1000) / app.stage.frameRate; // work for up to 75% of one frame time
	...

move () steps
say [yay]
change [ v] effect by (25)