added a comment - - edited
Do you mean the enum functions are missing the phpdocs? they are deprecated as such to be removed (since 2.1 actually). Thus i left the phpdoc out as we really don't want them to be used.
Actually, good that you mentioned that, i'll add the @todo to these deprecated functons referencing
1: http://docs.moodle.org/dev/Coding_style#.40package says @subpackage use is optional hence i used it to distinguish between ddl API and ddl implementations which aren't our API as they are instantiations of our abstract API.
2: @package should be 'core' as 'ddl' atm isn't a subsystem in get_core_subsystems() as mentioned in http://docs.moodle.org/dev/Coding_style#.40package
3. thanks for that, i'll clean that up
ok thats all cleaned up and rebased. pushing up for integration now.