← Back to team overview

lightspark-users team mailing list archive

Code review / @spec-checked tag

 

Hey,

currently it's quite hard to see which parts of AS3/ecma are
implemented and which are missing.
There are already a lot of LOG_NOT_IMPLEMENTED in the code, but not everywhere.

I therefore propose to do a (incremental) code review of the whole
codebase. For each AS3/ecma function
(like XMLList.valueOf or DisplayObject.parent etc), we add a
/* @spec-checked ECMA-357 10.1.8 */
(or something like this) tag
if the functionality exactly matches the description in the
specification. (For ecma methods, one can also add the section in the
spec like ecma 1.4.16.7, for as3 methods there is usually no better
spec than the adobe online help)

If there are missing parts, hacks, or wrong implementations, they
should also be flagged as such (probably by a LOG_NOT_IMPLEMENTED).

This should make the codebase more sane and reliable on the one hand,
and debugging easier (due to more LOG_NOT_IMPLEMENTED) on the other
hand.

What are your thoughs on this?

Reference:
https://github.com/lightspark/lightspark/issues/163