Breaking change from #201 not noted as such in changelog

[gsnedders]

#201 breaks idlharness.js, because extAttrs is no longer an array.

[saschanaz]

Right, but not sure how to fix this for github changelog generator...

BTW, I'd like to restore it as an array to remove the need of painful existence check. The problem is it needs array subclassing, do you think it's okay to do? I somehow am hesitant with no actual reason 🤔 @marcoscaceres

[gsnedders]

Unsurprising, plenty of the trivia changes are breaking, and I keep finding more. Bleh. :/

interface Foo : Bar now has an object for IdlInterface.inheritance, too, and array subclassing won't fix that.

[marcoscaceres]

BTW, I'd like to restore it as an array to remove the need of painful existence check. The problem is it needs array subclassing, do you think it's okay to do? I somehow am hesitant with no actual reason 🤔 @marcoscaceres

I don't have a strong opinion... if it lessens some pain for @gsnedders on the WPT side, then probably worth doing.

@gsnedders, how bad are the breakages?

[gsnedders]

@marcoscaceres They're all over the place, and without any static types I scarcely know how to find all the places where things have changed to be objects to include tokens etc., but as noted in my previous comment, there are things which have changed from string to an object, and that's a bigger change that can't be so easily be worked around.

[marcoscaceres]

@gsnedders, sorry, one last bit of info: what's using WebIDL2.js on the WPT side? Is it IDLHarness?

[gsnedders]

@marcoscaceres yes

[saschanaz]

plenty of the trivia changes are breaking

I recently reverted many of those string-to-object things. Might be easier to jump to the latest version.

[saschanaz]

c8cca88 marked it as a breaking change.