Fix SDF loadFont rejecting promise with target instead of error

[guilhermesimoes]

No description provided.

[guilhermesimoes]

Btw, this has been an issue in our migration from L2 to L3.

Given that in none of the listeners we ever use the first argument (target), would it be possible to change this:

renderer/src/common/EventEmitter.ts

Line 93 in 3015383

listeners[i]!(this, data);

to simply

listeners[i]!(data); 

?
In our app we have never needed access to the target.

Usually we do:

this.on(someEvent => {
  // here we have access to `this`
});

Or we do:

const child = this.child;
child.on(someEvent => {
  // here we have access to `child`
});

Don't see why we have to receive the target as the first argument.

[jfboeve]

maybe we can implement this for v4.0 @wouterlucas? internally in the renderer I don't encounter use of target either?

[wouterlucas]

@jfboeve sure!