Remove sarcastic comments

richvdh · richvdh · commit a05bafed6af9 · 2017-06-14T23:03:40.000+01:00
... apparently I wrote them, and I don't understand them. There is *supposed* to be a global flux dispatcher. This reverts commit c0de087.
diff --git a/src/dispatcher.js b/src/dispatcher.js
@@ -41,9 +41,6 @@ class MatrixDispatcher extends flux.Dispatcher {
     }
 }
 
-// XXX this is a big anti-pattern, and makes testing hard. Because dispatches
-// happen asynchronously, it is possible for actions dispatched in one thread
-// to arrive in another, with *hilarious* consequences.
 if (global.mxDispatcher === undefined) {
     global.mxDispatcher = new MatrixDispatcher();
 }

Original file line number	Diff line number	Diff line change
`@@ -41,9 +41,6 @@ class MatrixDispatcher extends flux.Dispatcher {`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`
`44`		`-// XXX this is a big anti-pattern, and makes testing hard. Because dispatches`
`45`		`-// happen asynchronously, it is possible for actions dispatched in one thread`
`46`		`-// to arrive in another, with hilarious consequences.`
`47`	`44`	`if (global.mxDispatcher === undefined) {`
`48`	`45`	`global.mxDispatcher = new MatrixDispatcher();`
`49`	`46`	`}`