Panos Astithas (he/him) [:past] (please ni?) Assignee
	Description • 12 years ago

From bug 772119 comment 64.

Green try push:

https://tbpl.mozilla.org/?tree=Try&rev=bf22677d74a1

	Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]
	Comment 2 • 12 years ago

Comment on attachment 738000 [details] [diff] [review]
Patch v1

Review of attachment 738000 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/devtools/debugger/server/dbg-script-actors.js
@@ +684,1 @@
>                         for (s of this.dbg.findScripts())]);

Nitiest of nits: This line should be re-indented so that the "for" vertically aligns with "this" on the line above.

	Panos Astithas (he/him) [:past] (please ni?) Assignee
	Comment 3 • 12 years ago

(In reply to Nick Fitzgerald [:fitzgen] from comment #2)
> Nitiest of nits: This line should be re-indented so that the "for"
> vertically aligns with "this" on the line above.

I was just about to make it a single line, but OK :-)

	Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]
	Comment 4 • 12 years ago

(In reply to Panos Astithas [:past] from comment #3)
> Created attachment 738034 [details] [diff] [review]
> Patch v2
> 
> (In reply to Nick Fitzgerald [:fitzgen] from comment #2)
> > Nitiest of nits: This line should be re-indented so that the "for"
> > vertically aligns with "this" on the line above.
> 
> I was just about to make it a single line, but OK :-)

Ah yes, that works too :)

	Rob Campbell [:rc] (:robcee)
	Comment 5 • 12 years ago

Comment on attachment 738034 [details] [diff] [review]
Patch v2

Review of attachment 738034 [details] [diff] [review]:
-----------------------------------------------------------------

no comment!

	Panos Astithas (he/him) [:past] (please ni?) Assignee
	Comment 6 • 12 years ago

https://hg.mozilla.org/integration/fx-team/rev/335029350f8e

	Tim Taubert [:ttaubert] (inactive)
	Comment 7 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/335029350f8e