libdvdread video_attr_t defines

[sorry if this is a dup, sending again now that I'm subscribed]

Are there any plans to expand dvdread/ifo_types.h to include #defines
for video attributes?  For example, ifoPrint_video_attributes() has
switch statements for most fields, but the values are hard coded in the
function.  Outside programs using libdvdread have to interpret the
fields on their own.  I think a set of defines (DVDREAD_VATTR_*) would
enhance libdvdread a lot.  I will consider sending in a patch to do
this, so please let me know if there are any particular names/convention
you would like to use.

FYI, I am writing a Perl XS module for libdvdread.  I have already
written enough of the XS module to port the 3 of the included utils to
Perl (disc_id, ifo_dump, and title_info), as well as write some of my
own tools.

-James