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

[Nicolas George]

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120731/8f8dc9c9/attachment.asc>