[FFmpeg-devel] [PATCH] doc/ffprobe.xsd: regroup and sort elements, realign

Stefano Sabatini stefasab at gmail.com
Tue Jul 31 21:01:15 CEST 2012


On date Tuesday 2012-07-31 16:24:08 +0200, Nicolas George encoded:
> Le quartidi 14 thermidor, an CCXX, Stefano Sabatini a écrit :
> > -            <xsd:element name="packets"  type="ffprobe:packetsType" minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="frames"   type="ffprobe:framesType"  minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="streams"  type="ffprobe:streamsType" minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="format"   type="ffprobe:formatType"  minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="error"    type="ffprobe:errorType"   minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="program_version"  type="ffprobe:programVersionType"  minOccurs="0" maxOccurs="1" />
> > -            <xsd:element name="library_versions" type="ffprobe:libraryVersionsType" minOccurs="0" maxOccurs="1" />
> > +            <xsd:element name="error"            type="ffprobe:errorType"   minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="format"           type="ffprobe:formatType"  minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="frames"           type="ffprobe:framesType"  minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="library_versions" type="ffprobe:libraryVersionsType" minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="packets"          type="ffprobe:packetsType" minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="program_version"  type="ffprobe:programVersionType"  minOccurs="0" maxOccurs="1"/>
> > +            <xsd:element name="streams"          type="ffprobe:streamsType" minOccurs="0" maxOccurs="1"/>
> 
> I like it better when library_versions is near program_versions.
> 
> > -      <xsd:attribute name="codec_type"    type="xsd:string" use="required" />
> > -      <xsd:attribute name="stream_index"  type="xsd:int" use="required" />
> > -      <xsd:attribute name="pts"           type="xsd:long"  />
> > -      <xsd:attribute name="pts_time"      type="xsd:float" />
> > -      <xsd:attribute name="dts"           type="xsd:long"  />
> > -      <xsd:attribute name="dts_time"      type="xsd:float" />
> > -      <xsd:attribute name="duration"      type="xsd:long"  />
> > -      <xsd:attribute name="duration_time" type="xsd:float" />
> > -      <xsd:attribute name="convergence_duration"      type="xsd:long"  />
> > -      <xsd:attribute name="convergence_duration_time" type="xsd:float" />
> > -      <xsd:attribute name="size"          type="xsd:long" use="required" />
> > -      <xsd:attribute name="pos"           type="xsd:long"  />
> > -      <xsd:attribute name="flags"         type="xsd:string" use="required" />
> > +      <xsd:attribute name="codec_type"                type="xsd:string" use="required"/>
> > +      <xsd:attribute name="convergence_duration"      type="xsd:long" />
> > +      <xsd:attribute name="convergence_duration_time" type="xsd:float"/>
> > +      <xsd:attribute name="dts"                       type="xsd:long" />
> > +      <xsd:attribute name="dts_time"                  type="xsd:float"/>
> > +      <xsd:attribute name="duration"                  type="xsd:long" />
> > +      <xsd:attribute name="duration_time"             type="xsd:float"/>
> > +      <xsd:attribute name="flags"                     type="xsd:string" use="required"/>
> > +      <xsd:attribute name="pos"                       type="xsd:long" />
> > +      <xsd:attribute name="pts"                       type="xsd:long" />
> > +      <xsd:attribute name="pts_time"                  type="xsd:float"/>
> > +      <xsd:attribute name="size"                      type="xsd:long"   use="required"/>
> > +      <xsd:attribute name="stream_index"              type="xsd:int"    use="required"/>
> 
> Same here: pts near dts is more logical. Thematic sorting is not as
> self-evident but more practical IMHO.

Well I believe I'll just drop this patch and let ffprobe.xsd rest in
peace.
-- 
FFmpeg = Faithful & Fantastic Mournful Plastic Esoteric Gorilla


More information about the ffmpeg-devel mailing list