libdvdread video_attr_t defines

[Not subscribed, please maintain CC.]

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