On Monday 16 January 2012 15:09:06 Kamil Debski wrote: > Hi Hans, > > > From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] > > Sent: 16 January 2012 14:36 > > > > Hi all, > > > > This patch fixes several control names with inconsistent capitalization > > and other inconsistencies (and a spelling mistake in one name as well). > > > > Kamil, Sakari, please take a look as most of the affected strings are > > either MPEG or Flash controls. > > Thank you for your patch. > I've had a look at the codec controls and all seems fine. > > > Note that I saw a few strings as well that are longer then 31 characters. > > Those will be cut off when returns in queryctrl. I'm not sure yet what to > > do about those. > > I think it's sensible to abbreviate them. You can find one suggestion > below. ... > > + case V4L2_CID_MPEG_VIDEO_MAX_REF_PIC: return "Max > > Number of Reference Pictures"; > > This could be "Max Number of Reference Pics" or "Max Number of Ref Pictures". I'm going with the first one. I'll also change "Minimum Number of Capture Buffers" to "Min Number of Capture Buffers". Ditto for Output Buffers. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
