Bah, I'm an idiot. Hit Ctrl+Enter by accident. Heres the bugzilla entry with the patch:
https://bugzilla.gnome.org/show_bug.cgi?id=627033 On Mon, 2010-08-16 at 12:47 +0100, Daniel Hams wrote: > Hi all, > > Just a quick mail to announce to those interested I have put a patch in > bugzilla that refactors the .metadata handling in vapigen. Patch applies > to HEAD. > > > Changes to vapigen: > > * Check if the metadata attributes (is_array, ellipsis etc) are > applicable to the symbol they are placed on. If they are not, a warning > is generated > > * Enumerate explicitly all metadata attribute keywords inside the vala > code > > * Centralise parsing/split to double/int/comma delimited etc into one place > > * Allow output of a symbols file containing a list of valid attribute keys > for each symbol discovered during execution > > I have regression tested the changes against the existing .vapi packages in > git and the changes pass regeneration tests there. > > For reference, this patched vapigen did actually find some .metadata errors > in the existing git package .metadata files - so it seems to be useful. > > Cheers, > > Dan _______________________________________________ vala-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/vala-list
