As a further improvement to Luke's changes, noted here a couple days ago, I 
just added the Value interface stuff to Rooted<Value>, EncapsulatedValue, and 
MutableHandle<Value>.

As an added bonus, the additions only replicate the isInt32, isBoolean, 
setObject, etc. methods once -- so they exist in Value, and in only one other 
place for all these other Value-looking classes.  So if you find any methods 
are missing, you can add them to all these classes all at once, rather than 
having to update N different lists.

If you're curious how this was implemented, two pointers: 1) CRTP, and 2) bug 
773049.

Jeff
_______________________________________________
dev-tech-js-engine-internals mailing list
dev-tech-js-engine-internals@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-tech-js-engine-internals

Reply via email to